Re: [PATCH] cpufreq: cpufreq_stats in the absence of frequency table

["Prakash, Prashanth" <pprakash@codeaurora.org>]

Hi Rafael,

On 8/2/2016 5:49 PM, Rafael J. Wysocki wrote:
> On Tuesday, August 02, 2016 05:41:24 PM Prakash, Prashanth wrote:
>> Hi Rafael,
>>
>> On 8/2/2016 5:27 PM, Rafael J. Wysocki wrote:
>>> On Tuesday, August 02, 2016 05:18:57 PM Prashanth Prakash wrote:
>>>> cpufreq drivers such as CPPC works on contigious scale and does
>>>> not have a static frequency table. This commit adds cpufreq_stats
>>>> support for such drivers by creating a pseudo frequency table by
>>>> discretizing the contigious scale.
>>> No.
>>>
>>> And why?
>> The motivation is to be able to collect the frequency distribution data via
>> time_in_state sysfs entry as we can collect for other cpufreq drivers that
>> uses a frequency table.
> Which can be done by using the cpu_frequency trace point just fine, can't it?
Yes, but wanted to make sure existing userspace tools can be used with
platforms using CPPC as well.
> Plus, cpufreq_stats don't work with the schedutil governor and fast frequency
> switching anyway.
If we are expecting that usage of cpufreq_stats will slowly go away as we
switch to schedutil and more drivers enable fast_switching, then I agree
it doesn't make much sense to add support for cpufreq_stats.

Thanks,
Prashanth