Buildresult: powerpc-next/arm64-defconfig/arm64-gcc12 built on Aug 24, 03:43

kisskb

Revisions | Branches | Compilers | Configs | Build Results | Build Failures |

Status:	OK
Date/Time:	Aug 24, 03:43
Duration:	0:03:45.071892
Builder:	ka4
Revision:	powerpc/iommu: Fix notifiers being shared by PCI and VIO buses (`8acecf9eda1846b9be8114d9cd665fbc38af223e)`
Target:	powerpc-next/arm64-defconfig/arm64-gcc12
Branch:	powerpc-next
Compiler:	arm64-gcc12 (aarch64-linux-gcc (GCC) 12.2.0 / GNU ld (GNU Binutils) 2.39)
Config:	defconfig (download)
Log:	Download original No warnings found in log. Full Log # git rev-parse -q --verify 8acecf9eda1846b9be8114d9cd665fbc38af223e^{commit} 8acecf9eda1846b9be8114d9cd665fbc38af223e already have revision, skipping fetch # git checkout -q -f -B kisskb 8acecf9eda1846b9be8114d9cd665fbc38af223e # git clean -qxdf # < git log -1 # commit 8acecf9eda1846b9be8114d9cd665fbc38af223e # Author: Russell Currey <ruscur@russell.cc> # Date: Wed Mar 22 14:53:22 2023 +1100 # # powerpc/iommu: Fix notifiers being shared by PCI and VIO buses # # fail_iommu_setup() registers the fail_iommu_bus_notifier struct to both # PCI and VIO buses. struct notifier_block is a linked list node, so this # causes any notifiers later registered to either bus type to also be # registered to the other since they share the same node. # # This causes issues in (at least) the vgaarb code, which registers a # notifier for PCI buses. pci_notify() ends up being called on a vio # device, converted with to_pci_dev() even though it's not a PCI device, # and finally makes a bad access in vga_arbiter_add_pci_device() as # discovered with KASAN: # # BUG: KASAN: slab-out-of-bounds in vga_arbiter_add_pci_device+0x60/0xe00 # Read of size 4 at addr c000000264c26fdc by task swapper/0/1 # # Call Trace: # dump_stack_lvl+0x1bc/0x2b8 (unreliable) # print_report+0x3f4/0xc60 # kasan_report+0x244/0x698 # __asan_load4+0xe8/0x250 # vga_arbiter_add_pci_device+0x60/0xe00 # pci_notify+0x88/0x444 # notifier_call_chain+0x104/0x320 # blocking_notifier_call_chain+0xa0/0x140 # device_add+0xac8/0x1d30 # device_register+0x58/0x80 # vio_register_device_node+0x9ac/0xce0 # vio_bus_scan_register_devices+0xc4/0x13c # __machine_initcall_pseries_vio_device_init+0x94/0xf0 # do_one_initcall+0x12c/0xaa8 # kernel_init_freeable+0xa48/0xba8 # kernel_init+0x64/0x400 # ret_from_kernel_thread+0x5c/0x64 # # Fix this by creating separate notifier_block structs for each bus type. # # Fixes: d6b9a81b2a45 ("powerpc: IOMMU fault injection") # Reported-by: Nageswara R Sastry <rnsastry@linux.ibm.com> # Signed-off-by: Russell Currey <ruscur@russell.cc> # Tested-by: Nageswara R Sastry <rnsastry@linux.ibm.com> # Reviewed-by: Andrew Donnellan <ajd@linux.ibm.com> # Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> # Link: https://msgid.link/20230322035322.328709-1-ruscur@russell.cc # < /opt/cross/kisskb/korg/gcc-12.2.0-nolibc/aarch64-linux/bin/aarch64-linux-gcc --version # < /opt/cross/kisskb/korg/gcc-12.2.0-nolibc/aarch64-linux/bin/aarch64-linux-ld --version # < git log --format=%s --max-count=1 8acecf9eda1846b9be8114d9cd665fbc38af223e # make -s -j 40 ARCH=arm64 O=/kisskb/build/powerpc-next_defconfig_arm64-gcc12 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-12.2.0-nolibc/aarch64-linux/bin/aarch64-linux- defconfig # < make -s -j 40 ARCH=arm64 O=/kisskb/build/powerpc-next_defconfig_arm64-gcc12 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-12.2.0-nolibc/aarch64-linux/bin/aarch64-linux- help # make -s -j 40 ARCH=arm64 O=/kisskb/build/powerpc-next_defconfig_arm64-gcc12 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-12.2.0-nolibc/aarch64-linux/bin/aarch64-linux- olddefconfig # make -s -j 40 ARCH=arm64 O=/kisskb/build/powerpc-next_defconfig_arm64-gcc12 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-12.2.0-nolibc/aarch64-linux/bin/aarch64-linux- Completed OK # rm -rf /kisskb/build/powerpc-next_defconfig_arm64-gcc12 # Build took: 0:03:45.071892

© Michael Ellerman 2006-2018.