Messages in this thread Patch in this message | | | From | Davidlohr Bueso <> | Subject | [PATCH 2/5] futex: Check for pi futex_q only once | Date | Fri, 22 Nov 2013 16:56:34 -0800 |
| |
All wake_futex() callers already verify that the we are not dealing with a pi futex_q, so we can remove the redundant WARN() check, as this is never triggered anyway.
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> Cc: Jason Low <jason.low2@hp.com> Signed-off-by: Davidlohr Bueso <davidlohr@hp.com> --- kernel/futex.c | 3 --- 1 file changed, 3 deletions(-)
diff --git a/kernel/futex.c b/kernel/futex.c index e6ffe73..0768c68 100644 --- a/kernel/futex.c +++ b/kernel/futex.c @@ -844,9 +844,6 @@ static void wake_futex(struct futex_q *q) { struct task_struct *p = q->task; - if (WARN(q->pi_state || q->rt_waiter, "refusing to wake PI futex\n")) - return; - /* * We set q->lock_ptr = NULL _before_ we wake up the task. If * a non-futex wake up happens on another CPU then the task -- 1.8.1.4
| |