[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20080410173951.GA9482@wotan.suse.de>
Date: Thu, 10 Apr 2008 19:39:51 +0200
From: Nick Piggin <npiggin@...e.de>
To: Yinghai Lu <yhlu.kernel@...il.com>
Cc: Andrew Morton <akpm@...ux-foundation.org>,
Andi Kleen <andi@...stfloor.org>, linux-kernel@...r.kernel.org,
linux-mm@...ck.org, pj@....com, kniht@...ux.vnet.ibm.com
Subject: Re: [patch 10/17] mm: fix bootmem alignment
On Thu, Apr 10, 2008 at 10:33:50AM -0700, Yinghai Lu wrote:
> On Thu, Apr 10, 2008 at 10:02 AM, <npiggin@...e.de> wrote:
> > Without this fix bootmem can return unaligned addresses when the start of a
> > node is not aligned to the align value. Needed for reliably allocating
> > gigabyte pages.
> >
> > I removed the offset variable because all tests should align themself correctly
> > now. Slight drawback might be that the bootmem allocator will spend
> > some more time skipping bits in the bitmap initially, but that shouldn't
> > be a big issue.
> >
>
>
> this patch from Andi was obsoleted by the one in -mm
Ah, great thanks for letting me know.
> The patch titled
> mm: offset align in alloc_bootmem
> has been added to the -mm tree. Its filename is
> mm-offset-align-in-alloc_bootmem.patch
>
> ------------------------------------------------------
> Subject: mm: offset align in alloc_bootmem
> From: Yinghai Lu <yhlu.kernel.send@...il.com>
>
> Need offset alignment when node_boot_start's alignment is less than align
> required
>
> Use local node_boot_start to match align. so don't add extra opteration in
> search loop.
>
> Signed-off-by: Yinghai Lu <yhlu.kernel@...il.com>
> Cc: Andi Kleen <ak@...e.de>
> Cc: Yasunori Goto <y-goto@...fujitsu.com>
> Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@...fujitsu.com>
> Cc: Ingo Molnar <mingo@...e.hu>
> Cc: Christoph Lameter <clameter@....com>
> Cc: Mel Gorman <mel@....ul.ie>
> Signed-off-by: Andrew Morton <akpm@...ux-foundation.org>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists