linux-kernel - Re: [PATCH -mm 00/14] bootmem rewrite v4

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 06 Jun 2008 10:15:33 +0900
From:	Yasunori Goto <y-goto@...fujitsu.com>
To:	Johannes Weiner <hannes@...urebad.de>
Cc:	Andrew Morton <akpm@...ux-foundation.org>,
	Ingo Molnar <mingo@...e.hu>,
	Yinghai Lu <yhlu.kernel@...il.com>,
	Andi Kleen <andi@...stfloor.org>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH -mm 00/14] bootmem rewrite v4


OK.
I reviewed this patch set, and my ia64 box can boot up with it. :-)

Thanks.

Acked-by: Yasunori Goto <y-goto@...fujitsu.com>


> Hi,
> 
> This is a complete overhaul of the bootmem allocator while preserving
> most of its original functionality.
> 
> free_bootmem and reserve_bootmem become a bit stricter than they are
> right now, callsites have to make sure that the PFN range is
> contiguous but it might go across node boundaries.
> 
> alloc_bootmem satisfying the allocation goal is more likely as the
> routines will try to allocate on the node holding the goal first
> before falling back as opposed to the original behaviour that
> satisfies the goal only if it is on the first node.
> 
> All in all, I think the code has become simpler and cleaner.  All
> public interfaces have been documented, too.
> 
> version 4: based on -mmotm from 08-06-05
>   o fixed fallback logic in alloc_bootmem_core (take the fallback even
>     in cases where sidx is zero) to make Yasunori's machine boot again
>   o fixed bogus range checks in ___alloc_bootmem_nopanic
>   o removed bogus `fallback'-offset calculation in alloc_bootmem_core
>   o renamed last_offset to last_end_off
>   o renamed last_success to hint_idx
>   o start searching after the end of the previous allocation, not at
>     its beginning
>   o improved more identifier names for offsets, pfns, indices and
>     addresses
>   o adjusted debugging output to output hex numbers consistently
> 
> version 3:
>   o fixed kerneldoc as noted by Chris Malley
>   o fixed incorrect bdata->last_success treatment as noted by
>     Yasunori Goto
> 
> version 2:
>   o broken up patches into logical chunks
> 
> ---
> 
>  arch/alpha/mm/numa.c     |    2 +-
>  arch/arm/plat-omap/fb.c  |    4 +-
>  arch/avr32/mm/init.c     |    3 +-
>  arch/ia64/mm/discontig.c |   19 +-
>  arch/m32r/mm/discontig.c |    3 +-
>  arch/m32r/mm/init.c      |    4 +-
>  arch/mn10300/mm/init.c   |    6 +-
>  arch/sh/mm/init.c        |    2 +-
>  include/linux/bootmem.h  |   87 +++---
>  mm/bootmem.c             |  934 +++++++++++++++++++++++++---------------------
>  10 files changed, 570 insertions(+), 494 deletions(-)
> 

-- 
Yasunori Goto 


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