# git rev-parse -q --verify 8b7f59de92ac65aa21c7d779274dbfa577ae2d2c^{commit}
8b7f59de92ac65aa21c7d779274dbfa577ae2d2c
already have revision, skipping fetch
# git checkout -q -f -B kisskb 8b7f59de92ac65aa21c7d779274dbfa577ae2d2c
# git clean -qxdf
# < git log -1
# commit 8b7f59de92ac65aa21c7d779274dbfa577ae2d2c
# Author: Michael Ellerman <mpe@ellerman.id.au>
# Date:   Sat Jul 6 22:08:33 2024 +1000
# 
#     selftests/powerpc: Fix build with USERCFLAGS set
#     
#     Currently building the powerpc selftests with USERCFLAGS set to anything
#     causes the build to break:
#     
#       $ make -C tools/testing/selftests/powerpc V=1 USERCFLAGS=-Wno-error
#       ...
#       gcc -Wno-error    cache_shape.c ...
#       cache_shape.c:18:10: fatal error: utils.h: No such file or directory
#          18 | #include "utils.h"
#             |          ^~~~~~~~~
#       compilation terminated.
#     
#     This happens because the USERCFLAGS are added to CFLAGS in lib.mk, which
#     causes the check of CFLAGS in powerpc/flags.mk to skip setting CFLAGS at
#     all, resulting in none of the usual CFLAGS being passed. That can
#     be seen in the output above, the only flag passed to the compiler is
#     -Wno-error.
#     
#     Fix it by dropping the conditional setting of CFLAGS in flags.mk.
#     Instead always set CFLAGS, but also append USERCFLAGS if they are set.
#     
#     Note that appending to CFLAGS (with +=) wouldn't work, because flags.mk
#     is included by multiple Makefiles (to support partial builds), causing
#     CFLAGS to be appended to multiple times. Additionally that would place
#     the USERCFLAGS prior to the standard CFLAGS, meaning the USERCFLAGS
#     couldn't override the standard flags. Being able to override the
#     standard flags is desirable, for example for adding -Wno-error.
#     
#     With the fix in place, the CFLAGS are set correctly, including the
#     USERCFLAGS:
#     
#       $ make -C tools/testing/selftests/powerpc V=1 USERCFLAGS=-Wno-error
#       ...
#       gcc -std=gnu99 -O2 -Wall -Werror -DGIT_VERSION='"v6.10-rc2-7-gdea17e7e56c3"'
#       -I/home/michael/linux/tools/testing/selftests/powerpc/include -Wno-error
#       cache_shape.c ...
#     
#     Fixes: 5553a79387e9 ("selftests/powerpc: Add flags.mk to support pmu buildable")
#     Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
#     Link: https://msgid.link/20240706120833.909853-1-mpe@ellerman.id.au
# < /opt/cross/kisskb/korg/gcc-13.1.0-nolibc/sparc64-linux/bin/sparc64-linux-gcc --version
# < /opt/cross/kisskb/korg/gcc-13.1.0-nolibc/sparc64-linux/bin/sparc64-linux-ld --version
# < git log --format=%s --max-count=1 8b7f59de92ac65aa21c7d779274dbfa577ae2d2c
# make -s -j 32 ARCH=sparc64 O=/kisskb/build/powerpc-fixes_defconfig_sparc64-gcc13 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-13.1.0-nolibc/sparc64-linux/bin/sparc64-linux-  defconfig
# < make -s -j 32 ARCH=sparc64 O=/kisskb/build/powerpc-fixes_defconfig_sparc64-gcc13 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-13.1.0-nolibc/sparc64-linux/bin/sparc64-linux-  help
# make -s -j 32 ARCH=sparc64 O=/kisskb/build/powerpc-fixes_defconfig_sparc64-gcc13 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-13.1.0-nolibc/sparc64-linux/bin/sparc64-linux-  olddefconfig
# make -s -j 32 ARCH=sparc64 O=/kisskb/build/powerpc-fixes_defconfig_sparc64-gcc13 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-13.1.0-nolibc/sparc64-linux/bin/sparc64-linux-  
<stdin>:1519:2: warning: #warning syscall clone3 not implemented [-Wcpp]
/kisskb/src/arch/sparc/vdso/vclock_gettime.c:254:1: warning: no previous prototype for '__vdso_clock_gettime' [-Wmissing-prototypes]
  254 | __vdso_clock_gettime(clockid_t clock, struct __kernel_old_timespec *ts)
      | ^~~~~~~~~~~~~~~~~~~~
