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

FUJITA Tomonori
2009-11-09T19:25:11Z

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

>
> * 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:

ok

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

Can you give ACKs?

Thanks,