Re: [PATCH] posix-timers: Remove the stack variable clockid_t introduced as part of Protect posix clock array access against speculation

[Thomas Gleixner]

On Wed, 1 Aug 2018, Aashish Lakhwara wrote:

> Subject: [PATCH] posix-timers: Remove the stack variable clockid_t introduced as part of
    Protect posix clock array access against speculation

Documentation/process/submitting-patches.rst:

  For these reasons, the ``summary`` must be no more than 70-75
  characters, and it must describe both what the patch changes, as well
  as why the patch might be necessary.  It is challenging to be both
  succinct and descriptive, but that is what a well-written summary
  should do.

The above $subject violates all of these rules.

> The "array_index_mask_nospec" code has been updated to allow index
> argument to have const-qualified type.

> Now the stack variable "idx" is no longer required and can be removed.
> We should directly pass the const variable "id" to array_index_mask_nospec

We should? Changelogs are about facts. So having:

  The "array_index_mask_nospec" code has been updated to allow index
  argument to have const-qualified type.

  So the stack variable "idx" which was introduced in commit 19b558db12f9
  ("posix-timers: Protect posix clock array access against speculation") to
  cast the const argument 'id' is not longer required.

is factual and precise. Hmm?

> Signed-off-by: Aashish Lakhwara <aashish.l@xxxxxxxxxxx>
> Signed-off-by: Himanshu Shukla <himanshu.sh@xxxxxxxxxxx>

This SOB chain is wrong. Please read and follow the Docuentation.

Thanks,

	tglx