# git rev-parse -q --verify b0b9f351fd616a01eea7c8d44e798db5a122a005^{commit} b0b9f351fd616a01eea7c8d44e798db5a122a005 already have revision, skipping fetch # git checkout -q -f -B kisskb b0b9f351fd616a01eea7c8d44e798db5a122a005 # git clean -qxdf # < git log -1 # commit b0b9f351fd616a01eea7c8d44e798db5a122a005 # Author: Masahiro Yamada # Date: Fri Jul 5 19:01:44 2019 +0900 # # powerpc/boot: pass CONFIG options in a simpler and more robust way # # Commit 5e9dcb6188a4 ("powerpc/boot: Expose Kconfig symbols to wrapper") # was wrong, but commit e41b93a6be57 ("powerpc/boot: Fix build failures # with -j 1") was also wrong. # # The correct dependency is: # # $(obj)/serial.o: $(obj)/autoconf.h # # However, I do not see the reason why we need to copy autoconf.h to # arch/power/boot/. Nor do I see consistency in the way of passing # CONFIG options. # # decompress.c references CONFIG_KERNEL_GZIP and CONFIG_KERNEL_XZ, which # are passed via the command line. # # serial.c includes autoconf.h to reference a couple of CONFIG options, # but this is fragile because we often forget to include "autoconf.h" # from source files. # # In fact, it is already broken. # # ppc_asm.h references CONFIG_PPC_8xx, but utils.S is not given any way # to access CONFIG options. So, CONFIG_PPC_8xx is never defined here. # # Pass $(LINUXINCLUDE) to make sure CONFIG options are accessible from # all .c and .S files in arch/powerpc/boot/. # # I also removed the -traditional flag to make include/linux/kconfig.h # work. This flag makes the preprocessor imitate the behavior of the # pre-standard C compiler, but I do not understand why it is necessary. # # Signed-off-by: Masahiro Yamada # Signed-off-by: Michael Ellerman # Link: https://lore.kernel.org/r/20190705100144.28785-2-yamada.masahiro@socionext.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 b0b9f351fd616a01eea7c8d44e798db5a122a005 # < make -s -j 10 ARCH=powerpc O=/kisskb/build/powerpc-next_pseries_defconfig+FA_DUMP_powerpc-gcc5 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux- pseries_defconfig # Added to kconfig CONFIG_CRASH_DUMP=y # Added to kconfig CONFIG_FA_DUMP=y # yes \n | make -s -j 10 ARCH=powerpc O=/kisskb/build/powerpc-next_pseries_defconfig+FA_DUMP_powerpc-gcc5 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux- oldconfig yes: standard output: Broken pipe # make -s -j 10 ARCH=powerpc O=/kisskb/build/powerpc-next_pseries_defconfig+FA_DUMP_powerpc-gcc5 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux- /kisskb/src/kernel/futex.c: In function 'do_futex': /kisskb/src/kernel/futex.c:1658:17: warning: 'oldval' may be used uninitialized in this function [-Wmaybe-uninitialized] return oldval == cmparg; ^ /kisskb/src/kernel/futex.c:1633:6: note: 'oldval' was declared here int oldval, ret; ^ Completed OK # rm -rf /kisskb/build/powerpc-next_pseries_defconfig+FA_DUMP_powerpc-gcc5 # Build took: 0:05:19.130786