lkml.org 
[lkml]   [2017]   [Feb]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH -v4 03/10] futex: Cleanup variable names for futex_top_waiter()
On Fri, Dec 16, 2016 at 04:13:10PM -0800, Darren Hart wrote:
> On Tue, Dec 13, 2016 at 09:36:41AM +0100, Peter Zijlstra wrote:
> > futex_top_waiter() returns the top-waiter on the pi_mutex. Assinging
> > this to a variable 'match' totally obscures the code.
> >
>
> Yes please.
>
> One wording nit...
>
> > Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
> > ---
> > kernel/futex.c | 30 +++++++++++++++---------------
> > 1 file changed, 15 insertions(+), 15 deletions(-)
> >
> > --- a/kernel/futex.c
> > +++ b/kernel/futex.c
>
> > @@ -1317,11 +1317,11 @@ static int wake_futex_pi(u32 __user *uad
> >
> > /*
> > * It is possible that the next waiter (the one that brought
> > - * this owner to the kernel) timed out and is no longer
> > + * top_waiter owner to the kernel) timed out and is no longer
> > * waiting on the lock.
> > */
>
> This breaks my parser (and did before too).
>
> Consider:
>
> /*
> * It is possible that the next waiter (that caused top_waiter to call
> * into the kernel) has since timed out and is no longer waiting on the
> * lock.
> */
>
> Is that clearer?

No :-)

Fear not, this wee comment here is what the next many patches are about.
This is the 'little' hole that opens up sooo much pain. In any case,
fear not, the comment will die later on.

\
 
 \ /
  Last update: 2017-02-22 15:07    [W:0.069 / U:0.732 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site