From mboxrd@z Thu Jan 1 00:00:00 1970 From: Benjamin Herrenschmidt Subject: Re: [PATCH 02/37] lmb: Prepare x86 to use lmb to replace early_res Date: Wed, 16 Jun 2010 18:48:27 +1000 Message-ID: <1276678107.2552.214.camel@pasglop> References: <1276666966-14259-1-git-send-email-yinghai@kernel.org> <1276666966-14259-3-git-send-email-yinghai@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1276666966-14259-3-git-send-email-yinghai@kernel.org> Sender: linux-kernel-owner@vger.kernel.org To: Yinghai Lu Cc: Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , Andrew Morton , David Miller , Linus Torvalds , Johannes Weiner , linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org List-Id: linux-arch.vger.kernel.org On Tue, 2010-06-15 at 22:42 -0700, Yinghai Lu wrote: > 1. expose lmb_debug > 2. expose lmb_reserved_init_regions > > -v2: drop lmb_add_region() and LMB_ERROR export > -v3: seperate wrong return of lmb_fin_base to another patch > -v4: expose lmb_can_resize to handle x86 EFI that could have more than > 128 entries I don't like those too much but I suppose we can go with that for now and fix things up later. Ack. Ben. > Signed-off-by: Yinghai Lu > --- > include/linux/lmb.h | 3 +++ > lib/lmb.c | 5 +++-- > 2 files changed, 6 insertions(+), 2 deletions(-) > > diff --git a/include/linux/lmb.h b/include/linux/lmb.h > index 6f8c4bd..1e96224 100644 > --- a/include/linux/lmb.h > +++ b/include/linux/lmb.h > @@ -39,6 +39,9 @@ struct lmb { > }; > > extern struct lmb lmb; > +extern int lmb_debug; > +extern int lmb_can_resize; > +extern struct lmb_region lmb_reserved_init_regions[]; > > extern void __init lmb_init(void); > extern void __init lmb_analyze(void); > diff --git a/lib/lmb.c b/lib/lmb.c > index 13d1a04..bbdb1ec 100644 > --- a/lib/lmb.c > +++ b/lib/lmb.c > @@ -22,9 +22,10 @@ > > struct lmb lmb; > > -static int lmb_debug, lmb_can_resize; > +int lmb_debug; > +int lmb_can_resize; > static struct lmb_region lmb_memory_init_regions[INIT_LMB_REGIONS + 1]; > -static struct lmb_region lmb_reserved_init_regions[INIT_LMB_REGIONS + 1]; > +struct lmb_region lmb_reserved_init_regions[INIT_LMB_REGIONS + 1]; > > #define LMB_ERROR (~(phys_addr_t)0) >