Re: [PATCH V4 00/15] blk-throttle: add .high limit

From: Bart Van Assche
Date: Mon Nov 14 2016 - 17:46:42 EST


On 11/14/2016 02:22 PM, Shaohua Li wrote:
The background is we don't have an ioscheduler for blk-mq yet, so we can't
prioritize processes/cgroups. This patch set tries to add basic arbitration
between cgroups with blk-throttle. It adds a new limit io.high for
blk-throttle. It's only for cgroup2.

Hello Shaohua,

My understanding of this work is that a significant part of it will have to be reverted once blk-mq supports I/O scheduling, e.g. the code for detecting whether the I/O submitter is idle. Shouldn't this kind of infrastructure be added after support has been added in blk-mq for I/O scheduling?

Thanks,

Bart.