[PATCH 0/2] ARM: Refresh Raspberry Pi defconfig and enable SMP support

[PATCH 0/2] ARM: Refresh Raspberry Pi defconfig and enable SMP support

[Jonathan Neuschäfer]

This series enables SMP support in bcm2835_defconfig, so that Linux can
use all cores of later Raspberry Pi SoCs.

Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
---
Jonathan Neuschäfer (2):
      ARM: Refresh bcm2835_defconfig
      ARM: bcm2835_defconfig: Enable SMP support

 arch/arm/configs/bcm2835_defconfig | 15 +++------------
 1 file changed, 3 insertions(+), 12 deletions(-)
---
base-commit: 22a40d14b572deb80c0648557f4bd502d7e83826
change-id: 20240704-raspi-config-83210de0f5f0

Best regards,
--
Jonathan Neuschäfer <j.neuschaefer@gmx.net>

[PATCH 1/2] ARM: Refresh bcm2835_defconfig

[Jonathan Neuschäfer]

Regenerate bcm2835_defconfig with "make savedefconfig" in order to pick
up Kconfig changes that happened since the last refresh.

Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
---
 arch/arm/configs/bcm2835_defconfig | 11 +----------
 1 file changed, 1 insertion(+), 10 deletions(-)

diff --git a/arch/arm/configs/bcm2835_defconfig b/arch/arm/configs/bcm2835_defconfig
index b5f0bd8dd53699..da49dcfd359050 100644
--- a/arch/arm/configs/bcm2835_defconfig
+++ b/arch/arm/configs/bcm2835_defconfig
@@ -9,7 +9,6 @@ CONFIG_LOG_BUF_SHIFT=18
 CONFIG_CFS_BANDWIDTH=y
 CONFIG_RT_GROUP_SCHED=y
 CONFIG_CGROUP_FREEZER=y
-CONFIG_CPUSETS=y
 CONFIG_CGROUP_DEVICE=y
 CONFIG_CGROUP_CPUACCT=y
 CONFIG_CGROUP_PERF=y
@@ -21,14 +20,10 @@ CONFIG_CC_OPTIMIZE_FOR_SIZE=y
 CONFIG_EXPERT=y
 CONFIG_KALLSYMS_ALL=y
 CONFIG_PROFILING=y
-CONFIG_CC_STACKPROTECTOR_REGULAR=y
-CONFIG_CRASH_DUMP=y
+CONFIG_KEXEC=y
 CONFIG_ARCH_MULTI_V6=y
 CONFIG_ARCH_BCM=y
 CONFIG_ARCH_BCM2835=y
-CONFIG_AEABI=y
-CONFIG_SECCOMP=y
-CONFIG_KEXEC=y
 CONFIG_CPU_FREQ=y
 CONFIG_CPU_FREQ_STAT=y
 CONFIG_CPU_FREQ_DEFAULT_GOV_CONSERVATIVE=y
@@ -103,13 +98,11 @@ CONFIG_REGULATOR=y
 CONFIG_REGULATOR_FIXED_VOLTAGE=y
 CONFIG_REGULATOR_GPIO=y
 CONFIG_MEDIA_SUPPORT=y
-CONFIG_MEDIA_CAMERA_SUPPORT=y
 CONFIG_DRM=y
 CONFIG_DRM_V3D=y
 CONFIG_DRM_VC4=y
 CONFIG_DRM_SIMPLEDRM=y
 CONFIG_FB=y
-CONFIG_FRAMEBUFFER_CONSOLE=y
 CONFIG_SOUND=y
 CONFIG_SND=y
 CONFIG_SND_SOC=y
@@ -178,8 +171,6 @@ CONFIG_PRINTK_TIME=y
 CONFIG_BOOT_PRINTK_DELAY=y
 CONFIG_DYNAMIC_DEBUG=y
 CONFIG_DEBUG_INFO_DWARF_TOOLCHAIN_DEFAULT=y
