> On 04/09/2010 05:13 PM, John David Anglin wrote:
> > On Fri, 09 Apr 2010, Carlos O'Donell wrote:
> >> For example, are the futex fixes anywhere for Kyle to pickup?
> >
> > The futex fixes are Helge's and were posted to the list on Wed,
> > 03 Feb 2010 23:03:49 +0100 along with Helge's minifail3.c.
>
> Yes, but there is still a bug in the patch I sent, and it's still in Dave's big
> patchset. See below...
>
> --- a/arch/parisc/include/asm/atomic.h
> +++ b/arch/parisc/include/asm/atomic.h
> #else
> # define _atomic_spin_lock_irqsave(l,f) do { local_irq_save(f); } while (0)
> # define _atomic_spin_unlock_irqrestore(l,f) do { local_irq_restore(f); } while (0)
> +# define _atomic_spin_lock_irqsave_user(l,f) _atomic_spin_lock_irqsave(l,f)
> +# define _atomic_spin_unlock_irqrestore_user(l,f) _atomic_spin_lock_irqsave_user(l,f)
>
> atomic_spin_lock_irqsave_user() is wrong.
> It needs to be:
> -> atomic_spin_lock_irqsave_user(l,f) _atomic_spin_lock_irqsave_user(l,f)
Huh? I see the following line is wrong...
Dave
--
J. David Anglin dave.anglin@xxxxxxxxxxxxxx
National Research Council of Canada (613) 990-0752 (FAX: 952-6602)
--
To unsubscribe from this list: send the line "unsubscribe linux-parisc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
