From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: Failure to Parse and Load C-States Date: Fri, 4 Jan 2013 16:16:18 -0500 Message-ID: <20130104211618.GB7996@phenom.dumpdata.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Tom Goetz Cc: Konrad Rzeszutek Wilk , Ben Guthro , xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On Thu, Jan 03, 2013 at 01:53:04PM -0500, Tom Goetz wrote: > Konrad, > > We're seeing a failure where c-states are not parsed and loaded to the hypervisor. > > What is happening is when the following calls happen, there are no c-states in the per CPU data. > > [ 1.954368] [] push_cxx_to_hypervisor+0x1a5/0x2d5 > [ 1.954374] [] upload_pm_data+0x6d/0xa2 > [ 1.954380] [] xen_acpi_processor_init+0x3c3/0x488 > > It looks like the c-state parsing should have happened in acpi_processor_power_init but didn't because of the following check at the top: > > if (disabled_by_idle_boot_param()) > return 0; > > The preceding is true because boot_option_idle_override is set to IDLE_HALT by arch/x86/xen/setup.c xen_arch_setup. > > Is the Xen version of acpi_processor supposed to do it's own c-state parsing or rely on the base acpi_processor c-state parsing? I'm assuming Xen sets IDLE_HALT to stop runtime usage of Linux c-state management, but it also stops parsing c-states. > > Linux: 3.2.23 I believe you are missing some patches to cpufreq.c and setup.c, which are: t 48cdd8287f47a3cdad5b9273a5ef81bf605f7826 Author: Konrad Rzeszutek Wilk Date: Tue Mar 13 20:06:57 2012 -0400 xen/cpufreq: Disable the cpu frequency scaling drivers from loading. commit cc7335b2f6acc0f24c7fac80ce536301f7d52214 Author: Konrad Rzeszutek Wilk Date: Mon Jan 23 10:53:57 2012 -0500 xen/setup/pm/acpi: Remove the call to boot_option_idle_override. commit a7b422cda5084db7265c3b23310a959b43b47529 Author: Konrad Rzeszutek Wilk Date: Tue Mar 13 19:18:39 2012 -0400 provide disable_cpufreq() function to disable the API. commit e5fd47bfab2df0c2184cc0bf4245d8e1bb7724fb Author: Konrad Rzeszutek Wilk Date: Mon Nov 21 18:02:02 2011 -0500 xen/pm_idle: Make pm_idle be default_idle under Xen. > Xen: 4.0.4 > > Thanks for any help, > > Tom Goetz > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xen.org > http://lists.xen.org/xen-devel >