Re: [PATCH 3/4] bootmem: remove alloc_arch_preferred_bootmem()

2012-11-12 Thread Johannes Weiner
On Tue, Nov 13, 2012 at 01:31:54AM +0900, Joonsoo Kim wrote: > The name of function is not suitable for now. > And removing function and inlining it's code to each call sites > makes code more understandable. > > Additionally, we shouldn't do allocation from bootmem > when slab_is_available(), so

[PATCH 3/4] bootmem: remove alloc_arch_preferred_bootmem()

2012-11-12 Thread Joonsoo Kim
The name of function is not suitable for now. And removing function and inlining it's code to each call sites makes code more understandable. Additionally, we shouldn't do allocation from bootmem when slab_is_available(), so directly return kmalloc*'s return value. Signed-off-by: Joonsoo Kim

[PATCH 3/4] bootmem: remove alloc_arch_preferred_bootmem()

2012-11-12 Thread Joonsoo Kim
The name of function is not suitable for now. And removing function and inlining it's code to each call sites makes code more understandable. Additionally, we shouldn't do allocation from bootmem when slab_is_available(), so directly return kmalloc*'s return value. Signed-off-by: Joonsoo Kim

Re: [PATCH 3/4] bootmem: remove alloc_arch_preferred_bootmem()

2012-11-12 Thread Johannes Weiner
On Tue, Nov 13, 2012 at 01:31:54AM +0900, Joonsoo Kim wrote: The name of function is not suitable for now. And removing function and inlining it's code to each call sites makes code more understandable. Additionally, we shouldn't do allocation from bootmem when slab_is_available(), so