On 26/08/2019 13:24, Peter Zijlstra wrote: > On Mon, Aug 26, 2019 at 11:51:17AM +0200, Dietmar Eggemann wrote: > >> Not sure about the extra 'if trace_cpu_frequency_enabled()' but I guess >> it doesn't hurt. > > Without that you do that for_each_cpu() iteration unconditionally, even > if the tracepoint is disabled.
Makes sense, I'm wondering if we want this in cpufreq_notify_transition() CPUFREQ_POSTCHANGE for the non-fast-switching drivers as well.