Re: [PATCH 2/3] x86/mm: Simplify for-loop in free_init_pages()

2012-07-14 Thread Joe Perches
On Sat, 2012-07-14 at 23:48 +0300, Pekka Enberg wrote: > On Sat, Jul 14, 2012 at 11:43 PM, Joe Perches wrote: > > On Sat, 2012-07-14 at 13:36 -0700, Yinghai Lu wrote: > >> On Sat, Jul 14, 2012 at 6:41 AM, Pekka Enberg wrote: > >> > As a cleanup, move initial "addr" assignment to the for-loop

Re: [PATCH 2/3] x86/mm: Simplify for-loop in free_init_pages()

2012-07-14 Thread Joe Perches
On Sat, 2012-07-14 at 23:48 +0300, Pekka Enberg wrote: > On Sat, Jul 14, 2012 at 11:43 PM, Joe Perches wrote: > > On Sat, 2012-07-14 at 13:36 -0700, Yinghai Lu wrote: > >> On Sat, Jul 14, 2012 at 6:41 AM, Pekka Enberg wrote: > >> > As a cleanup, move initial "addr" assignment to the for-loop

Re: [PATCH 2/3] x86/mm: Simplify for-loop in free_init_pages()

2012-07-14 Thread Pekka Enberg
On Sat, Jul 14, 2012 at 11:43 PM, Joe Perches wrote: > On Sat, 2012-07-14 at 13:36 -0700, Yinghai Lu wrote: >> On Sat, Jul 14, 2012 at 6:41 AM, Pekka Enberg wrote: >> > As a cleanup, move initial "addr" assignment to the for-loop construct >> > in free_init_pages(). > > Not really a good idea.

Re: [PATCH 2/3] x86/mm: Simplify for-loop in free_init_pages()

2012-07-14 Thread Joe Perches
On Sat, 2012-07-14 at 13:36 -0700, Yinghai Lu wrote: > On Sat, Jul 14, 2012 at 6:41 AM, Pekka Enberg wrote: > > As a cleanup, move initial "addr" assignment to the for-loop construct > > in free_init_pages(). Not really a good idea. > > diff --git a/arch/x86/mm/init.c b/arch/x86/mm/init.c [] >

Re: [PATCH 2/3] x86/mm: Simplify for-loop in free_init_pages()

2012-07-14 Thread Yinghai Lu
On Sat, Jul 14, 2012 at 6:41 AM, Pekka Enberg wrote: > As a cleanup, move initial "addr" assignment to the for-loop construct > in free_init_pages(). > > Cc: Tejun Heo > Cc: Yinghai Lu > Signed-off-by: Pekka Enberg Acked-by: Yinghai Lu > --- > arch/x86/mm/init.c |4 +--- > 1 files

[PATCH 2/3] x86/mm: Simplify for-loop in free_init_pages()

2012-07-14 Thread Pekka Enberg
As a cleanup, move initial "addr" assignment to the for-loop construct in free_init_pages(). Cc: Tejun Heo Cc: Yinghai Lu Signed-off-by: Pekka Enberg --- arch/x86/mm/init.c |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/arch/x86/mm/init.c b/arch/x86/mm/init.c

[PATCH 2/3] x86/mm: Simplify for-loop in free_init_pages()

2012-07-14 Thread Pekka Enberg
As a cleanup, move initial addr assignment to the for-loop construct in free_init_pages(). Cc: Tejun Heo t...@kernel.org Cc: Yinghai Lu ying...@kernel.org Signed-off-by: Pekka Enberg penb...@kernel.org --- arch/x86/mm/init.c |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff

Re: [PATCH 2/3] x86/mm: Simplify for-loop in free_init_pages()

2012-07-14 Thread Yinghai Lu
On Sat, Jul 14, 2012 at 6:41 AM, Pekka Enberg penb...@kernel.org wrote: As a cleanup, move initial addr assignment to the for-loop construct in free_init_pages(). Cc: Tejun Heo t...@kernel.org Cc: Yinghai Lu ying...@kernel.org Signed-off-by: Pekka Enberg penb...@kernel.org Acked-by: Yinghai

Re: [PATCH 2/3] x86/mm: Simplify for-loop in free_init_pages()

2012-07-14 Thread Joe Perches
On Sat, 2012-07-14 at 13:36 -0700, Yinghai Lu wrote: On Sat, Jul 14, 2012 at 6:41 AM, Pekka Enberg penb...@kernel.org wrote: As a cleanup, move initial addr assignment to the for-loop construct in free_init_pages(). Not really a good idea. diff --git a/arch/x86/mm/init.c

Re: [PATCH 2/3] x86/mm: Simplify for-loop in free_init_pages()

2012-07-14 Thread Pekka Enberg
On Sat, Jul 14, 2012 at 11:43 PM, Joe Perches j...@perches.com wrote: On Sat, 2012-07-14 at 13:36 -0700, Yinghai Lu wrote: On Sat, Jul 14, 2012 at 6:41 AM, Pekka Enberg penb...@kernel.org wrote: As a cleanup, move initial addr assignment to the for-loop construct in free_init_pages(). Not

Re: [PATCH 2/3] x86/mm: Simplify for-loop in free_init_pages()

2012-07-14 Thread Joe Perches
On Sat, 2012-07-14 at 23:48 +0300, Pekka Enberg wrote: On Sat, Jul 14, 2012 at 11:43 PM, Joe Perches j...@perches.com wrote: On Sat, 2012-07-14 at 13:36 -0700, Yinghai Lu wrote: On Sat, Jul 14, 2012 at 6:41 AM, Pekka Enberg penb...@kernel.org wrote: As a cleanup, move initial addr

Re: [PATCH 2/3] x86/mm: Simplify for-loop in free_init_pages()

2012-07-14 Thread Joe Perches
On Sat, 2012-07-14 at 23:48 +0300, Pekka Enberg wrote: On Sat, Jul 14, 2012 at 11:43 PM, Joe Perches j...@perches.com wrote: On Sat, 2012-07-14 at 13:36 -0700, Yinghai Lu wrote: On Sat, Jul 14, 2012 at 6:41 AM, Pekka Enberg penb...@kernel.org wrote: As a cleanup, move initial addr