Re: [PATCH v2 00/11] fw_devlink improvements

From: Marc Zyngier
Date: Mon Jan 30 2023 - 05:49:24 EST


On Mon, 30 Jan 2023 08:55:42 +0000,
Naresh Kamboju <naresh.kamboju@xxxxxxxxxx> wrote:
>
> Build test pass on arm, arm64, i386, mips, parisc, powerpc, riscv, s390, sh,
> sparc and x86_64.
>
> Boot and LTP smoke pass on qemu-arm64, qemu-armv7, qemu-i386 and qemu-x86_64.
> Boot failed on FVP.
>
> Reported-by: Linux Kernel Functional Testing <lkft@xxxxxxxxxx>
>
> Please refer following link for details of testing.
> FVP boot log failed.
> https://qa-reports.linaro.org/~anders.roxell/linux-mainline-patches/build/lore_kernel_org_linux-devicetree_20230127001141_407071-1-saravanak_google_com/testrun/14389034/suite/boot/test/gcc-12-lkftconfig-64k_page_size/details/
>
>
> [ 2.613437] Unable to handle kernel NULL pointer dereference at virtual address 0000000000000010
> [ 2.613628] Mem abort info:
> [ 2.613756] ESR = 0x0000000096000005
> [ 2.613904] EC = 0x25: DABT (current EL), IL = 32 bits
> [ 2.614071] SET = 0, FnV = 0
> [ 2.614215] EA = 0, S1PTW = 0
> [ 2.614358] FSC = 0x05: level 1 translation fault
> [ 2.614517] Data abort info:
> [ 2.614647] ISV = 0, ISS = 0x00000005
> [ 2.614792] CM = 0, WnR = 0
> [ 2.614934] [0000000000000010] user address but active_mm is swapper
> [ 2.615105] Internal error: Oops: 0000000096000005 [#1] PREEMPT SMP
> [ 2.615219] Modules linked in:
> [ 2.615310] CPU: 1 PID: 1 Comm: swapper/0 Not tainted 6.2.0-rc5 #1
> [ 2.615445] Hardware name: FVP Base RevC (DT)
> [ 2.615533] pstate: 61400009 (nZCv daif +PAN -UAO -TCO +DIT -SSBS BTYPE=--)
> [ 2.615685] pc : gpiochip_setup_dev (include/linux/err.h:41 include/linux/fwnode.h:201 drivers/gpio/gpiolib.c:586)
> [ 2.615816] lr : gpiochip_add_data_with_key (drivers/gpio/gpiolib.c:871)
> [ 2.615970] sp : ffff8000081af5e0
> [ 2.616051] x29: ffff8000081af5e0 x28: 0000000000000000 x27: ffff0008027cb5a0
> [ 2.616261] x26: 0000000000000000 x25: ffffd7c5d6745910 x24: ffff0008027f4800
> [ 2.616472] x23: 0000000000000000 x22: ffffd7c5d62b99a8 x21: 0000000000000202
> [ 2.616679] x20: 0000000000000000 x19: ffff0008027f4800 x18: ffffffffffffffff
> [ 2.616890] x17: ffffd7c5d6467928 x16: 0000000013e3690a x15: ffff8000081af3b0
> [ 2.617102] x14: ffff00080275cd8a x13: ffff00080275cd88 x12: 0000000000000001
> [ 2.617312] x11: 62726568746f6d3a x10: 0000000000000000 x9 : ffffd7c5d3b3ebe0
> [ 2.617522] x8 : ffff8000081af548 x7 : 0000000000000000 x6 : 0000000000000001
> [ 2.617727] x5 : 0000000000000000 x4 : ffff000800640000 x3 : ffffd7c5d62b99c8
> [ 2.617933] x2 : 0000000000000000 x1 : 0000000000000000 x0 : 0000000000000000
> [ 2.618138] Call trace:
> [ 2.618204] gpiochip_setup_dev (include/linux/err.h:41 include/linux/fwnode.h:201 drivers/gpio/gpiolib.c:586)
> [ 2.618337] gpiochip_add_data_with_key (drivers/gpio/gpiolib.c:871)
> [ 2.618493] devm_gpiochip_add_data_with_key (drivers/gpio/gpiolib-devres.c:478)
> [ 2.618654] bgpio_pdev_probe (drivers/gpio/gpio-mmio.c:793)
> [ 2.618785] platform_probe (drivers/base/platform.c:1401)
> [ 2.618928] really_probe (drivers/base/dd.c:560 drivers/base/dd.c:639)
> [ 2.619056] __driver_probe_device (drivers/base/dd.c:778)
> [ 2.619193] driver_probe_device (drivers/base/dd.c:808)
> [ 2.619329] __device_attach_driver (drivers/base/dd.c:937)
> [ 2.619464] bus_for_each_drv (drivers/base/bus.c:427)
> [ 2.619590] __device_attach (drivers/base/dd.c:1010)
> [ 2.619722] device_initial_probe (drivers/base/dd.c:1058)
> [ 2.619861] bus_probe_device (drivers/base/bus.c:489)
> [ 2.619988] device_add (drivers/base/core.c:3637)
> [ 2.620102] platform_device_add (drivers/base/platform.c:717)
> [ 2.620251] mfd_add_device (drivers/mfd/mfd-core.c:297)
> [ 2.620397] devm_mfd_add_devices (drivers/mfd/mfd-core.c:351 drivers/mfd/mfd-core.c:449)
> [ 2.620548] vexpress_sysreg_probe (drivers/mfd/vexpress-sysreg.c:115)
> [ 2.620672] platform_probe (drivers/base/platform.c:1401)
> [ 2.620814] really_probe (drivers/base/dd.c:560 drivers/base/dd.c:639)
> [ 2.620940] __driver_probe_device (drivers/base/dd.c:778)
> [ 2.621080] driver_probe_device (drivers/base/dd.c:808)
> [ 2.621216] __driver_attach (drivers/base/dd.c:1195)
> [ 2.621344] bus_for_each_dev (drivers/base/bus.c:301)
> [ 2.621467] driver_attach (drivers/base/dd.c:1212)
> [ 2.621596] bus_add_driver (drivers/base/bus.c:618)
> [ 2.621720] driver_register (drivers/base/driver.c:246)
> [ 2.621859] __platform_driver_register (drivers/base/platform.c:868)
> [ 2.622012] vexpress_sysreg_driver_init (drivers/mfd/vexpress-sysreg.c:134)
> [ 2.622145] do_one_initcall (init/main.c:1306)
> [ 2.622269] kernel_init_freeable (init/main.c:1378 init/main.c:1395 init/main.c:1414 init/main.c:1634)
> [ 2.622394] kernel_init (init/main.c:1526)
> [ 2.622531] ret_from_fork (arch/arm64/kernel/entry.S:864)
> [ 2.622692] Code: 910003fd a90153f3 aa0003f3 f9414c00 (f9400801)
> All code
> ========
> 0:* fd std <-- trapping instruction
> 1: 03 00 add (%rax),%eax
> 3: 91 xchg %eax,%ecx
> 4: f3 53 repz push %rbx
> 6: 01 a9 f3 03 00 aa add %ebp,-0x55fffc0d(%rcx)
> c: 00 4c 41 f9 add %cl,-0x7(%rcx,%rax,2)
> 10: 01 08 add %ecx,(%rax)
> 12: 40 f9 rex stc

Could you please fix your scripts so that they report something that
matches the tested architecture? I like x86 asm as much as the next
guy, but this is an arm64 crash... :-/

Thanks,

M.

--
Without deviation from the norm, progress is not possible.