-# CONFIG_ENABLE_MUST_CHECK is not set
-CONFIG_LOCKUP_DETECTOR=y
 CONFIG_DEBUG_FS=y
 CONFIG_KGDB=y
 CONFIG_KGDB_KDB=y

--
2.43.0

Re: [PATCH 1/2] ARM: Refresh bcm2835_defconfig

[Florian Fainelli]

On 7/7/24 14:48, Jonathan Neuschäfer wrote:
> Regenerate bcm2835_defconfig with "make savedefconfig" in order to pick
> up Kconfig changes that happened since the last refresh.
> 
> Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>

Stefan, any objections to taking this change in isolation?

> ---
>   arch/arm/configs/bcm2835_defconfig | 11 +----------
>   1 file changed, 1 insertion(+), 10 deletions(-)
> 
> diff --git a/arch/arm/configs/bcm2835_defconfig b/arch/arm/configs/bcm2835_defconfig
> index b5f0bd8dd53699..da49dcfd359050 100644
> --- a/arch/arm/configs/bcm2835_defconfig
> +++ b/arch/arm/configs/bcm2835_defconfig
> @@ -9,7 +9,6 @@ CONFIG_LOG_BUF_SHIFT=18
>   CONFIG_CFS_BANDWIDTH=y
>   CONFIG_RT_GROUP_SCHED=y
>   CONFIG_CGROUP_FREEZER=y
> -CONFIG_CPUSETS=y
>   CONFIG_CGROUP_DEVICE=y
>   CONFIG_CGROUP_CPUACCT=y
>   CONFIG_CGROUP_PERF=y
> @@ -21,14 +20,10 @@ CONFIG_CC_OPTIMIZE_FOR_SIZE=y
>   CONFIG_EXPERT=y
>   CONFIG_KALLSYMS_ALL=y
>   CONFIG_PROFILING=y
> -CONFIG_CC_STACKPROTECTOR_REGULAR=y
> -CONFIG_CRASH_DUMP=y
> +CONFIG_KEXEC=y
>   CONFIG_ARCH_MULTI_V6=y
>   CONFIG_ARCH_BCM=y
>   CONFIG_ARCH_BCM2835=y
> -CONFIG_AEABI=y
> -CONFIG_SECCOMP=y
> -CONFIG_KEXEC=y
>   CONFIG_CPU_FREQ=y
>   CONFIG_CPU_FREQ_STAT=y
>   CONFIG_CPU_FREQ_DEFAULT_GOV_CONSERVATIVE=y
> @@ -103,13 +98,11 @@ CONFIG_REGULATOR=y
>   CONFIG_REGULATOR_FIXED_VOLTAGE=y
>   CONFIG_REGULATOR_GPIO=y
>   CONFIG_MEDIA_SUPPORT=y
> -CONFIG_MEDIA_CAMERA_SUPPORT=y
>   CONFIG_DRM=y
>   CONFIG_DRM_V3D=y
>   CONFIG_DRM_VC4=y
>   CONFIG_DRM_SIMPLEDRM=y
>   CONFIG_FB=y
> -CONFIG_FRAMEBUFFER_CONSOLE=y
>   CONFIG_SOUND=y
>   CONFIG_SND=y
>   CONFIG_SND_SOC=y
> @@ -178,8 +171,6 @@ CONFIG_PRINTK_TIME=y
>   CONFIG_BOOT_PRINTK_DELAY=y
>   CONFIG_DYNAMIC_DEBUG=y
>   CONFIG_DEBUG_INFO_DWARF_TOOLCHAIN_DEFAULT=y
> -# CONFIG_ENABLE_MUST_CHECK is not set
> -CONFIG_LOCKUP_DETECTOR=y
>   CONFIG_DEBUG_FS=y
>   CONFIG_KGDB=y
>   CONFIG_KGDB_KDB=y
> 
> --
> 2.43.0
> 


-- 
Florian

Re: [PATCH 1/2] ARM: Refresh bcm2835_defconfig

