blk-throttle: Fix io statistics for cgroup v1
authorJinke Han <hanjinke.666@bytedance.com>
Sun, 7 May 2023 17:06:31 +0000 (01:06 +0800)
committerJens Axboe <axboe@kernel.dk>
Sun, 25 Jun 2023 14:00:39 +0000 (08:00 -0600)
commitad7c3b41e86b59943a903d23c7b037d820e6270c
treec796b16a7c7487459ecb32bc10cd33259d8a9ae9
parent2c5555983bd27d24162534b682b10654639a5576
blk-throttle: Fix io statistics for cgroup v1

After commit f382fb0bcef4 ("block: remove legacy IO schedulers"),
blkio.throttle.io_serviced and blkio.throttle.io_service_bytes become
the only stable io stats interface of cgroup v1, and these statistics
are done in the blk-throttle code. But the current code only counts the
bios that are actually throttled. When the user does not add the throttle
limit, the io stats for cgroup v1 has nothing. I fix it according to the
statistical method of v2, and made it count all ios accurately.

Fixes: a7b36ee6ba29 ("block: move blk-throtl fast path inline")
Tested-by: Andrea Righi <andrea.righi@canonical.com>
Signed-off-by: Jinke Han <hanjinke.666@bytedance.com>
Acked-by: Muchun Song <songmuchun@bytedance.com>
Acked-by: Tejun Heo <tj@kernel.org>
Link: https://lore.kernel.org/r/20230507170631.89607-1-hanjinke.666@bytedance.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
block/blk-cgroup.c
block/blk-throttle.c
block/blk-throttle.h