Buildresult: powerpc-next/sparc64-defconfig/sparc64-gcc5 built on Aug 3 2018, 16:33

kisskb

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

Status:	OK
Date/Time:	Aug 3 2018, 16:33
Duration:	0:01:25.723235
Builder:	ka4
Revision:	powerpc/powernv: Fix concurrency issue with npu->mmio_atsd_usage (`a6fcdbf95d7907bc5d5a669adb86e6bda72d738b)`
Target:	powerpc-next/sparc64-defconfig/sparc64-gcc5
Branch:	powerpc-next
Compiler:	sparc64-gcc5 (sparc64-linux-gcc.br_real (Buildroot 2016.11-git-00613-ge98b4dd) 5.4.0)
Config:	defconfig (download)
Log:	Download original Possible warnings (4) <stdin>:1332:2: warning: #warning syscall io_pgetevents not implemented [-Wcpp] <stdin>:1335:2: warning: #warning syscall rseq not implemented [-Wcpp] WARNING: EXPORT symbol "_mcount" [vmlinux] version generation failed, symbol will not be versioned. WARNING: EXPORT symbol "_mcount" [vmlinux] version generation failed, symbol will not be versioned. Full Log # git rev-parse -q --verify a6fcdbf95d7907bc5d5a669adb86e6bda72d738b^{commit} a6fcdbf95d7907bc5d5a669adb86e6bda72d738b already have revision, skipping fetch # git checkout -q -f -B kisskb a6fcdbf95d7907bc5d5a669adb86e6bda72d738b # git clean -qxdf # < git log -1 # commit a6fcdbf95d7907bc5d5a669adb86e6bda72d738b # Author: Reza Arbab <arbab@linux.ibm.com> # Date: Thu Aug 2 23:03:36 2018 -0500 # # powerpc/powernv: Fix concurrency issue with npu->mmio_atsd_usage # # We've encountered a performance issue when multiple processors stress # {get,put}_mmio_atsd_reg(). These functions contend for # mmio_atsd_usage, an unsigned long used as a bitmask. # # The accesses to mmio_atsd_usage are done using test_and_set_bit_lock() # and clear_bit_unlock(). As implemented, both of these will require # a (successful) stwcx to that same cache line. # # What we end up with is thread A, attempting to unlock, being slowed by # other threads repeatedly attempting to lock. A's stwcx instructions # fail and retry because the memory reservation is lost every time a # different thread beats it to the punch. # # There may be a long-term way to fix this at a larger scale, but for # now resolve the immediate problem by gating our call to # test_and_set_bit_lock() with one to test_bit(), which is obviously # implemented without using a store. # # Fixes: 1ab66d1fbada ("powerpc/powernv: Introduce address translation services for Nvlink2") # Signed-off-by: Reza Arbab <arbab@linux.ibm.com> # Acked-by: Alistair Popple <alistair@popple.id.au> # Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> # < /opt/cross/kisskb/br-sparc64-full-2016.08-613-ge98b4dd/bin/sparc64-linux-gcc --version # < git log --format=%s --max-count=1 a6fcdbf95d7907bc5d5a669adb86e6bda72d738b # < make -s -j 10 ARCH=sparc64 O=/kisskb/build/powerpc-next_sparc64-defconfig_sparc64 CROSS_COMPILE=/opt/cross/kisskb/br-sparc64-full-2016.08-613-ge98b4dd/bin/sparc64-linux- defconfig # make -s -j 10 ARCH=sparc64 O=/kisskb/build/powerpc-next_sparc64-defconfig_sparc64 CROSS_COMPILE=/opt/cross/kisskb/br-sparc64-full-2016.08-613-ge98b4dd/bin/sparc64-linux- <stdin>:1332:2: warning: #warning syscall io_pgetevents not implemented [-Wcpp] <stdin>:1335:2: warning: #warning syscall rseq not implemented [-Wcpp] WARNING: EXPORT symbol "_mcount" [vmlinux] version generation failed, symbol will not be versioned. kernel: arch/sparc/boot/image is ready WARNING: EXPORT symbol "_mcount" [vmlinux] version generation failed, symbol will not be versioned. kernel: arch/sparc/boot/zImage is ready Completed OK # rm -rf /kisskb/build/powerpc-next_sparc64-defconfig_sparc64 # Build took: 0:01:25.723235

© Michael Ellerman 2006-2018.