# git rev-parse -q --verify 6a86fe746e2f69e49328b9ad8ad0f840c467ee59^{commit} 6a86fe746e2f69e49328b9ad8ad0f840c467ee59 already have revision, skipping fetch # git checkout -q -f -B kisskb 6a86fe746e2f69e49328b9ad8ad0f840c467ee59 # git clean -qxdf # < git log -1 # commit 6a86fe746e2f69e49328b9ad8ad0f840c467ee59 # Author: Michael Ellerman # Date: Thu Feb 18 23:30:58 2021 +1100 # # powerpc/4xx: Fix build errors from mfdcr() # # lkp reported a build error in fsp2.o: # # CC arch/powerpc/platforms/44x/fsp2.o # {standard input}:577: Error: unsupported relocation against base # # Which comes from: # # pr_err("GESR0: 0x%08x\n", mfdcr(base + PLB4OPB_GESR0)); # # Where our mfdcr() macro is stringifying "base + PLB4OPB_GESR0", and # passing that to the assembler, which obviously doesn't work. # # The mfdcr() macro already checks that the argument is constant using # __builtin_constant_p(), and if not calls the out-of-line version of # mfdcr(). But in this case GCC is smart enough to notice that "base + # PLB4OPB_GESR0" will be constant, even though it's not something we can # immediately stringify into a register number. # # Segher pointed out that passing the register number to the inline asm # as a constant would be better, and in fact it fixes the build error, # presumably because it gives GCC a chance to resolve the value. # # While we're at it, change mtdcr() similarly. # # Reported-by: kernel test robot # Suggested-by: Segher Boessenkool # Signed-off-by: Michael Ellerman # Acked-by: Feng Tang # Link: https://lore.kernel.org/r/20210218123058.748882-1-mpe@ellerman.id.au # < /opt/cross/kisskb/korg/gcc-4.9.4-nolibc/powerpc64-linux/bin/powerpc64-linux-gcc --version # < /opt/cross/kisskb/korg/gcc-4.9.4-nolibc/powerpc64-linux/bin/powerpc64-linux-ld --version # < git log --format=%s --max-count=1 6a86fe746e2f69e49328b9ad8ad0f840c467ee59 # < make -s -j 32 ARCH=powerpc O=/kisskb/build/powerpc-fixes_40x_makalu_defconfig_powerpc-gcc4.9 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-4.9.4-nolibc/powerpc64-linux/bin/powerpc64-linux- 40x/makalu_defconfig # < make -s -j 32 ARCH=powerpc O=/kisskb/build/powerpc-fixes_40x_makalu_defconfig_powerpc-gcc4.9 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-4.9.4-nolibc/powerpc64-linux/bin/powerpc64-linux- help # make -s -j 32 ARCH=powerpc O=/kisskb/build/powerpc-fixes_40x_makalu_defconfig_powerpc-gcc4.9 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-4.9.4-nolibc/powerpc64-linux/bin/powerpc64-linux- olddefconfig # make -s -j 32 ARCH=powerpc O=/kisskb/build/powerpc-fixes_40x_makalu_defconfig_powerpc-gcc4.9 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-4.9.4-nolibc/powerpc64-linux/bin/powerpc64-linux- /kisskb/src/block/genhd.c: In function 'diskstats_show': /kisskb/src/block/genhd.c:1314:1: warning: the frame size of 1160 bytes is larger than 1024 bytes [-Wframe-larger-than=] } ^ Completed OK # rm -rf /kisskb/build/powerpc-fixes_40x_makalu_defconfig_powerpc-gcc4.9 # Build took: 0:00:49.662602