futex: rename field in futex_q to clarify single waiter semantics
authorDarren Hart <dvhltc@us.ibm.com>
Thu, 18 Dec 2008 01:29:56 +0000 (17:29 -0800)
committerIngo Molnar <mingo@elte.hu>
Thu, 18 Dec 2008 10:42:23 +0000 (11:42 +0100)
commit73500ac545d24610eb2cf8579ffc88957e9c5847
tree1e36cc5585bd25a1d8cbe973d229250bee0cef32
parent1acdac104668a0834cfa267de9946fac7764d486
futex: rename field in futex_q to clarify single waiter semantics

Impact: simplify code

I've tripped over the naming of this field a couple times.

The futex_q uses a "waiters" list to represent a single blocked task and
then calles wake_up_all().

This can lead to confusion in trying to understand the intent of the code,
which is to have a single futex_q for every task waiting on a futex.

This patch corrects the problem, using a single pointer to the waiting
task, and an appropriate call to wake_up, rather than wake_up_all.

Compile and boot tested on an 8way x86_64 machine.

Signed-off-by: Darren Hart <dvhltc@us.ibm.com>
Acked-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
kernel/futex.c