[Stefan Wahren]

Hi Florian,

Am 11.10.24 um 00:48 schrieb Florian Fainelli:
> On 7/7/24 14:48, Jonathan Neuschäfer wrote:
>> Regenerate bcm2835_defconfig with "make savedefconfig" in order to pick
>> up Kconfig changes that happened since the last refresh.
>>
>> Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
>
> Stefan, any objections to taking this change in isolation?
sure this would apply, but isn't this outdated yet?

Btw please look at

https://lore.kernel.org/linux-arm-kernel/20240910-fix-riscv-rt_group_sched-v3-4-486e75e5ae6d@gmail.com/

[PATCH 2/2] ARM: bcm2835_defconfig: Enable SMP support

[Jonathan Neuschäfer]

Since there is only one Raspberry Pi related defconfig in the mainline
kernel, it's useful to have to work well on all 32-bit Raspberry Pis.
To that end, this patch enables CONFIG_SMP and CONFIG_SMP_ON_UP, which
allows the kernel to run well on multi- and single-CPU systems.

PM and suspend support is necessary in order to keep KEXEC enabled.

Tested on:
- Raspberry Pi 2 Model B V1.1 (BCM2836, SMP)
- Raspberry Pi Model B (BCM2835, UP)

Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
---
 arch/arm/configs/bcm2835_defconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/configs/bcm2835_defconfig b/arch/arm/configs/bcm2835_defconfig
index da49dcfd359050..d505058715b66f 100644
--- a/arch/arm/configs/bcm2835_defconfig
+++ b/arch/arm/configs/bcm2835_defconfig
@@ -24,6 +24,7 @@ CONFIG_KEXEC=y
 CONFIG_ARCH_MULTI_V6=y
 CONFIG_ARCH_BCM=y
 CONFIG_ARCH_BCM2835=y
+CONFIG_SMP=y
 CONFIG_CPU_FREQ=y
 CONFIG_CPU_FREQ_STAT=y
 CONFIG_CPU_FREQ_DEFAULT_GOV_CONSERVATIVE=y
@@ -33,8 +34,6 @@ CONFIG_CPU_FREQ_GOV_ONDEMAND=y
 CONFIG_CPUFREQ_DT=y
 CONFIG_ARM_RASPBERRYPI_CPUFREQ=y
 CONFIG_VFP=y
-# CONFIG_SUSPEND is not set
-CONFIG_PM=y
 CONFIG_JUMP_LABEL=y
 CONFIG_MODULES=y
 CONFIG_MODULE_UNLOAD=y
@@ -175,6 +174,7 @@ CONFIG_DEBUG_FS=y
 CONFIG_KGDB=y
 CONFIG_KGDB_KDB=y
 CONFIG_DEBUG_MEMORY_INIT=y
+# CONFIG_RCU_TRACE is not set
 CONFIG_FUNCTION_PROFILER=y
 CONFIG_STACK_TRACER=y
 CONFIG_SCHED_TRACER=y

--
2.43.0

Re: [PATCH 2/2] ARM: bcm2835_defconfig: Enable SMP support

[Stefan Wahren]

Hi Jonathan,

Am 07.07.24 um 23:48 schrieb Jonathan Neuschäfer:
> Since there is only one Raspberry Pi related defconfig in the mainline
> kernel, it's useful to have to work well on all 32-bit Raspberry Pis.
this wasn't intention of bcm2835_defconfig. It's more focused on BCM2835
SOC and kernel-ci (both non-SMP). If you want to use the BCM2836 &
BCM2837 (incl. SMP), please use multi_v7_defconfig instead. Applying
this change would decrease the test coverage.
> To that end, this patch enables CONFIG_SMP and CONFIG_SMP_ON_UP, which
> allows the kernel to run well on multi- and single-CPU systems.
>
> PM and suspend support is necessary in order to keep KEXEC enabled.
In this case the subject contains only half of the truth.

