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

Ingo Molnar
2009-11-08T10:01:12Z

* FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> wrote:

> On Wed, 28 Oct 2009 08:48:32 +0100
> Ingo Molnar <mingo@elte.hu> wrote:
>
> >
> > * FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> wrote:
> >
> > > From: Chris Wright <chrisw@sous-sol.org>
> > >
> > > 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@lab.ntt.co.jp>
> > > Signed-off-by: Chris Wright <chrisw@sous-sol.org>
> > > ---
> > > 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:

Johannes Weiner <hannes@cmpxchg.org>
Pekka Enberg <penberg@cs.helsinki.fi>
Andrew Morton <akpm@linux-foundation.org>
Tejun Heo <tj@kernel.org>

Thanks,

Ingo