From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Thu, 15 Mar 2007 21:30:17 -0700 From: "Mark A. Greer" To: Paul Mackerras Subject: Re: [POWERPC] 32-bit early_init() should zero from __bss_start to __bss_stop only Message-ID: <20070316043017.GB27209@mag.az.mvista.com> References: <20061025233649.GB17461@mag.az.mvista.com> <1161821177.22582.149.camel@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1161821177.22582.149.camel@localhost.localdomain> Cc: linuxppc-dev List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Paul, This one was dropped somewhere along the way... Mark --- On Thu, Oct 26, 2006 at 10:06:16AM +1000, Benjamin Herrenschmidt wrote: > On Wed, 2006-10-25 at 16:36 -0700, Mark A. Greer wrote: > > Currently, early_init() in setup_32.c zeroes from '_bss_start' to '_end'. > > It should only zero from '__bss_start' to '__bss_stop'. This patch does that. > > > > Signed-off-by: Mark A. Greer > > Acked-by: Benjamin Herrenschmidt > > > > > setup_32.c | 3 ++- > > 1 files changed, 2 insertions(+), 1 deletion(-) > > --- > > > > diff --git a/arch/powerpc/kernel/setup_32.c b/arch/powerpc/kernel/setup_32.c > > index a4c2964..c682e9a 100644 > > --- a/arch/powerpc/kernel/setup_32.c > > +++ b/arch/powerpc/kernel/setup_32.c > > @@ -95,7 +95,8 @@ unsigned long __init early_init(unsigned > > > > /* First zero the BSS -- use memset_io, some platforms don't have > > * caches on yet */ > > - memset_io((void __iomem *)PTRRELOC(&__bss_start), 0, _end - __bss_start); > > + memset_io((void __iomem *)PTRRELOC(&__bss_start), 0, > > + __bss_stop - __bss_start); > > > > /* > > * Identify the CPU type and fix up code sections > > _______________________________________________ > > Linuxppc-dev mailing list > > Linuxppc-dev@ozlabs.org > > https://ozlabs.org/mailman/listinfo/linuxppc-dev >