]> git.baikalelectronics.ru Git - kernel.git/commit
block, bfq: add weight symlink to the bfq.weight cgroup parameter
authorAngelo Ruocco <angeloruocco90@gmail.com>
Tue, 21 May 2019 08:01:55 +0000 (10:01 +0200)
committerJens Axboe <axboe@kernel.dk>
Fri, 7 Jun 2019 07:29:40 +0000 (01:29 -0600)
commitde04f5b3cef69789b225fb978cad082d0937937b
tree51d7ce57bef53ddb826aa95e6c7c00f80a15b9fb
parentee1af89944701c2c87a9366db69acee6f5931053
block, bfq: add weight symlink to the bfq.weight cgroup parameter

Many userspace tools and services use the proportional-share policy of
the blkio/io cgroups controller. The CFQ I/O scheduler implemented
this policy for the legacy block layer. To modify the weight of a
group in case CFQ was in charge, the 'weight' parameter of the group
must be modified. On the other hand, the BFQ I/O scheduler implements
the same policy in blk-mq, but, with BFQ, the parameter to modify has
a different name: bfq.weight (forced choice until legacy block was
present, because two different policies cannot share a common parameter
in cgroups).

Due to CFQ legacy, most if not all userspace configurations still use
the parameter 'weight', and for the moment do not seem likely to be
changed. But, when CFQ went away with legacy block, such a parameter
ceased to exist.

So, a simple workaround has been proposed [1] to make all
configurations work: add a symlink, named weight, to bfq.weight. This
commit adds such a symlink.

[1] https://lkml.org/lkml/2019/4/8/555

Suggested-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Angelo Ruocco <angeloruocco90@gmail.com>
Signed-off-by: Paolo Valente <paolo.valente@linaro.org>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
block/bfq-cgroup.c