On Tue, Jan 15, 2019 at 9:27 PM Christoph Hellwig <h...@infradead.org> wrote: > > > + if (!force) > > + cpu = cpumask_any_and(mask_val, cpu_online_mask); > > + else > > + cpu = cpumask_first(mask_val); > > Any reason for the inverted test?
Okay, I will not use inverted test here. > > Otherwise this looks fine to me: > > Reviewed-by: Christoph Hellwig <h...@lst.de> Thanks, Anup