From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: Kernel crash with acpi_processor, cpu_idle and intel_idle =y Date: Tue, 31 Jul 2012 19:55:27 -0400 Message-ID: <20120731235527.GM32698@phenom.dumpdata.com> References: <20120722181611.7ae03506@internecto.net> <20120723142025.GB793@phenom.dumpdata.com> <20120726022527.4d3f6e4a@internecto.net> <20120726140552.GD28024@phenom.dumpdata.com> <501176DD0200007800090A32@nat28.tlf.novell.com> <20120726145535.GA5743@phenom.dumpdata.com> <20120728145515.50bf4d45@internecto.net> <501657C70200007800091357@nat28.tlf.novell.com> <20120731134054.GD4789@phenom.dumpdata.com> <20120801015941.22f773d3@internecto.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <20120801015941.22f773d3@internecto.net> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Mark van Dijk Cc: Jan Beulich , xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On Wed, Aug 01, 2012 at 01:59:41AM +0200, Mark van Dijk wrote: > Message 20120731134054.GD4789@phenom.dumpdata.com contained: > > >Duh! That is easy enough to fix. Mark, can you please try testing with > >this patch (and obviously enable the CONFIG_INTEL_IDLE) > > > >diff --git a/drivers/idle/intel_idle.c b/drivers/idle/intel_idle.c > >index d0f59c3..46a9884 100644 > >--- a/drivers/idle/intel_idle.c > >+++ b/drivers/idle/intel_idle.c > >@@ -557,7 +557,7 @@ static int __init intel_idle_init(void) > > retval = cpuidle_register_driver(&intel_idle_driver); > > if (retval) { > > printk(KERN_DEBUG PREFIX "intel_idle yielding to %s", > >- cpuidle_get_driver()->name); > >+ cpuidle_get_driver() ? > >cpuidle_get_driver()->name : "none"); > > return retval; > > } > > > > Good going Jan and Konrad. The above patch works: no more crashes. For > fun I have also tested the patch from my previous email (the link to > the lkml, it has been acked so was worth a try). I tested it in > correlation with your patch and I still see no crashes. Ok, is it Ok if I include your name as Reported-by and Tested-by? Thx > > > > -- > Stay in touch, > Mark van Dijk. ,-------------------------------- > ----------------------------' Tue Jul 31 22:19 UTC 2012 > Today is Pungenday, the 67th day of Confusion in the YOLD 3178