> - queue->io_cpu = (qid == 0) ? 0 : qid - 1; > + n = (qid ? qid - 1 : 0) % num_online_cpus(); > + queue->io_cpu = cpumask_next_wrap(n - 1, cpu_online_mask, -1, false); Nitpick: can you use an if/else here? The ? : construct just looks like obsfucation. If nothing else pops up I can fix that up when applying. Otherwise: Reviewed-by: Christoph Hellwig <hch@xxxxxx>
