On Thu, Jul 12, 2018 at 01:29:40PM -0400, Johannes Weiner wrote: > +struct psi_group { > + struct psi_group_cpu *cpus; That one wants a __percpu annotation on I think. Also, maybe a rename. > + > + struct mutex stat_lock; > + > + u64 some[NR_PSI_RESOURCES]; > + u64 full[NR_PSI_RESOURCES]; > + > + unsigned long period_expires; > + > + u64 last_some[NR_PSI_RESOURCES]; > + u64 last_full[NR_PSI_RESOURCES]; > + > + unsigned long avg_some[NR_PSI_RESOURCES][3]; > + unsigned long avg_full[NR_PSI_RESOURCES][3]; > + > + struct delayed_work clock_work; > +};