在 2022/01/27 1:29, Tejun Heo 写道:
On Fri, Jan 14, 2022 at 05:30:00PM +0800, Yu Kuai wrote:
RFC patch: https://lkml.org/lkml/2021/9/9/1432
There is a proformance problem in our environment:
A host can provide a remote device to difierent client. If one client is
under high io pressure, other clients might be affected.
Limit the overall iops/bps(io.max) from the client can fix the problem,
however, config files do not exist in root cgroup currently, which makes
it impossible.
This patch enables io throttle for root cgroup:
- enable "io.max" and "io.low" in root
- don't skip root group in tg_iops_limit() and tg_bps_limit()
- don't skip root group in tg_conf_updated()
Signed-off-by: Yu Kuai <yukuai3@xxxxxxxxxx>
Yeah, I'm kinda split. It's a simple change with some utility, but it's also
something which doesn't fit with the cgroup feature or interface. It's
regulating the whole system behavior. There's no reason for any of the
control "groups" to be involved here and semantically the interface would
fit a lot better under /proc, /sys or some other system-wide location. Here
are some points to consider:
* As a comparison, it'd be rather absurd to enable memory.max at system root
in terms of interface and most likely break whole lot of mm operations.
* Resource control knobs of a cgroup belong to the parent as the parent is
responsible for divvying up the available resources to its children. Here
too, the knobs are making sense because there's a higher level parent
(whether that's hypervisor or some network server).
Is your use case VMs or network attached storage?
Hi,
In our case, the disk is provided by server, and such disk can be shared
by multipul clients. Thus for the client side, the server is a higher
level parent.
Theoretically, limit the io from server for each client is feasible,
however, the main reason we don't want to do this is the following
shortcoming:
client can still send io to server unlimited, we can just limit the
amount of io that can complete from server, which might cause too much
pressure on the server side.
Thanks,
Kuai
