Re: [PATCH v5] futex: Fix regression with read only mappings

From: Thomas Gleixner
Date: Mon Jul 25 2011 - 15:29:18 EST


On Mon, 25 Jul 2011, Shawn Bohrer wrote:
> On Thu, Jun 30, 2011 at 11:21:32AM -0500, Shawn Bohrer wrote:
> > commit 7485d0d3758e8e6491a5c9468114e74dc050785d (futexes: Remove rw
> > parameter from get_futex_key()) in 2.6.33 fixed two problems: First, It
> >
> > - ret = get_futex_key(uaddr2, flags & FLAGS_SHARED, &key2);
> > + ret = get_futex_key(uaddr2, flags & FLAGS_SHARED, &key2, VERIFY_WRITE);
> > if (unlikely(ret != 0))
> > goto out;
> >
>
> Thomas,
>
> What do I need to do to get this patch accepted?

First of all, please use proper quoting. It's a nuisance to page down
10 times to find the interesting new text of a mail :)

Seriously, I'm sorry, it fell from my radar. Thanks for the
reminder. It's on top of my tomorrow morning (brain usable) todo list
now.

Thanks,

tglx
--
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/