Re: [PATCH] block/blk-mq: Don't complete locally if capacities are different

From: Jens Axboe
Date: Tue Jan 23 2024 - 10:59:16 EST


On 1/23/24 1:47 AM, Christoph Hellwig wrote:
> On Mon, Jan 22, 2024 at 10:42:20PM +0000, Qais Yousef wrote:
>> The logic in blk_mq_complete_need_ipi() assumes SMP systems where all
>> CPUs have equal capacities
>
> What is a capacity here?

It seems to be the chosen word to describe the performance potential of
the core in question, we use it elsewhere in the kernel. But yes, could
do with a bit more of an explanation.

>> + return arch_scale_cpu_capacity(this_cpu) >= arch_scale_cpu_capacity(that_cpu);
>
> oerly long line here.
>
> Also pleas split patches for different subsystems.

Yes please, the sched/topology thing should be a separate prep patch.

--
Jens Axboe