Re: [PATCH] robust futex thread exit race

From: David Miller
Date: Sun Sep 30 2007 - 19:42:16 EST


From: Ingo Molnar <mingo@xxxxxxx>
Date: Sun, 30 Sep 2007 21:55:38 +0200

>
> * Martin Schwidefsky <schwidefsky@xxxxxxxxxx> wrote:
>
> > On Sun, 2007-09-30 at 19:11 +0200, Ingo Molnar wrote:
> > > > > > good catch! A quick preliminary review of your patch indicates it's
> > > > > > fine - and it might be v2.6.23 material.
> > > > >
> > > > > there's a symmetric bug in kernel/compat_futex.c too.
> > > >
> > > > True. I'll update the patch and send it again with the Acked-bys.
> > >
> > > thanks! Logistics-wise: given that v2.6.22 has this same bug too, it's
> > > not a .23 regression per se and could in theory be merged after the
> > > 2.6.23 release too, into 2.6.23.1.
> >
> > Here we go. Tested with current git and latest glibc tst-robust8 on
> > s390-64, native and compat.
>
> excellent! This patch too looks good to me. I booted and tested it on
> x86 too (32-bit-native, 64-bit-native and 32-bit-compat-on-64-bit.)

I would personally prefer it if this patch got scheduled into
2.6.23, it is a bug fix and it might even account for some of
the futex weirdness we're still seeing on sparc64.

I'll toss this patch to some folks seeing those issues for testing.

But even if it doesn't fix their bug, it still belongs in 2.6.23 and
there is no reason to delay this fix.
-
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/