[PATCH] ARM: defconfig: Add IOSCHED_BFQ to the default configs

[PATCH] ARM: defconfig: Add IOSCHED_BFQ to the default configs

[Linus Walleij]

ARM 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.

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: Ulf Hansson <ulf.hansson@linaro.org>
Cc: Paolo Valente <paolo.valente@linaro.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
 arch/arm/configs/multi_v5_defconfig | 1 +
 arch/arm/configs/multi_v7_defconfig | 1 +
 2 files changed, 2 insertions(+)

diff --git a/arch/arm/configs/multi_v5_defconfig b/arch/arm/configs/multi_v5_defconfig
index 60fc52b95690..4cf455c3bdee 100644
--- a/arch/arm/configs/multi_v5_defconfig
+++ b/arch/arm/configs/multi_v5_defconfig
@@ -49,6 +49,7 @@ CONFIG_ARM_KIRKWOOD_CPUIDLE=y
 CONFIG_KPROBES=y
 CONFIG_MODULES=y
 CONFIG_MODULE_UNLOAD=y
+CONFIG_IOSCHED_BFQ=y
 CONFIG_NET=y
 CONFIG_PACKET=y
 CONFIG_UNIX=y
diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
index ee184eb37adc..7a0279a078fc 100644
--- a/arch/arm/configs/multi_v7_defconfig
+++ b/arch/arm/configs/multi_v7_defconfig
@@ -146,6 +146,7 @@ CONFIG_MODULES=y
 CONFIG_MODULE_UNLOAD=y
 CONFIG_PARTITION_ADVANCED=y
 CONFIG_CMDLINE_PARTITION=y
+CONFIG_IOSCHED_BFQ=y
 CONFIG_NET=y
 CONFIG_PACKET=y
 CONFIG_UNIX=y
-- 
2.34.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [PATCH] ARM: defconfig: Add IOSCHED_BFQ to the default configs

[Ulf Hansson]

On Thu, 2 Feb 2023 at 14:11, Linus Walleij <linus.walleij@linaro.org> wrote:
>
> ARM 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.
>
> 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: Ulf Hansson <ulf.hansson@linaro.org>
> Cc: Paolo Valente <paolo.valente@linaro.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

Seems reasonable to me! Feel free to add:

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

I assume you are planning to send a patch for the arm64 defconfig too?
At least to me, that seems like a reasonable thing to do as well.

Kind regards
Uffe


> ---
>  arch/arm/configs/multi_v5_defconfig | 1 +
>  arch/arm/configs/multi_v7_defconfig | 1 +
>  2 files changed, 2 insertions(+)
>
> diff --git a/arch/arm/configs/multi_v5_defconfig b/arch/arm/configs/multi_v5_defconfig
> index 60fc52b95690..4cf455c3bdee 100644
> --- a/arch/arm/configs/multi_v5_defconfig
> +++ b/arch/arm/configs/multi_v5_defconfig
> @@ -49,6 +49,7 @@ CONFIG_ARM_KIRKWOOD_CPUIDLE=y
>  CONFIG_KPROBES=y
>  CONFIG_MODULES=y
>  CONFIG_MODULE_UNLOAD=y
> +CONFIG_IOSCHED_BFQ=y
>  CONFIG_NET=y
>  CONFIG_PACKET=y
>  CONFIG_UNIX=y
> diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
> index ee184eb37adc..7a0279a078fc 100644
> --- a/arch/arm/configs/multi_v7_defconfig
> +++ b/arch/arm/configs/multi_v7_defconfig
> @@ -146,6 +146,7 @@ CONFIG_MODULES=y
>  CONFIG_MODULE_UNLOAD=y
>  CONFIG_PARTITION_ADVANCED=y
>  CONFIG_CMDLINE_PARTITION=y
> +CONFIG_IOSCHED_BFQ=y
>  CONFIG_NET=y
>  CONFIG_PACKET=y
>  CONFIG_UNIX=y
> --
> 2.34.1
>

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [PATCH] ARM: defconfig: Add IOSCHED_BFQ to the default configs

[Linus Walleij]

On Thu, Feb 2, 2023 at 2:32 PM Ulf Hansson <ulf.hansson@linaro.org> wrote:
> On Thu, 2 Feb 2023 at 14:11, Linus Walleij <linus.walleij@linaro.org> wrote:
> >
> > ARM 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.
> >
> > 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: Ulf Hansson <ulf.hansson@linaro.org>
> > Cc: Paolo Valente <paolo.valente@linaro.org>
> > Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> > Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
>
> Seems reasonable to me! Feel free to add:
>
> Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org>
>
> I assume you are planning to send a patch for the arm64 defconfig too?
> At least to me, that seems like a reasonable thing to do as well.

Yup provided we have consesus to turn it on for Arm32 I will follow
up with a patch to Arm64.

Yours,
Linus Walleij

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [PATCH] ARM: defconfig: Add IOSCHED_BFQ to the default configs

[Arnd Bergmann]

On Thu, Feb 2, 2023, at 14:59, Linus Walleij wrote:
> On Thu, Feb 2, 2023 at 2:32 PM Ulf Hansson <ulf.hansson@linaro.org> wrote:
>> On Thu, 2 Feb 2023 at 14:11, Linus Walleij <linus.walleij@linaro.org> wrote:
>> >
>> > ARM 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.
>> >
>> > 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: Ulf Hansson <ulf.hansson@linaro.org>
>> > Cc: Paolo Valente <paolo.valente@linaro.org>
>> > Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>> > Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
>>
>> Seems reasonable to me! Feel free to add:
>>
>> Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org>

Acked-by: Arnd Bergmann <arnd@arndb.de>

>> I assume you are planning to send a patch for the arm64 defconfig too?
>> At least to me, that seems like a reasonable thing to do as well.
>
> Yup provided we have consesus to turn it on for Arm32 I will follow
> up with a patch to Arm64.

Feel free to send both to soc@kernel.org directly, if there are no
objections.

     Arnd

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel