Re: v4.8-rc1: thinkpad x60: running at low frequency even during kernel build

From: Pandruvada, Srinivas
Date: Sat Nov 05 2016 - 09:55:34 EST


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