On 8 September 2016 at 04:57, Fenghua Yu <fenghua...@intel.com> wrote: > diff --git a/arch/x86/kernel/cpu/intel_rdt.c b/arch/x86/kernel/cpu/intel_rdt.c > index b25940a..9cf3a7d 100644 > --- a/arch/x86/kernel/cpu/intel_rdt.c > +++ b/arch/x86/kernel/cpu/intel_rdt.c > @@ -31,8 +31,22 @@ static struct clos_cbm_table *cctable; > * closid availability bit map. > */ > unsigned long *closmap; > +/* > + * Mask of CPUs for writing CBM values. We only need one CPU per-socket.
Does the second line make sense here? > + */ > +static cpumask_t rdt_cpumask; > +/* > + * Temporary cpumask used during hot cpu notificaiton handling. The usage > + * is serialized by hot cpu locks. s/notificaiton/notification -- Nilay
