Re: [PATCH -next RFC] blk-throttle: remove CONFIG_BLK_DEV_THROTTLING_LOW

From: Yu Kuai
Date: Sat Mar 23 2024 - 00:10:16 EST


Hi !

在 2024/03/23 11:17, Randy Dunlap 写道:
Hi--

On 3/22/24 19:53, Yu Kuai wrote:
From: Yu Kuai <yukuai3@xxxxxxxxxx>

One the one hand, it's marked EXPERIMENTAL since 2017, and looks like
there are no users since then, and no testers and no developers, it's
just not active at all.

On the other hand, even if the config is disabled, there are still many
fields in throtl_grp and throtl_data and many functions that are only
used for throtl low.

At last, currently blk-throtl is initialized during disk initialization,
and destroyed during disk removal, and it exposes many functions to be
called directly from block layer. Hence I'm planning to optimize
blk-throtl and finially support building it as kernel module. Remove
throtl low will make the work much easier.

Signed-off-by: Yu Kuai <yukuai3@xxxxxxxxxx>
---
block/Kconfig | 11 -
block/bio.c | 1 -
block/blk-stat.c | 3 -
block/blk-sysfs.c | 7 -
block/blk-throttle.c | 901 ++-----------------------------------------
block/blk-throttle.h | 26 +-
block/blk.h | 11 -
7 files changed, 45 insertions(+), 915 deletions(-)

Here are 2 more places to patch:

Documentation/ABI/stable/sysfs-block: CONFIG_BLK_DEV_THROTTLING_LOW is enabled.
arch/loongarch/configs/loongson3_defconfig:CONFIG_BLK_DEV_THROTTLING_LOW=y

Thanks for the notice, I'll update the patch if people agree to remove
throttle low.

Thanks,
Kuai