Hi Sergei, On Sunday, 13 August 2017 02:08:50 PDT Sergei Shtylyov wrote: > Hello! > > On 8/13/2017 7:36 AM, Paul Burton wrote: > > Currently in gic_set_affinity() we calculate a temporary cpumask holding > > the intersection of the provided cpumask & the CPUs that are online, > > then we call cpumask_first twice on it to find the first such CPU. Since > > we don't need to temporary cpumask for anything else & we only care > > s/to/the/? Indeed - nice to know someone is reading a 38 patch series :) I'll fix up but leave submitting a v2 until someone asks me to. Thanks, Paul > > > about the first CPU that's both online & in the provided cpumask, we can > > instead use cpumask_first_and to find that CPU & drop the temporary > > mask. > > > > Signed-off-by: Paul Burton <paul.burton@xxxxxxxxxx> > > Cc: Jason Cooper <jason@xxxxxxxxxxxxxx> > > Cc: Marc Zyngier <marc.zyngier@xxxxxxx> > > Cc: Ralf Baechle <ralf@xxxxxxxxxxxxxx> > > Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx> > > Cc: linux-mips@xxxxxxxxxxxxxx > > [...] > > MBR, Serge
Attachment:
signature.asc
Description: This is a digitally signed message part.