Re: [patch] PI-futex: -V2

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Ingo Molnar <[email protected]> wrote:
>
> > > +	for (;;) {
>  > > +		if (top_waiter)
>  > > +			plist_del(&top_waiter->pi_list_entry,
>  > > +				  &owner->pi_waiters);
>  > > +
>  > > +		if (waiter && waiter == rt_mutex_top_waiter(lock)) {
>  > 
>  > rt_mutex_top_waiter() can never return NULL, so the test for NULL 
>  > could be removed.
> 
>  it might be NULL if adjust_pi_chain() is called from remove_waiter(), 
>  and next_waiter there is NULL (because !rt_mutex_has_waiters() after the 
>  removal of the current waiter).

Yes, `waiter' might be NULL.  But rt_mutex_top_waiter() will never return
NULL.  So it might be possible to just do

	if (waiter == rt_mutex_top_waiter(lock))

Which might actually be less efficient, and more obscure.  Just pointing it
out.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux