Messages in this thread | | | Subject | Re: [PATCH 1/5] futex: Misc cleanups | From | Darren Hart <> | Date | Fri, 22 Nov 2013 22:52:53 -0800 |
| |
On Fri, 2013-11-22 at 16:56 -0800, Davidlohr Bueso wrote: > From: Jason Low <jason.low2@hp.com> > > - Remove unnecessary head variables. > - Delete unused parameter in queue_unlock(). > > Cc: Ingo Molnar <mingo@kernel.org> > Cc: Darren Hart <dvhart@linux.intel.com> > Cc: Peter Zijlstra <peterz@infradead.org> > Cc: Thomas Gleixner <tglx@linutronix.de> > Cc: Mike Galbraith <efault@gmx.de> > Cc: Jeff Mahoney <jeffm@suse.com> > Cc: Linus Torvalds <torvalds@linux-foundation.org> > Cc: Scott Norton <scott.norton@hp.com> > Cc: Tom Vaden <tom.vaden@hp.com> > Cc: Aswin Chandramouleeswaran <aswin@hp.com> > Cc: Waiman Long <Waiman.Long@hp.com> > Signed-off-by: Jason Low <jason.low2@hp.com> > Signed-off-by: Davidlohr Bueso <davidlohr@hp.com>
My concern was that we saved off head for a reason, but the hb lock is held in these cases, and the call is safe against removal, so I don't see a problem. Thanks.
Reviewed-by: Darren Hart <dvhart@linux.intel.com>
-- Darren Hart Intel Open Source Technology Center Yocto Project - Linux Kernel
| |