Re: [PATCH 4/4] bootmem: fix wrong call parameter for free_bootmem()

2012-11-12 Thread Johannes Weiner
On Mon, Nov 12, 2012 at 03:23:42PM -0800, Andrew Morton wrote: > On Tue, 13 Nov 2012 01:31:55 +0900 > Joonsoo Kim wrote: > > > It is somehow strange that alloc_bootmem return virtual address > > and free_bootmem require physical address. > > Anyway, free_bootmem()'s first parameter should be

Re: [PATCH 4/4] bootmem: fix wrong call parameter for free_bootmem()

2012-11-12 Thread Johannes Weiner
On Mon, Nov 12, 2012 at 08:10:27PM -0500, Johannes Weiner wrote: > On Tue, Nov 13, 2012 at 01:31:55AM +0900, Joonsoo Kim wrote: > > It is somehow strange that alloc_bootmem return virtual address > > and free_bootmem require physical address. > > Anyway, free_bootmem()'s first parameter should be

Re: [PATCH 4/4] bootmem: fix wrong call parameter for free_bootmem()

2012-11-12 Thread Johannes Weiner
On Tue, Nov 13, 2012 at 01:31:55AM +0900, Joonsoo Kim wrote: > It is somehow strange that alloc_bootmem return virtual address > and free_bootmem require physical address. > Anyway, free_bootmem()'s first parameter should be physical address. > > There are some call sites for free_bootmem() with

Re: [PATCH 4/4] bootmem: fix wrong call parameter for free_bootmem()

2012-11-12 Thread Johannes Weiner
On Tue, Nov 13, 2012 at 01:31:55AM +0900, Joonsoo Kim wrote: > It is somehow strange that alloc_bootmem return virtual address > and free_bootmem require physical address. > Anyway, free_bootmem()'s first parameter should be physical address. > > There are some call sites for free_bootmem() with

Re: [PATCH 4/4] bootmem: fix wrong call parameter for free_bootmem()

2012-11-12 Thread JoonSoo Kim
Hi, Andrew. 2012/11/13 Andrew Morton : > On Tue, 13 Nov 2012 01:31:55 +0900 > Joonsoo Kim wrote: > >> It is somehow strange that alloc_bootmem return virtual address >> and free_bootmem require physical address. >> Anyway, free_bootmem()'s first parameter should be physical address. >> >> There

Re: [PATCH 4/4] bootmem: fix wrong call parameter for free_bootmem()

2012-11-12 Thread Andrew Morton
On Tue, 13 Nov 2012 01:31:55 +0900 Joonsoo Kim wrote: > It is somehow strange that alloc_bootmem return virtual address > and free_bootmem require physical address. > Anyway, free_bootmem()'s first parameter should be physical address. > > There are some call sites for free_bootmem() with

[PATCH 4/4] bootmem: fix wrong call parameter for free_bootmem()

2012-11-12 Thread Joonsoo Kim
It is somehow strange that alloc_bootmem return virtual address and free_bootmem require physical address. Anyway, free_bootmem()'s first parameter should be physical address. There are some call sites for free_bootmem() with virtual address. So fix them. Cc: Andrew Morton Cc: Benjamin

[PATCH 4/4] bootmem: fix wrong call parameter for free_bootmem()

2012-11-12 Thread Joonsoo Kim
It is somehow strange that alloc_bootmem return virtual address and free_bootmem require physical address. Anyway, free_bootmem()'s first parameter should be physical address. There are some call sites for free_bootmem() with virtual address. So fix them. Cc: Andrew Morton

Re: [PATCH 4/4] bootmem: fix wrong call parameter for free_bootmem()

2012-11-12 Thread Andrew Morton
On Tue, 13 Nov 2012 01:31:55 +0900 Joonsoo Kim js1...@gmail.com wrote: It is somehow strange that alloc_bootmem return virtual address and free_bootmem require physical address. Anyway, free_bootmem()'s first parameter should be physical address. There are some call sites for free_bootmem()

Re: [PATCH 4/4] bootmem: fix wrong call parameter for free_bootmem()

2012-11-12 Thread JoonSoo Kim
Hi, Andrew. 2012/11/13 Andrew Morton a...@linux-foundation.org: On Tue, 13 Nov 2012 01:31:55 +0900 Joonsoo Kim js1...@gmail.com wrote: It is somehow strange that alloc_bootmem return virtual address and free_bootmem require physical address. Anyway, free_bootmem()'s first parameter should

Re: [PATCH 4/4] bootmem: fix wrong call parameter for free_bootmem()

2012-11-12 Thread Johannes Weiner
On Tue, Nov 13, 2012 at 01:31:55AM +0900, Joonsoo Kim wrote: It is somehow strange that alloc_bootmem return virtual address and free_bootmem require physical address. Anyway, free_bootmem()'s first parameter should be physical address. There are some call sites for free_bootmem() with

Re: [PATCH 4/4] bootmem: fix wrong call parameter for free_bootmem()

2012-11-12 Thread Johannes Weiner
On Tue, Nov 13, 2012 at 01:31:55AM +0900, Joonsoo Kim wrote: It is somehow strange that alloc_bootmem return virtual address and free_bootmem require physical address. Anyway, free_bootmem()'s first parameter should be physical address. There are some call sites for free_bootmem() with

Re: [PATCH 4/4] bootmem: fix wrong call parameter for free_bootmem()

2012-11-12 Thread Johannes Weiner
On Mon, Nov 12, 2012 at 08:10:27PM -0500, Johannes Weiner wrote: On Tue, Nov 13, 2012 at 01:31:55AM +0900, Joonsoo Kim wrote: It is somehow strange that alloc_bootmem return virtual address and free_bootmem require physical address. Anyway, free_bootmem()'s first parameter should be

Re: [PATCH 4/4] bootmem: fix wrong call parameter for free_bootmem()

2012-11-12 Thread Johannes Weiner
On Mon, Nov 12, 2012 at 03:23:42PM -0800, Andrew Morton wrote: On Tue, 13 Nov 2012 01:31:55 +0900 Joonsoo Kim js1...@gmail.com wrote: It is somehow strange that alloc_bootmem return virtual address and free_bootmem require physical address. Anyway, free_bootmem()'s first parameter should