Skip to contentSkip to conversation

blk-iocost: drop selecting undefined BLK_RQ_IO_DATA_LEN

PATCH Superseded Lukas Bulwahn 2 participants 2 messages Nov 12, 2021 – Nov 15, 2021

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 blk-iocost: drop selecting undefined BLK_RQ_IO_DATA_LEN

Commit cd006509b0a9 ("blk-iocost: account for IO size when testing latencies") selects the non-existing config BLK_RQ_IO_DATA_LEN in config BLK_CGROUP_IOCOST.

Hence, ./scripts/checkkconfigsymbols.py warns:

BLK_RQ_IO_DATA_LEN
Referencing files: block/Kconfig

Probably, this select is just some unintended left-over from an earlier draft version; BLK_RQ_IO_DATA_LEN was never defined in any commit in the repository. So, drop this dead select to an undefined config.

Fixes cd006509b0a9 ("blk-iocost: account for IO size when testing latencies")
---
 block/Kconfig | 1 -
 1 file changed, 1 deletion(-)
block/Kconfig +0 -1
index c6ce41a5e5b2..486bc6003b9e 100644
@@ -133,7 +133,6 @@ config BLK_CGROUP_FC_APPID
 config BLK_CGROUP_IOCOST
 	bool "Enable support for cost model based cgroup IO controller"
 	depends on BLK_CGROUP
-	select BLK_RQ_IO_DATA_LEN
 	select BLK_RQ_ALLOC_TIME
 	help
 	Enabling this option enables the .weight interface for cost
Signature · 1 line
2.26.2
Acked-by on v1 1/1
#1 lore raw
Quoted 9 lines · Lukas Bulwahn

Commit cd006509b0a9 ("blk-iocost: account for IO size when testing latencies") selects the non-existing config BLK_RQ_IO_DATA_LEN in config BLK_CGROUP_IOCOST.

Hence, ./scripts/checkkconfigsymbols.py warns:

BLK_RQ_IO_DATA_LEN
Referencing files: block/Kconfig

Quoted 6 lines

Probably, this select is just some unintended left-over from an earlier draft version; BLK_RQ_IO_DATA_LEN was never defined in any commit in the repository. So, drop this dead select to an undefined config.

Fixes cd006509b0a9 ("blk-iocost: account for IO size when testing latencies")

Thanks.

Signature · 1 line
tejun