Re: [patch 4/4] FUTEX: Tidy up the code

From: Ingo Molnar
Date: Fri Jun 08 2007 - 03:11:40 EST



* Thomas Gleixner <tglx@xxxxxxxxxxxxx> wrote:

> The recent PRIVATE and REQUEUE_PI changes to the futex code made it hard to read.
> Tidy it up.
>
> Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>

looks good to me:

Acked-by: Ingo Molnar <mingo@xxxxxxx>

> - if (!pi) {
> - if (uval & FUTEX_WAITERS)
> + if (!pi && (uval & FUTEX_WAITERS))
> futex_wake(uaddr, &curr->mm->mmap_sem, 1);
> - }

one too much tab left in the futex_wake line i guess?

Ingo
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/