Re: [patch 1/3] IA64: put kdump_find_rsvd_region in __init

From: Horms <horms_at_verge.net.au>
Date: 2007-03-07 12:03:21
On Tue, Mar 06, 2007 at 04:34:44PM +0800, Zou, Nanhai wrote:
> > -----Original Message-----
> > From: Horms [mailto:horms@verge.net.au]
> > Sent: 2007年3月6日 15:29
> > To: Linux-IA64; fastboot@lists.osdl.org
> > Cc: Luck, Tony; Zou, Nanhai; Magnus Damm
> > Subject: [patch 1/3] IA64: put kdump_find_rsvd_region in __init
> > 
> > kdump_find_rsvd_region() is only called by
> > reserve_memory() which is in __init, so it seems that
> > kdump_find_rsvd_region() should also be in there.
> > 
> > Signed-off-by: Simon Horman <horms@verge.net.au>
> > 
> > Index: linux-2.6/arch/ia64/kernel/efi.c
> > ===================================================================
> > --- linux-2.6.orig/arch/ia64/kernel/efi.c	2007-03-06 16:30:10.000000000
> > +0900
> > +++ linux-2.6/arch/ia64/kernel/efi.c	2007-03-06 16:30:13.000000000 +0900
> > @@ -1136,7 +1136,7 @@
> >  /* find a block of memory aligned to 64M exclude reserved regions
> >     rsvd_regions are sorted
> >   */
> > -unsigned long
> > +unsigned long __init
> >  kdump_find_rsvd_region (unsigned long size,
> >  		struct rsvd_region *r, int n)
> >  {
> > 
> > --
> > 
>  This patch looks good to me.

Thanks, it it ok if I repost the patch with Acked-by: you ?

-- 
Horms
  H: http://www.vergenet.net/~horms/
  W: http://www.valinux.co.jp/en/

-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Received on Wed Mar 07 12:06:57 2007

This archive was generated by hypermail 2.1.8 : 2007-03-07 12:12:39 EST