Re: regression caused by bb6ab52f2bef ("intel_pstate: Do not set utilization update hook too early")

From: Rafael J. Wysocki
Date: Fri Jun 24 2016 - 20:03:44 EST


On Thu, Jun 23, 2016 at 6:07 PM, Doug Smythies <dsmythies@xxxxxxxxx> wrote:
> On 2016.06.17 08:32 Jisheng Zhang wrote:
>
>> Second, sorry again for report incorrect commit,
>> I were too tired this morning so I remember the wrong commit.
>> The regression is caused by bb6ab52f2bef ("intel_pstate: Do not
>> set utilization update hook too early"), so I update the email title.
>
> Summary:
>
> Isn't running powertop itself a significant contributor here?

It shouldn't be, because the overhead in question comes from
->setpolicy and powertop itself should not cause it to be invoked that
often.

Thanks,
Rafael