# git rev-parse -q --verify f9bc9bbe8afdf83412728f0b464979a72a3b9ec2^{commit} f9bc9bbe8afdf83412728f0b464979a72a3b9ec2 already have revision, skipping fetch # git checkout -q -f -B kisskb f9bc9bbe8afdf83412728f0b464979a72a3b9ec2 # git clean -qxdf # < git log -1 # commit f9bc9bbe8afdf83412728f0b464979a72a3b9ec2 # Author: Nicholas Piggin # Date: Mon Oct 16 22:43:00 2023 +1000 # # powerpc/qspinlock: Fix stale propagated yield_cpu # # yield_cpu is a sample of a preempted lock holder that gets propagated # back through the queue. Queued waiters use this to yield to the # preempted lock holder without continually sampling the lock word (which # would defeat the purpose of MCS queueing by bouncing the cache line). # # The problem is that yield_cpu can become stale. It can take some time to # be passed down the chain, and if any queued waiter gets preempted then # it will cease to propagate the yield_cpu to later waiters. # # This can result in yielding to a CPU that no longer holds the lock, # which is bad, but particularly if it is currently in H_CEDE (idle), # then it appears to be preempted and some hypervisors (PowerVM) can # cause very long H_CONFER latencies waiting for H_CEDE wakeup. This # results in latency spikes and hard lockups on oversubscribed # partitions with lock contention. # # This is a minimal fix. Before yielding to yield_cpu, sample the lock # word to confirm yield_cpu is still the owner, and bail out of it is not. # # Thanks to a bunch of people who reported this and tracked down the # exact problem using tracepoints and dispatch trace logs. # # Fixes: 28db61e207ea ("powerpc/qspinlock: allow propagation of yield CPU down the queue") # Cc: stable@vger.kernel.org # v6.2+ # Reported-by: Srikar Dronamraju # Reported-by: Laurent Dufour # Reported-by: Shrikanth Hegde # Debugged-by: "Nysal Jan K.A" # Signed-off-by: Nicholas Piggin # Tested-by: Shrikanth Hegde # Signed-off-by: Michael Ellerman # Link: https://msgid.link/20231016124305.139923-2-npiggin@gmail.com # < /opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux-gcc --version # < /opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux-ld --version # < git log --format=%s --max-count=1 f9bc9bbe8afdf83412728f0b464979a72a3b9ec2 # make -s -j 40 ARCH=powerpc O=/kisskb/build/powerpc-fixes_mpc86xx_defconfig_powerpc-gcc5 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux- mpc86xx_defconfig Using /kisskb/src/arch/powerpc/configs/mpc86xx_base.config as base Merging /kisskb/src/arch/powerpc/configs/86xx-hw.config Merging /kisskb/src/arch/powerpc/configs/fsl-emb-nonhw.config # # merged configuration written to .config (needs make) # # < make -s -j 40 ARCH=powerpc O=/kisskb/build/powerpc-fixes_mpc86xx_defconfig_powerpc-gcc5 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux- help # make -s -j 40 ARCH=powerpc O=/kisskb/build/powerpc-fixes_mpc86xx_defconfig_powerpc-gcc5 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux- olddefconfig # make -s -j 40 ARCH=powerpc O=/kisskb/build/powerpc-fixes_mpc86xx_defconfig_powerpc-gcc5 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux- /kisskb/src/arch/powerpc/boot/dts/fsl/mvme7100.dts:135.22-137.4: Warning (pci_bridge): /pcie@f1008000: missing ranges for PCI bridge (or not a bridge) also defined at /kisskb/src/arch/powerpc/boot/dts/fsl/mpc8641si-post.dtsi:92.7-117.3 /kisskb/src/arch/powerpc/boot/dts/fsl/mpc8641si-post.dtsi:102.9-116.4: Warning (pci_bridge): /pcie@f1008000/pcie@0: missing ranges for PCI bridge (or not a bridge) /kisskb/src/arch/powerpc/boot/dts/fsl/mvme7100.dts:139.22-141.4: Warning (pci_bridge): /pcie@f1009000: missing ranges for PCI bridge (or not a bridge) also defined at /kisskb/src/arch/powerpc/boot/dts/fsl/mpc8641si-post.dtsi:119.7-144.3 /kisskb/src/arch/powerpc/boot/dts/fsl/mpc8641si-post.dtsi:129.9-143.4: Warning (pci_bridge): /pcie@f1009000/pcie@0: missing ranges for PCI bridge (or not a bridge) arch/powerpc/boot/dts/fsl/mvme7100.dtb: Warning (unit_address_format): Failed prerequisite 'pci_bridge' arch/powerpc/boot/dts/fsl/mvme7100.dtb: Warning (pci_device_reg): Failed prerequisite 'pci_bridge' arch/powerpc/boot/dts/fsl/mvme7100.dtb: Warning (pci_device_bus_num): Failed prerequisite 'pci_bridge' /kisskb/src/arch/powerpc/boot/dts/fsl/mvme7100.dts:30.11-32.6: Warning (i2c_bus_reg): /soc@f1000000/i2c@3000/rtc@68: missing or empty reg property Completed OK # rm -rf /kisskb/build/powerpc-fixes_mpc86xx_defconfig_powerpc-gcc5 # Build took: 0:01:12.241241