kbuild: do not redirect the first prerequisite for filechk
authorMasahiro Yamada <yamada.masahiro@socionext.com>
Wed, 25 Jul 2018 05:16:11 +0000 (14:16 +0900)
committerMasahiro Yamada <yamada.masahiro@socionext.com>
Sat, 28 Jul 2018 01:34:10 +0000 (10:34 +0900)
Currently, filechk unconditionally opens the first prerequisite and
redirects it as the stdin of a filechk_* rule.  Hence, every target
using $(call filechk,...) must list something as the first prerequisite
even if it is unneeded.

'< $<' is actually unneeded in most cases.  Each rule can explicitly
adds it if necessary.

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Makefile
arch/s390/kernel/syscalls/Makefile
arch/um/Makefile
scripts/Kbuild.include
scripts/Makefile.lib

index 433f87015c1d01200d019733945df61e8ca71666..bc3a65be90356740bb72481dbee6538458ac528c 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -1117,7 +1117,7 @@ define filechk_version.h
        echo '#define KERNEL_VERSION(a,b,c) (((a) << 16) + ((b) << 8) + (c))';)
 endef
 
-$(version_h): $(srctree)/Makefile FORCE
+$(version_h): FORCE
        $(call filechk,version.h)
        $(Q)rm -f $(old_version_h)
 
index 8ff96c08955f4af3f323fb0e7a7ac863329a4364..4d929edc80a6296eecdd9911457d1fb691de6d6b 100644 (file)
@@ -25,15 +25,15 @@ _dummy := $(shell [ -d '$(uapi)' ] || mkdir -p '$(uapi)') \
          $(shell [ -d '$(kapi)' ] || mkdir -p '$(kapi)')
 
 define filechk_syshdr
-       $(CONFIG_SHELL) '$(systbl)' -H -a $(syshdr_abi_$(basetarget)) -f "$2"
+       $(CONFIG_SHELL) '$(systbl)' -H -a $(syshdr_abi_$(basetarget)) -f "$2" < $<
 endef
 
 define filechk_sysnr
-       $(CONFIG_SHELL) '$(systbl)' -N -a $(sysnr_abi_$(basetarget))
+       $(CONFIG_SHELL) '$(systbl)' -N -a $(sysnr_abi_$(basetarget)) < $<
 endef
 
 define filechk_syscalls
-       $(CONFIG_SHELL) '$(systbl)' -S
+       $(CONFIG_SHELL) '$(systbl)' -S < $<
 endef
 
 syshdr_abi_unistd_32 := common,32
index e54dda8a036388438bcac0302d6dcf0259ef8a96..8c35261f5706c2db2ec95b6e0793cdd6c343f11a 100644 (file)
@@ -169,7 +169,7 @@ define filechk_gen-asm-offsets
          echo " *"; \
          echo " */"; \
          echo ""; \
-         sed -ne "/^->/{s:^->\([^ ]*\) [\$$#]*\([^ ]*\) \(.*\):#define \1 \2 /* \3 */:; s:->::; p;}"; \
+         sed -ne "/^->/{s:^->\([^ ]*\) [\$$#]*\([^ ]*\) \(.*\):#define \1 \2 /* \3 */:; s:->::; p;}" < $<; \
          echo ""; )
 endef
 
index 02bb3163cd8d6eb95864c6b6b7ae619f5c25bd63..7eabe80cb9fb57559f22ddbfed4ba5bd4ed34555 100644 (file)
@@ -56,7 +56,7 @@ kecho := $($(quiet)kecho)
 define filechk
        $(Q)set -e;                             \
        mkdir -p $(dir $@);                     \
-       $(filechk_$(1)) < $< > $@.tmp;          \
+       $(filechk_$(1)) > $@.tmp;               \
        if [ -r $@ ] && cmp -s $@ $@.tmp; then  \
                rm -f $@.tmp;                   \
        else                                    \
index f56bb4b8bc60dc4b83e8238427bf8413390811a7..df0fff2526194d37641b4bf34a865c2a3a867635 100644 (file)
@@ -415,7 +415,7 @@ define filechk_offsets
         echo " * This file was generated by Kbuild"; \
         echo " */"; \
         echo ""; \
-        sed -ne $(sed-offsets); \
+        sed -ne $(sed-offsets) < $<; \
         echo ""; \
         echo "#endif" )
 endef