Re: [PATCH] parisc: Switch from GENERIC_CPU_DEVICES to GENERIC_ARCH_TOPOLOGY

From: Helge Deller
Date: Fri Apr 01 2022 - 12:53:02 EST


Hi Günter,

On 4/1/22 17:41, Guenter Roeck wrote:
> On Thu, Mar 24, 2022 at 07:46:50PM +0100, Helge Deller wrote:
>> Switch away from the own cpu topology code to common code which is used
>> by ARM64 and RISCV. That will allow us to enable CPU hotplug later on.
>>
>> Signed-off-by: Helge Deller <deller@xxxxxx>
>
> This patch results in the following traceback when
> booting generic-32bit_defconfig - SMP in qemu.

That's strange, because I just built this generic-32bit_defconfig myself and
it boots up nicely in qemu for me. The only thing missing is CONFIG_CGROUPS=y so that
systemd can start.

I'm not sure how I can debug/reproduce your bug report...

Helge

> [ 9.432811] ------------[ cut here ]------------
> [ 9.433305] WARNING: CPU: 0 PID: 1 at fs/sysfs/group.c:115 internal_create_group+0x360/0x394
> [ 9.433825] Modules linked in:
> [ 9.434267] CPU: 0 PID: 1 Comm: swapper Not tainted 5.17.0-32bit+ #1
> [ 9.434621] Hardware name: 9000/778/B160L
> [ 9.434889]
> [ 9.435014] YZrvWESTHLNXBCVMcbcbcbcbOGFRQPDI
> [ 9.435201] PSW: 00000000000001001011111100001111 Not tainted
> [ 9.435392] r00-03 0004bf0f 114305c0 103518bc 11430600
> [ 9.435578] r04-07 0000003c 10db3d20 10b27ecc 00000000
> [ 9.435725] r08-11 106f6c88 00000001 00000000 00000000
> [ 9.435876] r12-15 10c59000 00000000 00000001 00000000
> [ 9.436022] r16-19 00000028 00080000 00000000 10db4ad8
> [ 9.436168] r20-23 0000012c 10eb0a00 106f6c88 00000000
> [ 9.436314] r24-27 10b27ecc 00000000 00000000 10cf00e0
> [ 9.436460] r28-31 00000000 ae147af8 114306c0 101a66d8
> [ 9.436625] sr00-03 00000000 00000000 00000000 00000000
> [ 9.436785] sr04-07 00000000 00000000 00000000 00000000
> [ 9.436942]
> [ 9.437006] IASQ: 00000000 00000000 IAOQ: 1035186c 10351870
> [ 9.437161] IIR: 03ffe01f ISR: 00000000 IOR: 10e02d68
> [ 9.437307] CPU: 0 CR30: 1141a010 CR31: 00000000
> [ 9.437470] ORIG_R28: 00000000
> [ 9.437579] IAOQ[0]: internal_create_group+0x360/0x394
> [ 9.437753] IAOQ[1]: internal_create_group+0x364/0x394
> [ 9.437913] RP(r2): sysfs_create_group+0x1c/0x2c
> [ 9.438180] Backtrace:
> [ 9.438892] [<103518bc>] sysfs_create_group+0x1c/0x2c
> [ 9.439116] [<106f6cb0>] topology_add_dev+0x28/0x38
> [ 9.439407] [<101a66d8>] cpuhp_invoke_callback.constprop.0+0xcc/0x17c
> [ 9.440321] [<101a68b0>] __cpuhp_setup_state_cpuslocked+0x128/0x210
> [ 9.440545] [<101a69c0>] __cpuhp_setup_state+0x28/0x38
> [ 9.440657] [<10126fa4>] topology_sysfs_init+0x3c/0x4c
> [ 9.440795] [<1018510c>] do_one_initcall+0x68/0x510
> [ 9.440918] [<10101420>] kernel_init_freeable+0x1d4/0x268
> [ 9.441040] [<10aa7bdc>] kernel_init+0x2c/0x150
> [ 9.441191] [<1018201c>] ret_from_kernel_thread+0x1c/0x24
> [ 9.441574]
> [ 9.441716] ---[ end trace 0000000000000000 ]---
>
> Reverting the offending commit fixes the problem.
> Bisect log is attached.
>
> Guenter
>
> ---
> # bad: [e8b767f5e04097aaedcd6e06e2270f9fe5282696] Merge tag 'for-linus-5.18-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/rw/uml
> # good: [d888c83fcec75194a8a48ccd283953bdba7b2550] fs: fix fd table size alignment properly
> git bisect start 'HEAD' 'd888c83fcec7'
> # good: [2a44cdaa01837355b14b9221e87d75963846296c] Merge tag 'dmaengine-5.18-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/vkoul/dmaengine
> git bisect good 2a44cdaa01837355b14b9221e87d75963846296c
> # bad: [b8321ed4a40c02054f930ca59d3570caa27bc86c] Merge tag 'kbuild-v5.18-v2' of git://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-kbuild
> git bisect bad b8321ed4a40c02054f930ca59d3570caa27bc86c
> # bad: [93235e3df29c084a37e0daed17801c6adfce4cb6] Merge tag 'v5.18-p1' of git://git.kernel.org/pub/scm/linux/kernel/git/herbert/crypto-2.6
> git bisect bad 93235e3df29c084a37e0daed17801c6adfce4cb6
> # good: [84b3b8df51e26b8bc2c7178b068ef447473d4609] dt-bindings: pwm: renesas,pwm: Include generic PWM schema
> git bisect good 84b3b8df51e26b8bc2c7178b068ef447473d4609
> # bad: [a9fe7fa7d874a536e0540469f314772c054a0323] parisc: Fix patch code locking and flushing
> git bisect bad a9fe7fa7d874a536e0540469f314772c054a0323
> # bad: [95370b4031ec67f9749e5873ae7139a53cc6bf53] parisc: Move store_cpu_topology() into text section
> git bisect bad 95370b4031ec67f9749e5873ae7139a53cc6bf53
> # good: [ed9794812b0a61f62317991c5c3e6c78749a0bdc] parisc: Detect hppa-suse-linux-gcc compiler for cross-building
> git bisect good ed9794812b0a61f62317991c5c3e6c78749a0bdc
> # good: [1e93848a385ed2c2df9cb246b073dc8c66d10793] parisc: Ensure set_firmware_width() is called only once
> git bisect good 1e93848a385ed2c2df9cb246b073dc8c66d10793
> # bad: [62773112acc55d29727465d075fc61ed08a0a532] parisc: Switch from GENERIC_CPU_DEVICES to GENERIC_ARCH_TOPOLOGY
> git bisect bad 62773112acc55d29727465d075fc61ed08a0a532
> # first bad commit: [62773112acc55d29727465d075fc61ed08a0a532] parisc: Switch from GENERIC_CPU_DEVICES to GENERIC_ARCH_TOPOLOGY