Re: [PATCH 07/10] bootmem: add free_bootmem_late

From: FUJITA Tomonori
Date: Mon Nov 09 2009 - 14:25:26 EST


On Sun, 8 Nov 2009 11:00:29 +0100
Ingo Molnar <mingo@xxxxxxx> wrote:

>
> * FUJITA Tomonori <fujita.tomonori@xxxxxxxxxxxxx> wrote:
>
> > On Wed, 28 Oct 2009 08:48:32 +0100
> > Ingo Molnar <mingo@xxxxxxx> wrote:
> >
> > >
> > > * FUJITA Tomonori <fujita.tomonori@xxxxxxxxxxxxx> wrote:
> > >
> > > > From: Chris Wright <chrisw@xxxxxxxxxxxx>
> > > >
> > > > Add a new function for freeing bootmem after the bootmem allocator has
> > > > been released and the unreserved pages given to the page allocator.
> > > > This allows us to reserve bootmem and then release it if we later
> > > > discover it was not needed.
> > > >
> > > > Reviewed-by: FUJITA Tomonori <fujita.tomonori@xxxxxxxxxxxxx>
> > > > Signed-off-by: Chris Wright <chrisw@xxxxxxxxxxxx>
> > > > ---
> > > > include/linux/bootmem.h | 1 +
> > > > mm/bootmem.c | 43 ++++++++++++++++++++++++++++++++++++++-----
> > > > 2 files changed, 39 insertions(+), 5 deletions(-)
> > >
> > > Hm, we are now further complicating the bootmem model.
> >
> > Yeah, agreed. But reorganizing the allocator during boot is not easy.
> > I'll investigate it later on but VT-d people want to fix this IOMMU
> > issue now. Can we accept this for now?
>
> Since the patchset weighs heavily towards the iommu and x86 side i can
> do that in tip:core/iommu tree, if there's an Acked-by for this
> bootmem.c patch from at least one of these gents:

ok

> Johannes Weiner <hannes@xxxxxxxxxxx>
> Pekka Enberg <penberg@xxxxxxxxxxxxxx>
> Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
> Tejun Heo <tj@xxxxxxxxxx>

Can you give ACKs?


Thanks,
--
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/