mbox series

[v2,0/9] kbuild: clean-up modversion, TRIM_UNUSED_KSYMS, if_changed_rule, etc.

Message ID 1542675930-21114-1-git-send-email-yamada.masahiro@socionext.com
Headers show
Series kbuild: clean-up modversion, TRIM_UNUSED_KSYMS, if_changed_rule, etc. | expand

Message

Masahiro Yamada Nov. 20, 2018, 1:05 a.m. UTC
As a Kbuild maintainer, I always struggle to keep the core makefiles
clean because people tend to squeeze more and more clutter code into
the kbuild core in order to do what they want to do.

The biggest step forward in this series is to re-implement
the build trick of CONFIG_TRIM_UNUSED_KSYMS in a cleaner way.
scripts/Kbuild.include now looks nice again.
Also, in my rough estimation, building with CONFIG_TRIM_UNUSED_KSYMS
became 40-50 % faster.

Besides those, nice cleanups are here and there.

Masahiro Yamada (9):
  kbuild: let fixdep directly write to .*.cmd files
  kbuild: remove redundant 'set -e' from filechk_* defines
  kbuild: remove redundant 'set -e' from sub_cmd_record_mcount
  kbuild: refactor modversions build rules
  kbuild: simplify dependency generation for CONFIG_TRIM_UNUSED_KSYMS
  kbuild: change if_changed_rule for multi-line recipe
  kbuild: remove trailing semicolon from cmd_* passed to if_changed_rule
  kbuild: refactor if_changed and if_changed_dep
  kbuild: remove redundant 'set -e' from cmd_* defines

 Makefile                     |  13 +++---
 arch/um/Makefile             |   2 +-
 include/asm-generic/export.h |  13 +++---
 include/linux/export.h       |  18 ++++----
 scripts/Kbuild.include       |  49 +++-----------------
 scripts/Makefile.build       | 105 ++++++++++++++++++-------------------------
 scripts/Makefile.lib         |   2 +-
 scripts/basic/fixdep.c       |  31 ++-----------
 scripts/gen_ksymdeps.sh      |  25 +++++++++++
 scripts/package/Makefile     |   1 -
 10 files changed, 106 insertions(+), 153 deletions(-)
 create mode 100755 scripts/gen_ksymdeps.sh

Comments

Masahiro Yamada Nov. 21, 2018, 2:39 p.m. UTC | #1
On Tue, Nov 20, 2018 at 10:11 AM Masahiro Yamada
<yamada.masahiro@socionext.com> wrote:
>
> As a Kbuild maintainer, I always struggle to keep the core makefiles
> clean because people tend to squeeze more and more clutter code into
> the kbuild core in order to do what they want to do.
>
> The biggest step forward in this series is to re-implement
> the build trick of CONFIG_TRIM_UNUSED_KSYMS in a cleaner way.
> scripts/Kbuild.include now looks nice again.
> Also, in my rough estimation, building with CONFIG_TRIM_UNUSED_KSYMS
> became 40-50 % faster.
>
> Besides those, nice cleanups are here and there.
>
> Masahiro Yamada (9):
>   kbuild: let fixdep directly write to .*.cmd files
>   kbuild: remove redundant 'set -e' from filechk_* defines
>   kbuild: remove redundant 'set -e' from sub_cmd_record_mcount
>   kbuild: refactor modversions build rules
>   kbuild: simplify dependency generation for CONFIG_TRIM_UNUSED_KSYMS
>   kbuild: change if_changed_rule for multi-line recipe
>   kbuild: remove trailing semicolon from cmd_* passed to if_changed_rule
>   kbuild: refactor if_changed and if_changed_dep
>   kbuild: remove redundant 'set -e' from cmd_* defines

Series, applied to linux-kbuild.





>  Makefile                     |  13 +++---
>  arch/um/Makefile             |   2 +-
>  include/asm-generic/export.h |  13 +++---
>  include/linux/export.h       |  18 ++++----
>  scripts/Kbuild.include       |  49 +++-----------------
>  scripts/Makefile.build       | 105 ++++++++++++++++++-------------------------
>  scripts/Makefile.lib         |   2 +-
>  scripts/basic/fixdep.c       |  31 ++-----------
>  scripts/gen_ksymdeps.sh      |  25 +++++++++++
>  scripts/package/Makefile     |   1 -
>  10 files changed, 106 insertions(+), 153 deletions(-)
>  create mode 100755 scripts/gen_ksymdeps.sh
>
> --
> 2.7.4
>