On Wed, Jan 18, 2023 at 8:51 AM Masahiro Yamada <masahiroy@xxxxxxxxxx> wrote: > > On Wed, Jan 18, 2023 at 4:45 PM Sedat Dilek <sedat.dilek@xxxxxxxxx> wrote: > > > > On Mon, Jan 16, 2023 at 4:21 AM Masahiro Yamada <masahiroy@xxxxxxxxxx> wrote: > > > > > > On Mon, Jan 16, 2023 at 11:02 AM Sedat Dilek <sedat.dilek@xxxxxxxxx> wrote: > > > > > > > > Hi Masahiro, > > > > > > > > I saw these warnings in my build-log: > > > > > > > > if ! command -v sha1sum >/dev/null; then echo "warning: cannot check > > > > the header due to sha1sum missing"; exit 0; fi; if [ "$(sed -n '$s:// > > > > ::p' include/linux/atomic/ > > > > atomic-arch-fallback.h)" != "$(sed '$d' > > > > include/linux/atomic/atomic-arch-fallback.h | sha1sum | sed 's/ > > > > .*//')" ]; then echo "error: include/linux/atomic/atomic-arch-f > > > > allback.h has been modified." >&2; exit 1; fi; touch > > > > .checked-atomic-arch-fallback.h > > > > if ! command -v sha1sum >/dev/null; then echo "warning: cannot check > > > > the header due to sha1sum missing"; exit 0; fi; if [ "$(sed -n '$s:// > > > > ::p' include/linux/atomic/ > > > > atomic-instrumented.h)" != "$(sed '$d' > > > > include/linux/atomic/atomic-instrumented.h | sha1sum | sed 's/ .*//')" > > > > ]; then echo "error: include/linux/atomic/atomic-instrume > > > > nted.h has been modified." >&2; exit 1; fi; touch .checked-atomic-instrumented.h > > > > if ! command -v sha1sum >/dev/null; then echo "warning: cannot check > > > > the header due to sha1sum missing"; exit 0; fi; if [ "$(sed -n '$s:// > > > > ::p' include/linux/atomic/ > > > > atomic-long.h)" != "$(sed '$d' include/linux/atomic/atomic-long.h | > > > > sha1sum | sed 's/ .*//')" ]; then echo "error: > > > > include/linux/atomic/atomic-long.h has been modified > > > > ." >&2; exit 1; fi; touch .checked-atomic-long.h > > > > > > > > NOTE: I did a `make distclean` before I started my build. > > > > > > > > Can you please comment on this? > > > > > > > > > > > > > > > Please clarify your problem. > > > > > > My best guess is, you just added V=1 option to > > > print the full log, didn't you? > > > > > > > > > > > > I can see the same build log by running the following command. > > > > > > > Hi Masahiro, > > > > you had a chance to look into this? > > > I already replied. > This is the output of V=1. > There is no issue. > Hi Masahiro, No, I disagree. You demonstrated by: $ make V=1 mrproper defconfig prepare | grep sha1sum That you see these warnings, too. There is no statement that these warnings are OK. If so, why? Are those headers generated in build-time thus getting no/wrong sha1sums? Please, clarify. If you have time: kbuild-next with a rework/refactoring of make V=xx does show these warnings? Sorry for being pedantic - I started with Linux v6.2-rc4, again doing some kernel-testing. My goal is to have a warning-free build with LLVM-15 (ThinLTO + KCFI). Right now, I will do some more testing. Thanks. BR, -Sedat- > > > > > Thanks. > > > > Regards, > > -Sedat- > > > > > > > > $ make V=1 mrproper defconfig prepare | grep sha1sum > > > + [ clean = clean ] > > > + cleanup > > > + rm -f .btf.* > > > + rm -f System.map > > > + rm -f vmlinux > > > + rm -f vmlinux.map > > > + exit 0 > > > if ! command -v sha1sum >/dev/null; then echo "warning: cannot check > > > the header due to sha1sum missing"; exit 0; fi; if [ "$(sed -n '$s:// > > > ::p' include/linux/atomic/atomic-arch-fallback.h)" != "$(sed '$d' > > > include/linux/atomic/atomic-arch-fallback.h | sha1sum | sed 's/ > > > .*//')" ]; then echo "error: > > > include/linux/atomic/atomic-arch-fallback.h has been modified." >&2; > > > exit 1; fi; touch .checked-atomic-arch-fallback.h > > > if ! command -v sha1sum >/dev/null; then echo "warning: cannot check > > > the header due to sha1sum missing"; exit 0; fi; if [ "$(sed -n '$s:// > > > ::p' include/linux/atomic/atomic-instrumented.h)" != "$(sed '$d' > > > include/linux/atomic/atomic-instrumented.h | sha1sum | sed 's/ .*//')" > > > ]; then echo "error: include/linux/atomic/atomic-instrumented.h has > > > been modified." >&2; exit 1; fi; touch .checked-atomic-instrumented.h > > > if ! command -v sha1sum >/dev/null; then echo "warning: cannot check > > > the header due to sha1sum missing"; exit 0; fi; if [ "$(sed -n '$s:// > > > ::p' include/linux/atomic/atomic-long.h)" != "$(sed '$d' > > > include/linux/atomic/atomic-long.h | sha1sum | sed 's/ .*//')" ]; then > > > echo "error: include/linux/atomic/atomic-long.h has been modified." > > > >&2; exit 1; fi; touch .checked-atomic-long.h > > > > > > > > > > > > > > > > > > > > > > > > > > Thanks. > > > > > > > > Best regards, > > > > -Sedat- > > > > > > > > > > > > > > > -- > > > Best Regards > > > Masahiro Yamada > > > > -- > Best Regards > Masahiro Yamada
