Re: [PATCH V4 2/2] cpufreq: intel_pstate: Implement QoS supported freq constraints

2019-08-08 Thread Viresh Kumar
On 08-08-19, 23:35, Doug Smythies wrote: > O.K. While I understand the explanations, I still struggle with > this scenario: > > doug@s15:~/temp$ cat /sys/devices/system/cpu/intel_pstate/max_perf_pct > 50<<< Note: 50% = 1.9 GHz in my system) > doug@s15:~/temp$ grep . > /sys/devices/system/cpu

RE: [PATCH V4 2/2] cpufreq: intel_pstate: Implement QoS supported freq constraints

2019-08-08 Thread Doug Smythies
On 2019.08.08 19:16 Viresh Kumar wrote: > On 08-08-19, 09:25, Doug Smythies wrote: >> On 2019.08.07 00:06 Viresh Kumar wrote: >> Tested by: Doug Smythies >> Thermald seems to now be working O.K. for all the governors. > > Thanks for testing Doug. > >> I do note that if one sets >> /sys/devices/sy

Re: [PATCH V4 2/2] cpufreq: intel_pstate: Implement QoS supported freq constraints

2019-08-08 Thread Viresh Kumar
On 08-08-19, 09:25, Doug Smythies wrote: > On 2019.08.07 00:06 Viresh Kumar wrote: > Tested by: Doug Smythies > Thermald seems to now be working O.K. for all the governors. Thanks for testing Doug. > I do note that if one sets > /sys/devices/system/cpu/cpufreq/policy*/scaling_max_freq > It seems

Re: [PATCH V4 2/2] cpufreq: intel_pstate: Implement QoS supported freq constraints

2019-08-08 Thread Rafael J. Wysocki
, On Thu, Aug 8, 2019 at 6:25 PM Doug Smythies wrote: > > On 2019.08.07 00:06 Viresh Kumar wrote: > > Thanks for your work on this. > > > Intel pstate driver exposes min_perf_pct and max_perf_pct sysfs files, > > which can be used to force a limit on the min/max P state of the driver. > > Though t

RE: [PATCH V4 2/2] cpufreq: intel_pstate: Implement QoS supported freq constraints

2019-08-08 Thread Doug Smythies
On 2019.08.07 00:06 Viresh Kumar wrote: Thanks for your work on this. > Intel pstate driver exposes min_perf_pct and max_perf_pct sysfs files, > which can be used to force a limit on the min/max P state of the driver. > Though these files eventually control the min/max frequencies that the > CPUs