Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1600647 > unrolled thread

RE: [PATCH 10/14] cpufreq: intel_pstate: Use different utilization update callbacks

Started by"Doug Smythies" <dsmythies@telus.net>
First post2017-03-14 17:10 +0100
Last post2017-03-14 17:50 +0100
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  RE: [PATCH 10/14] cpufreq: intel_pstate: Use different utilization update callbacks "Doug Smythies" <dsmythies@telus.net> - 2017-03-14 17:10 +0100
    Re: [PATCH 10/14] cpufreq: intel_pstate: Use different utilization update callbacks "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2017-03-14 17:50 +0100

#1600647 — RE: [PATCH 10/14] cpufreq: intel_pstate: Use different utilization update callbacks

From"Doug Smythies" <dsmythies@telus.net>
Date2017-03-14 17:10 +0100
SubjectRE: [PATCH 10/14] cpufreq: intel_pstate: Use different utilization update callbacks
Message-ID<tkXbj-2zg-1@gated-at.bofh.it>
Hi Rafael,

I can not get this patch to apply.
My start point is kernel 4.11-rc2, so maybe (probably) it is my problem.

On 2017.03.12 10:22 Rafael J. Wysocki wrote:

... [cut] ...

> drivers/cpufreq/intel_pstate.c |   77 ++++++++++++++++++++++++++++-------------
> 1 file changed, 53 insertions(+), 24 deletions(-)
>
> Index: linux-pm/drivers/cpufreq/intel_pstate.c
> ===================================================================
> --- linux-pm.orig/drivers/cpufreq/intel_pstate.c
> +++ linux-pm/drivers/cpufreq/intel_pstate.c
> @@ -37,6 +37,9 @@
> #include <asm/cpufeature.h>
> #include <asm/intel-family.h>
> 
> +#define INTEL_PSTATE_DEFAULT_SAMPLING_INTERVAL	(10 * NSEC_PER_MSEC)
> +#define INTEL_PSTATE_HWP_SAMPLING_INTERVAL	(50 * NSEC_PER_MSEC)
> +
> #define INTEL_CPUFREQ_TRANSITION_LATENCY	20000
> 
> #define ATOM_RATIOS		0x66a

"git am" is getting upset because the above line is still there.
However it was removed in Len Brown's patch turbostat stuff
patch 15 of 44, which is in kernel 4.11-rc1.
(92134bdbc6272da6e98e9242cc6f1576cedc0735)

... [cut] ...

By the way, scripts/checkpatch.pl also complains about a line
length in the patch.

$ scripts/checkpatch.pl rjw_10_14.txt
WARNING: line over 80 characters
#148: FILE: drivers/cpufreq/intel_pstate.c:2031:
+       cpufreq_add_update_util_hook(cpu_num, &cpu->update_util, update_util_cb);

total: 0 errors, 1 warnings, 130 lines checked
 

[toc] | [next] | [standalone]


#1600691

From"Rafael J. Wysocki" <rjw@rjwysocki.net>
Date2017-03-14 17:50 +0100
Message-ID<tkXO2-2Py-31@gated-at.bofh.it>
In reply to#1600647
On Tuesday, March 14, 2017 09:07:48 AM Doug Smythies wrote:
> Hi Rafael,
> 
> I can not get this patch to apply.
> My start point is kernel 4.11-rc2, so maybe (probably) it is my problem.

I need to rebase the series.

There are fixes in flight that will need to go in first anyway.

Thanks,
Rafael

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web