/kisskb/src/arch/sparc/vdso/vclock_gettime.c:282:1: warning: no previous prototype for '__vdso_clock_gettime_stick' [-Wmissing-prototypes]
  282 | __vdso_clock_gettime_stick(clockid_t clock, struct __kernel_old_timespec *ts)
      | ^~~~~~~~~~~~~~~~~~~~~~~~~~
/kisskb/src/arch/sparc/vdso/vclock_gettime.c:307:1: warning: no previous prototype for '__vdso_gettimeofday' [-Wmissing-prototypes]
  307 | __vdso_gettimeofday(struct __kernel_old_timeval *tv, struct timezone *tz)
      | ^~~~~~~~~~~~~~~~~~~
/kisskb/src/arch/sparc/vdso/vclock_gettime.c:343:1: warning: no previous prototype for '__vdso_gettimeofday_stick' [-Wmissing-prototypes]
  343 | __vdso_gettimeofday_stick(struct __kernel_old_timeval *tv, struct timezone *tz)
      | ^~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /kisskb/src/arch/sparc/vdso/vdso32/vclock_gettime.c:22:
/kisskb/src/arch/sparc/vdso/vdso32/../vclock_gettime.c:254:1: warning: no previous prototype for '__vdso_clock_gettime' [-Wmissing-prototypes]
  254 | __vdso_clock_gettime(clockid_t clock, struct __kernel_old_timespec *ts)
      | ^~~~~~~~~~~~~~~~~~~~
/kisskb/src/arch/sparc/vdso/vdso32/../vclock_gettime.c:282:1: warning: no previous prototype for '__vdso_clock_gettime_stick' [-Wmissing-prototypes]
  282 | __vdso_clock_gettime_stick(clockid_t clock, struct __kernel_old_timespec *ts)
      | ^~~~~~~~~~~~~~~~~~~~~~~~~~
/kisskb/src/arch/sparc/vdso/vdso32/../vclock_gettime.c:307:1: warning: no previous prototype for '__vdso_gettimeofday' [-Wmissing-prototypes]
  307 | __vdso_gettimeofday(struct __kernel_old_timeval *tv, struct timezone *tz)
      | ^~~~~~~~~~~~~~~~~~~
/kisskb/src/arch/sparc/vdso/vdso32/../vclock_gettime.c:343:1: warning: no previous prototype for '__vdso_gettimeofday_stick' [-Wmissing-prototypes]
  343 | __vdso_gettimeofday_stick(struct __kernel_old_timeval *tv, struct timezone *tz)
      | ^~~~~~~~~~~~~~~~~~~~~~~~~
/opt/cross/kisskb/korg/gcc-13.1.0-nolibc/sparc64-linux/bin/sparc64-linux-ld: warning: arch/sparc/vdso/vdso-note.o: missing .note.GNU-stack section implies executable stack
/opt/cross/kisskb/korg/gcc-13.1.0-nolibc/sparc64-linux/bin/sparc64-linux-ld: NOTE: This behaviour is deprecated and will be removed in a future version of the linker
/opt/cross/kisskb/korg/gcc-13.1.0-nolibc/sparc64-linux/bin/sparc64-linux-ld: warning: arch/sparc/vdso/vdso32/vdso-note.o: missing .note.GNU-stack section implies executable stack
/opt/cross/kisskb/korg/gcc-13.1.0-nolibc/sparc64-linux/bin/sparc64-linux-ld: NOTE: This behaviour is deprecated and will be removed in a future version of the linker
/kisskb/src/arch/sparc/prom/misc_64.c:165:5: warning: no previous prototype for 'prom_get_mmu_ihandle' [-Wmissing-prototypes]
  165 | int prom_get_mmu_ihandle(void)
      |     ^~~~~~~~~~~~~~~~~~~~
/kisskb/src/arch/sparc/prom/p1275.c:52:6: warning: no previous prototype for 'prom_cif_init' [-Wmissing-prototypes]
   52 | void prom_cif_init(void *cif_handler, void *cif_stack)
      |      ^~~~~~~~~~~~~
WARNING: modpost: EXPORT symbol "_mcount" [vmlinux] version generation failed, symbol will not be versioned.
Is "_mcount" prototyped in <asm/asm-prototypes.h>?
Completed OK
# rm -rf /kisskb/build/powerpc-fixes_defconfig_sparc64-gcc13
# Build took: 0:01:27.977758