# git rev-parse -q --verify 308f765116e9176bf508c10b72ca80ad594e3119^{commit} 308f765116e9176bf508c10b72ca80ad594e3119 already have revision, skipping fetch # git checkout -q -f -B kisskb 308f765116e9176bf508c10b72ca80ad594e3119 # git clean -qxdf # < git log -1 # commit 308f765116e9176bf508c10b72ca80ad594e3119 # Author: Gustavo A. R. Silva # Date: Mon Aug 24 15:20:04 2020 -0500 # # lib: Revert use of fallthrough pseudo-keyword # # The following build error was reported by Nathan Chancellor: # # "$ scripts/config --file arch/powerpc/configs/powernv_defconfig -e KERNEL_XZ # # $ make -skj"$(nproc)" ARCH=powerpc CROSS_COMPILE=powerpc64le-linux- distclean powernv_defconfig zImage # ... # In file included from arch/powerpc/boot/../../../lib/decompress_unxz.c:234, # from arch/powerpc/boot/decompress.c:38: # arch/powerpc/boot/../../../lib/xz/xz_dec_stream.c: In function 'dec_main': # arch/powerpc/boot/../../../lib/xz/xz_dec_stream.c:586:4: error: 'fallthrough' undeclared (first use in this function) # 586 | fallthrough; # | ^~~~~~~~~~~ # arch/powerpc/boot/../../../lib/xz/xz_dec_stream.c:586:4: note: each undeclared identifier is reported only once for each function it appears in # In file included from arch/powerpc/boot/../../../lib/decompress_unxz.c:235, # from arch/powerpc/boot/decompress.c:38: # arch/powerpc/boot/../../../lib/xz/xz_dec_lzma2.c: In function 'xz_dec_lzma2_run': # arch/powerpc/boot/../../../lib/xz/xz_dec_lzma2.c:1046:4: error: 'fallthrough' undeclared (first use in this function) # 1046 | fallthrough; # | ^~~~~~~~~~~ # make[2]: *** [arch/powerpc/boot/Makefile:215: arch/powerpc/boot/decompress.o] Error 1 # make[2]: Target 'arch/powerpc/boot/zImage' not remade because of errors. # make[1]: *** [arch/powerpc/Makefile:295: zImage] Error 2 # make: *** [Makefile:335: __build_one_by_one] Error 2 # make: Target 'distclean' not remade because of errors. # make: Target 'powernv_defconfig' not remade because of errors. # make: Target 'zImage' not remade because of errors. # # This will end up affecting distribution configurations such as Debian # and OpenSUSE according to my testing. I am not sure what the solution # is, the PowerPC wrapper does not set -D__KERNEL__ so I am not sure that # compiler_attributes.h can be safely included." # # In order to avoid these sort of problems, it seems that the best # solution is to use /* fall through */ comments instead of the # fallthrough pseudo-keyword macro in lib/, for now. # # Reported-by: Nathan Chancellor # Fixes: df561f6688fe ("treewide: Use fallthrough pseudo-keyword") # Signed-off-by: Gustavo A. R. Silva # < /opt/cross/kisskb/korg/gcc-8.1.0-nolibc/hppa-linux/bin/hppa-linux-gcc --version # < /opt/cross/kisskb/korg/gcc-8.1.0-nolibc/hppa-linux/bin/hppa-linux-ld --version # < git log --format=%s --max-count=1 308f765116e9176bf508c10b72ca80ad594e3119 # < make -s -j 8 ARCH=parisc O=/kisskb/build/gustavoars_generic-32bit_defconfig_parisc-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.1.0-nolibc/hppa-linux/bin/hppa-linux- generic-32bit_defconfig # < make -s -j 8 ARCH=parisc O=/kisskb/build/gustavoars_generic-32bit_defconfig_parisc-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.1.0-nolibc/hppa-linux/bin/hppa-linux- help # make -s -j 8 ARCH=parisc O=/kisskb/build/gustavoars_generic-32bit_defconfig_parisc-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.1.0-nolibc/hppa-linux/bin/hppa-linux- olddefconfig # make -s -j 8 ARCH=parisc O=/kisskb/build/gustavoars_generic-32bit_defconfig_parisc-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.1.0-nolibc/hppa-linux/bin/hppa-linux- /kisskb/src/block/genhd.c: In function 'diskstats_show': /kisskb/src/block/genhd.c:1667:1: warning: the frame size of 1720 bytes is larger than 1280 bytes [-Wframe-larger-than=] } ^ /kisskb/src/lib/xxhash.c: In function 'xxh64': /kisskb/src/lib/xxhash.c:236:1: warning: the frame size of 1616 bytes is larger than 1280 bytes [-Wframe-larger-than=] } ^ Completed OK # rm -rf /kisskb/build/gustavoars_generic-32bit_defconfig_parisc-gcc8 # Build took: 0:02:08.871290