Regards
> - Raspberry Pi 2 Model B V1.1 (BCM2836, SMP)
> - Raspberry Pi Model B (BCM2835, UP)
>
> Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
> ---
>   arch/arm/configs/bcm2835_defconfig | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/configs/bcm2835_defconfig b/arch/arm/configs/bcm2835_defconfig
> index da49dcfd359050..d505058715b66f 100644
> --- a/arch/arm/configs/bcm2835_defconfig
> +++ b/arch/arm/configs/bcm2835_defconfig
> @@ -24,6 +24,7 @@ CONFIG_KEXEC=y
>   CONFIG_ARCH_MULTI_V6=y
>   CONFIG_ARCH_BCM=y
>   CONFIG_ARCH_BCM2835=y
> +CONFIG_SMP=y
>   CONFIG_CPU_FREQ=y
>   CONFIG_CPU_FREQ_STAT=y
>   CONFIG_CPU_FREQ_DEFAULT_GOV_CONSERVATIVE=y
> @@ -33,8 +34,6 @@ CONFIG_CPU_FREQ_GOV_ONDEMAND=y
>   CONFIG_CPUFREQ_DT=y
>   CONFIG_ARM_RASPBERRYPI_CPUFREQ=y
>   CONFIG_VFP=y
> -# CONFIG_SUSPEND is not set
> -CONFIG_PM=y
>   CONFIG_JUMP_LABEL=y
>   CONFIG_MODULES=y
>   CONFIG_MODULE_UNLOAD=y
> @@ -175,6 +174,7 @@ CONFIG_DEBUG_FS=y
>   CONFIG_KGDB=y
>   CONFIG_KGDB_KDB=y
>   CONFIG_DEBUG_MEMORY_INIT=y
> +# CONFIG_RCU_TRACE is not set
>   CONFIG_FUNCTION_PROFILER=y
>   CONFIG_STACK_TRACER=y
>   CONFIG_SCHED_TRACER=y
>
> --
> 2.43.0
>
>

Re: [PATCH 2/2] ARM: bcm2835_defconfig: Enable SMP support

[J. Neuschäfer]

On Mon, Jul 08, 2024 at 05:54:25PM +0200, Stefan Wahren wrote:
> Hi Jonathan,
>
> Am 07.07.24 um 23:48 schrieb Jonathan Neuschäfer:
> > Since there is only one Raspberry Pi related defconfig in the mainline
> > kernel, it's useful to have to work well on all 32-bit Raspberry Pis.
> this wasn't intention of bcm2835_defconfig. It's more focused on BCM2835
> SOC and kernel-ci (both non-SMP). If you want to use the BCM2836 &
> BCM2837 (incl. SMP), please use multi_v7_defconfig instead. Applying
> this change would decrease the test coverage.

Ah, fair enough.


Jonathan

Re: [PATCH 2/2] ARM: bcm2835_defconfig: Enable SMP support

[Florian Fainelli]

On 7/9/24 12:33, J. Neuschäfer wrote:
> On Mon, Jul 08, 2024 at 05:54:25PM +0200, Stefan Wahren wrote:
>> Hi Jonathan,
>>
>> Am 07.07.24 um 23:48 schrieb Jonathan Neuschäfer:
>>> Since there is only one Raspberry Pi related defconfig in the mainline
>>> kernel, it's useful to have to work well on all 32-bit Raspberry Pis.
>> this wasn't intention of bcm2835_defconfig. It's more focused on BCM2835
>> SOC and kernel-ci (both non-SMP). If you want to use the BCM2836 &
>> BCM2837 (incl. SMP), please use multi_v7_defconfig instead. Applying
>> this change would decrease the test coverage.
> 
> Ah, fair enough.

It is fair that CONFIG_SMP_ON_UP is different from not enabling 
CONFIG_SMP, but if we care to test CONFIG_SMP disabled, should not there 
be a specific modification of multi_v7_defconfig or another 
configuration specifically for that purpose?
-- 
Florian