diff mbox series

arm64: defconfig: Add IOSCHED_BFQ to the default configs

Message ID 20230224135230.2295826-1-linus.walleij@linaro.org (mailing list archive)
State New, archived
Headers show
Series arm64: defconfig: Add IOSCHED_BFQ to the default configs | expand

Commit Message

Linus Walleij Feb. 24, 2023, 1:52 p.m. UTC
ARM64 systems are often memory constrained and more often than not
use slow single-channel storage such as flash memory or MMC/SD-cards.
For any interactive systems (such as mobile phones, tablets,
chromebooks...) the BFQ I/O scheduler will be desireable.

Make sure the BFQ I/O scheduler is available on these systems.

ARM (32), Loongarch, MIPS, m68k, UM and S390 has also enabled BFQ in
their defconfigs, cf commit b495dfed706c4c5873c0dab8930ad6eb1d276a6c
"um: Cleanup CONFIG_IOSCHED_CFQ" where the motivation is that
it replaces the former CFQ scheduler.

Cc: linux-arm-kernel@lists.infradead.org
Cc: Catalin Marinas <catalin.marinas@arm.com>
Cc: Will Deacon <will@kernel.org>
Cc: Paolo Valente <paolo.valente@linaro.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: Ulf Hansson <ulf.hansson@linaro.org>
Cc: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
SoC maintainers: please apply this directly to the ARM64
defconfig branch if we have consensus.
---
 arch/arm64/configs/defconfig | 1 +
 1 file changed, 1 insertion(+)

Comments

Ulf Hansson Feb. 28, 2023, 7:55 p.m. UTC | #1
On Fri, 24 Feb 2023 at 14:52, Linus Walleij <linus.walleij@linaro.org> wrote:
>
> ARM64 systems are often memory constrained and more often than not
> use slow single-channel storage such as flash memory or MMC/SD-cards.
> For any interactive systems (such as mobile phones, tablets,
> chromebooks...) the BFQ I/O scheduler will be desireable.
>
> Make sure the BFQ I/O scheduler is available on these systems.
>
> ARM (32), Loongarch, MIPS, m68k, UM and S390 has also enabled BFQ in
> their defconfigs, cf commit b495dfed706c4c5873c0dab8930ad6eb1d276a6c
> "um: Cleanup CONFIG_IOSCHED_CFQ" where the motivation is that
> it replaces the former CFQ scheduler.
>
> Cc: linux-arm-kernel@lists.infradead.org
> Cc: Catalin Marinas <catalin.marinas@arm.com>
> Cc: Will Deacon <will@kernel.org>
> Cc: Paolo Valente <paolo.valente@linaro.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Cc: Arnd Bergmann <arnd@arndb.de>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

Seems reasonable to me!

Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org>

Kind regards
Uffe

> ---
> SoC maintainers: please apply this directly to the ARM64
> defconfig branch if we have consensus.
> ---
>  arch/arm64/configs/defconfig | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index 851e8f9be06d..503cc08cf112 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -126,6 +126,7 @@ CONFIG_CRYPTO_AES_ARM64_BS=m
>  CONFIG_JUMP_LABEL=y
>  CONFIG_MODULES=y
>  CONFIG_MODULE_UNLOAD=y
> +CONFIG_IOSCHED_BFQ=y
>  # CONFIG_CORE_DUMP_DEFAULT_ELF_HEADERS is not set
>  # CONFIG_COMPAT_BRK is not set
>  CONFIG_MEMORY_HOTPLUG=y
> --
> 2.34.1
>
diff mbox series

Patch

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index 851e8f9be06d..503cc08cf112 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -126,6 +126,7 @@  CONFIG_CRYPTO_AES_ARM64_BS=m
 CONFIG_JUMP_LABEL=y
 CONFIG_MODULES=y
 CONFIG_MODULE_UNLOAD=y
+CONFIG_IOSCHED_BFQ=y
 # CONFIG_CORE_DUMP_DEFAULT_ELF_HEADERS is not set
 # CONFIG_COMPAT_BRK is not set
 CONFIG_MEMORY_HOTPLUG=y