Re: [PATCH -v2 12/13] x86, mm: Use func pointer to table size calculation and mapping.

2012-09-03 Thread Yinghai Lu
On Sun, Sep 2, 2012 at 10:43 PM, Pekka Enberg wrote: > On Sun, Sep 2, 2012 at 10:46 AM, Yinghai Lu wrote: >> +static void __init with_all_ram_ranges( >> + void (*work_fn)(unsigned long, unsigned long, void >> *), >> + void *data) > >> +static void

Re: [PATCH -v2 12/13] x86, mm: Use func pointer to table size calculation and mapping.

2012-09-03 Thread Yinghai Lu
On Sun, Sep 2, 2012 at 10:43 PM, Pekka Enberg penb...@kernel.org wrote: On Sun, Sep 2, 2012 at 10:46 AM, Yinghai Lu ying...@kernel.org wrote: +static void __init with_all_ram_ranges( + void (*work_fn)(unsigned long, unsigned long, void *), + void

Re: [PATCH -v2 12/13] x86, mm: Use func pointer to table size calculation and mapping.

2012-09-02 Thread Pekka Enberg
On Sun, Sep 2, 2012 at 10:46 AM, Yinghai Lu wrote: > +static void __init with_all_ram_ranges( > + void (*work_fn)(unsigned long, unsigned long, void *), > + void *data) > +static void __init size_work_fn(unsigned long start, unsigned long end, void >

[PATCH -v2 12/13] x86, mm: Use func pointer to table size calculation and mapping.

2012-09-02 Thread Yinghai Lu
They all need to go over ram range in same sequence. So add shared function to reduce duplicated code. Signed-off-by: Yinghai Lu --- arch/x86/mm/init.c | 64 ++- 1 files changed, 23 insertions(+), 41 deletions(-) diff --git a/arch/x86/mm/init.c

[PATCH -v2 12/13] x86, mm: Use func pointer to table size calculation and mapping.

2012-09-02 Thread Yinghai Lu
They all need to go over ram range in same sequence. So add shared function to reduce duplicated code. Signed-off-by: Yinghai Lu ying...@kernel.org --- arch/x86/mm/init.c | 64 ++- 1 files changed, 23 insertions(+), 41 deletions(-) diff --git

Re: [PATCH -v2 12/13] x86, mm: Use func pointer to table size calculation and mapping.

2012-09-02 Thread Pekka Enberg
On Sun, Sep 2, 2012 at 10:46 AM, Yinghai Lu ying...@kernel.org wrote: +static void __init with_all_ram_ranges( + void (*work_fn)(unsigned long, unsigned long, void *), + void *data) +static void __init size_work_fn(unsigned long start, unsigned long