diff options
author | Masahiro Yamada <yamada.masahiro@socionext.com> | 2017-11-13 13:29:37 +0300 |
---|---|---|
committer | Masahiro Yamada <yamada.masahiro@socionext.com> | 2017-11-16 03:07:35 +0300 |
commit | 8a78756eb545a6fb8007fa154a626ca2bc208027 (patch) | |
tree | a5f6f791cf742128df2d9af0cfba1393cae0b0c4 /fs/xfs/mrlock.h | |
parent | 591f66899784ae0afa13ff9a3eb5ce0a4358e48b (diff) | |
download | linux-8a78756eb545a6fb8007fa154a626ca2bc208027.tar.xz |
kbuild: create object directories simpler and faster
For the out-of-tree build, scripts/Makefile.build creates output
directories, but this operation is not efficient.
scripts/Makefile.lib calculates obj-dirs as follows:
obj-dirs := $(dir $(multi-objs) $(obj-y))
Please notice $(sort ...) is not used here. Usually the result is
as many "./" as objects here.
For a lot of duplicated paths, the following command is invoked.
_dummy := $(foreach d,$(obj-dirs), $(shell [ -d $(d) ] || mkdir -p $(d)))
Then, the costly shell command is run over and over again.
I see many points for optimization:
[1] Use $(sort ...) to cut down duplicated paths before passing them
to system call
[2] Use single $(shell ...) instead of repeating it with $(foreach ...)
This will reduce forking.
[3] We can calculate obj-dirs more simply. Most of objects are already
accumulated in $(targets). So, $(dir $(targets)) is fine and more
comprehensive.
I also removed ugly code in arch/x86/entry/vdso/Makefile. This is now
really unnecessary.
Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Acked-by: Ingo Molnar <mingo@kernel.org>
Tested-by: Douglas Anderson <dianders@chromium.org>
Diffstat (limited to 'fs/xfs/mrlock.h')
0 files changed, 0 insertions, 0 deletions