From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755553AbcKENz1 (ORCPT ); Sat, 5 Nov 2016 09:55:27 -0400 Received: from mga02.intel.com ([134.134.136.20]:31816 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754044AbcKENzY (ORCPT ); Sat, 5 Nov 2016 09:55:24 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.31,597,1473145200"; d="scan'208";a="898112599" From: "Pandruvada, Srinivas" To: "pavel@ucw.cz" CC: "linux-kernel@vger.kernel.org" , "Zhang, Rui" , "linux-pm@vger.kernel.org" , "platform-driver-x86@vger.kernel.org" , "rjw@rjwysocki.net" , "viresh.kumar@linaro.org" , "ibm-acpi-devel@lists.sourceforge.net" , "ibm-acpi@hmh.eng.br" , "linux-acpi@vger.kernel.org" Subject: Re: v4.8-rc1: thinkpad x60: running at low frequency even during kernel build Thread-Topic: v4.8-rc1: thinkpad x60: running at low frequency even during kernel build Thread-Index: AQHSNqR6E9PvsljNE0SLAUXkZpyWbKDJwJWAgAAZhgCAABIcgIAA706AgAAFAoA= Date: Sat, 5 Nov 2016 13:55:22 +0000 Message-ID: <1478354115.19557.17.camel@intel.com> References: <20161104083849.GA32688@amd> <20161104085830.GA4089@amd> <1478268311.26953.17.camel@intel.com> <20161104204439.GA2581@amd> <20161104221600.GA7007@amd> <1478301649.7947.3.camel@intel.com> <20161105133719.GA3933@amd> In-Reply-To: <20161105133719.GA3933@amd> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.255.157.235] Content-Type: text/plain; charset="utf-8" Content-ID: MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id uA5DtYKY025627 On Sat, 2016-11-05 at 14:37 +0100, Pavel Machek wrote: > Hi! > > > > > [...] > > > > > > > > So we seem to have thermal or ACPI regression in v4.9-rc3. > > > > > It is possible. Can you add either add printk > > in acpi_processor_ppc_has_changed() or use ftrace and see do you > > get to > > these functions > > > > acpi_processor_ppc_init() > > acpi_processor_ppc_has_changed() > > acpi_processor_ppc_notifier() > > > > ? > > Yes, these seem to be called. Here's the log: Try this 1. Either enable dyndebug or add  #define DEBUG 1  at the top of acpi-cpufreq.c 2. diff --git a/drivers/acpi/processor_perflib.c b/drivers/acpi/processor_perflib.c index bb01dea..6074995 100644 --- a/drivers/acpi/processor_perflib.c +++ b/drivers/acpi/processor_perflib.c @@ -94,9 +94,14 @@ static int acpi_processor_ppc_notifier(struct notifier_block *nb,           ppc = (unsigned int)pr->performance_platform_limit;   +       printk(KERN_ERR "ppc = %d\n", ppc); +         if (ppc >= pr->performance->state_count)                 goto out;   +       printk(KERN_ERR "ppc = %d freq-limit %d\n", ppc, pr- >performance->states[ppc]. +                                     core_frequency * 1000); +         cpufreq_verify_within_limits(policy, 0,                                      pr->performance->states[ppc].                                      core_frequency * 1000); Thanks, Srinivas