> -     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 <h...@lst.de>

Reply via email to