Re: [RFC/RFT] [PATCH v3 4/4] cpufreq: intel_pstate: enable boost for SKX

From: Srinivas Pandruvada
Date: Fri Jun 01 2018 - 10:57:55 EST


On Fri, 2018-06-01 at 14:01 +0200, Giovanni Gherdovich wrote:
> On Thu, May 31, 2018 at 03:51:43PM -0700, Srinivas Pandruvada wrote:
> > Enable HWP boost on Skylake server platform.
> >
> > Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@xxxxxxxxxxx
> > .com>
> > ---
> > Âdrivers/cpufreq/intel_pstate.c | 9 +++++++++
> > Â1 file changed, 9 insertions(+)
> >
> > diff --git a/drivers/cpufreq/intel_pstate.c
> > b/drivers/cpufreq/intel_pstate.c
> > index 254e30a6c1e0..75c5a95bc0ac 100644
> > --- a/drivers/cpufreq/intel_pstate.c
> > +++ b/drivers/cpufreq/intel_pstate.c
> > @@ -1794,6 +1794,11 @@ static const struct x86_cpu_id
> > intel_pstate_cpu_ee_disable_ids[] = {
> > Â {}
> > Â};
> > Â
> > +static const struct x86_cpu_id intel_pstate_hwp_boost_ids[]
> > __initconst = {
> > + ICPU(INTEL_FAM6_SKYLAKE_X, core_funcs),
> > + {}
> > +};
> > +
> > Âstatic int intel_pstate_init_cpu(unsigned int cpunum)
> > Â{
> > Â struct cpudata *cpu;
> > @@ -1824,6 +1829,10 @@ static int intel_pstate_init_cpu(unsigned
> > int cpunum)
> > Â intel_pstate_disable_ee(cpunum);
> > Â
> > Â intel_pstate_hwp_enable(cpu);
> > +
> > + id = x86_match_cpu(intel_pstate_hwp_boost_ids);
> > + if (id)
> > + hwp_boost = true;
> > Â }
> > Â
> > Â intel_pstate_get_cpu_pstates(cpu);
> > --Â
> > 2.13.6
> >
> >
>
> I have a Skylake server, HWP capable, but it doesn't trigger this
> bit; I had
> to do "echo 1 >
> /sys/devices/system/cpu/intel_pstate/hwp_dynamic_boost".
>
> Just out of curiosity, do you see reasons for leaving some SKL out of
> the
> default setting? Or maybe I'm missing something.
This is entry level server using SKL desktop CPU model. We will see
some improvements, but not as big as Xeon E5s. So we can add based on
your test results.

Thanks,
Srinivas

>
> /proc/cpuinfo says that the model name is "Intel(R) Xeon(R) CPU E3-
> 1240 v5";
> the outputs of `cpuid -1` and `cpuid -1 --raw` are in this pastebin:
> http://paste.opensuse.org/view/raw/34255499


>
>
> Giovanni Gherdovich
> SUSE Labs