Re: [PATCH 3.2 056/185] mm: ensure get_unmapped_area() returnshigher address than mmap_min_addr

From: Ben Hutchings
Date: Sun Feb 09 2014 - 13:19:25 EST


On Mon, 2014-01-06 at 19:19 +0900, Akira Takeuchi wrote:
> On Fri, 03 Jan 2014 04:26:43 +0000
> Ben Hutchings <ben@xxxxxxxxxxxxxxx> wrote:
>
> > On Sun, 2013-12-29 at 03:08 +0100, Ben Hutchings wrote:
> > > 3.2.54-rc1 review patch. If anyone has any objections, please let me know.
> > >
> > > ------------------
> > >
> > > From: Akira Takeuchi <takeuchi.akr@xxxxxxxxxxxxxxxx>
> > >
> > > commit 2afc745f3e3079ab16c826be4860da2529054dd2 upstream.
> > [...]
> > > [bwh: Backported to 3.2:
> > > As we do not have vm_unmapped_area(), make arch_get_unmapped_area_topdown()
> > > calculate the lower limit for the new area's end address and then compare
> > > addresses with this instead of with len. In the process, fix an off-by-one
> > > error which could result in returning 0 if mm->mmap_base == len.]
> >
> > I'm dropping this as I have no good way to test the backport (it's not
> > used on x86) and I didn't get any confirmation that it's right.
>
> I'm sorry for delayed reply.
>
> Your backport seems right.
> Additionally, I've confirmed the problem is resolved by your backport patch.

Thanks, I've re-queued it for 3.2.55.

Ben.

--
Ben Hutchings
If at first you don't succeed, you're doing about average.

Attachment: signature.asc
Description: This is a digitally signed message part