On 16-10-19, 04:56, Frederic Weisbecker wrote: > Now that we have a vtime safe kcpustat accessor for CPUTIME_SYSTEM, use > it to start fixing frozen kcpustat values on nohz_full CPUs. > > Reported-by: Yauheni Kaliuta <yauheni.kali...@redhat.com> > Signed-off-by: Frederic Weisbecker <frede...@kernel.org> > Cc: Rafael J. Wysocki <r...@rjwysocki.net> > Cc: Viresh Kumar <viresh.ku...@linaro.org> > Cc: Yauheni Kaliuta <yauheni.kali...@redhat.com> > Cc: Thomas Gleixner <t...@linutronix.de> > Cc: Rik van Riel <r...@surriel.com> > Cc: Peter Zijlstra <pet...@infradead.org> > Cc: Wanpeng Li <wanpen...@tencent.com> > Cc: Ingo Molnar <mi...@kernel.org> > --- > drivers/cpufreq/cpufreq.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c > index c52d6fa32aac..a37ebfd7e0e8 100644 > --- a/drivers/cpufreq/cpufreq.c > +++ b/drivers/cpufreq/cpufreq.c > @@ -120,7 +120,7 @@ static inline u64 get_cpu_idle_time_jiffy(unsigned int > cpu, u64 *wall) > cur_wall_time = jiffies64_to_nsecs(get_jiffies_64()); > > busy_time = kcpustat_cpu(cpu).cpustat[CPUTIME_USER]; > - busy_time += kcpustat_cpu(cpu).cpustat[CPUTIME_SYSTEM]; > + busy_time += kcpustat_field(&kcpustat_cpu(cpu), CPUTIME_SYSTEM, cpu); > busy_time += kcpustat_cpu(cpu).cpustat[CPUTIME_IRQ]; > busy_time += kcpustat_cpu(cpu).cpustat[CPUTIME_SOFTIRQ]; > busy_time += kcpustat_cpu(cpu).cpustat[CPUTIME_STEAL];
Acked-by: Viresh Kumar <viresh.ku...@linaro.org> -- viresh