From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ozlabs.org (ozlabs.org [203.10.76.45]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "mx.ozlabs.org", Issuer "CA Cert Signing Authority" (verified OK)) by bilbo.ozlabs.org (Postfix) with ESMTPS id 958BDB71D9 for ; Mon, 22 Jun 2009 08:57:32 +1000 (EST) Received: from gate.crashing.org (gate.crashing.org [63.228.1.57]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id D2815DDDA0 for ; Mon, 22 Jun 2009 08:57:31 +1000 (EST) Subject: Re: Badness on the Warp From: Benjamin Herrenschmidt To: Sean MacLennan In-Reply-To: <20090621183658.2b408e69@lappy.seanm.ca> References: <20090620154824.72b5cd50@lappy.seanm.ca> <200906202256.46073.elendil@planet.nl> <20090620194250.5a3e826f@lappy.seanm.ca> <200906210628.35553.elendil@planet.nl> <84144f020906210320n2984807dw2d4b4fb38afd22cf@mail.gmail.com> <1245623104.16880.26.camel@pasglop> <20090621183658.2b408e69@lappy.seanm.ca> Content-Type: text/plain Date: Mon, 22 Jun 2009 08:57:15 +1000 Message-Id: <1245625035.16880.31.camel@pasglop> Mime-Version: 1.0 Cc: Frans Pop , linux-kernel@vger.kernel.org, linuxppc-dev@ozlabs.org, Pekka Enberg , Paul Mackerras , David Gibson List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Sun, 2009-06-21 at 18:36 -0400, Sean MacLennan wrote: > On Mon, 22 Jun 2009 08:25:04 +1000 > Benjamin Herrenschmidt wrote: > > > Right, our interrupt controllers need those fixes, they are low > > on my priority list since it's a reasonably harmless warning and I'm > > still chasing some actual breakage (though maybe not directly related > > to your patches). > > > > Kumar already submitted a couple, Frans, feel free to beat me > > at converting UIC (just use kmalloc directly in there instead > > of alloc_bootmem). > > I can make the changes to UIC if you want. They badness is harmless (it > ends up calling kzalloc anyway), but hard to explain to our PV (Product > Verification) department that they can ignore what looks like a crash ;) Heh, right, but hopefully we should have that fixed before .31 is out :-) Cheers, Ben.