From: Nick Desaulniers <ndesaulniers@google.com>
To: Masahiro Yamada <masahiroy@kernel.org>
Cc: linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-arch@vger.kernel.org, Nathan Chancellor <nathan@kernel.org>
Subject: Re: [PATCH v3 0/7] kbuild: various cleanups
Date: Mon, 26 Sep 2022 14:39:39 -0700 [thread overview]
Message-ID: <CAKwvOdkREP8kFuE2UqMV4a+CRykKMO44X-12N6zcQoBbRYdJ4g@mail.gmail.com> (raw)
In-Reply-To: <20220924181915.3251186-1-masahiroy@kernel.org>
On Sat, Sep 24, 2022 at 11:20 AM Masahiro Yamada <masahiroy@kernel.org> wrote:
>
> - Link vmlinux and modules in parallel
I was curious if this resulted in an observable improvement. N=1
Mainline:
$ /usr/bin/time -v make ARCH=arm64 LLVM=1 -j128 -s
Elapsed (wall clock) time (h:mm:ss or m:ss): 2:15.68
With this series:
$ /usr/bin/time -v make ARCH=arm64 LLVM=1 -j128 -s
Elapsed (wall clock) time (h:mm:ss or m:ss): 2:12.22
I'll take it!
Tested-by: Nick Desaulniers <ndesaulniers@google.com>
> - Remove head-y syntax
>
>
> Masahiro Yamada (7):
> kbuild: hard-code KBUILD_ALLDIRS in scripts/Makefile.package
> kbuild: list sub-directories in ./Kbuild
> kbuild: move .vmlinux.objs rule to Makefile.modpost
> kbuild: move vmlinux.o rule to the top Makefile
> kbuild: unify two modpost invocations
> kbuild: use obj-y instead extra-y for objects placed at the head
> kbuild: remove head-y syntax
>
> Documentation/kbuild/makefiles.rst | 27 +----
> Kbuild | 24 ++++
> Makefile | 119 +++++++++++---------
> arch/alpha/Makefile | 2 -
> arch/alpha/kernel/Makefile | 4 +-
> arch/arc/Makefile | 2 -
> arch/arc/kernel/Makefile | 4 +-
> arch/arm/Makefile | 3 -
> arch/arm/kernel/Makefile | 4 +-
> arch/arm64/Makefile | 3 -
> arch/arm64/kernel/Makefile | 4 +-
> arch/csky/Makefile | 2 -
> arch/csky/kernel/Makefile | 4 +-
> arch/hexagon/Makefile | 2 -
> arch/hexagon/kernel/Makefile | 3 +-
> arch/ia64/Makefile | 1 -
> arch/ia64/kernel/Makefile | 4 +-
> arch/loongarch/Makefile | 2 -
> arch/loongarch/kernel/Makefile | 4 +-
> arch/m68k/68000/Makefile | 2 +-
> arch/m68k/Makefile | 9 --
> arch/m68k/coldfire/Makefile | 2 +-
> arch/m68k/kernel/Makefile | 23 ++--
> arch/microblaze/Makefile | 1 -
> arch/microblaze/kernel/Makefile | 4 +-
> arch/mips/Makefile | 2 -
> arch/mips/kernel/Makefile | 4 +-
> arch/nios2/Makefile | 1 -
> arch/nios2/kernel/Makefile | 2 +-
> arch/openrisc/Makefile | 2 -
> arch/openrisc/kernel/Makefile | 4 +-
> arch/parisc/Makefile | 2 -
> arch/parisc/kernel/Makefile | 4 +-
> arch/powerpc/Makefile | 12 --
> arch/powerpc/kernel/Makefile | 20 ++--
> arch/riscv/Makefile | 2 -
> arch/riscv/kernel/Makefile | 2 +-
> arch/s390/Makefile | 2 -
> arch/s390/kernel/Makefile | 4 +-
> arch/sh/Makefile | 2 -
> arch/sh/kernel/Makefile | 4 +-
> arch/sparc/Makefile | 2 -
> arch/sparc/kernel/Makefile | 3 +-
> arch/x86/Makefile | 5 -
> arch/x86/kernel/Makefile | 10 +-
> arch/xtensa/Makefile | 2 -
> arch/xtensa/kernel/Makefile | 4 +-
> scripts/Makefile.lib | 2 +
> scripts/Makefile.modfinal | 2 +-
> scripts/Makefile.modpost | 112 ++++++++----------
> scripts/Makefile.package | 5 +-
> scripts/Makefile.vmlinux_o | 6 +-
> scripts/clang-tools/gen_compile_commands.py | 19 +---
> scripts/head-object-list.txt | 53 +++++++++
> scripts/link-vmlinux.sh | 34 +-----
> 55 files changed, 273 insertions(+), 314 deletions(-)
> create mode 100644 scripts/head-object-list.txt
>
> --
> 2.34.1
>
--
Thanks,
~Nick Desaulniers
prev parent reply other threads:[~2022-09-26 21:40 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-09-24 18:19 [PATCH v3 0/7] kbuild: various cleanups Masahiro Yamada
2022-09-24 18:19 ` [PATCH v3 1/7] kbuild: hard-code KBUILD_ALLDIRS in scripts/Makefile.package Masahiro Yamada
2022-09-28 19:09 ` Nicolas Schier
2022-09-24 18:19 ` [PATCH v3 2/7] kbuild: list sub-directories in ./Kbuild Masahiro Yamada
2022-09-24 19:37 ` kernel test robot
2022-09-24 20:27 ` kernel test robot
2022-09-25 0:28 ` Masahiro Yamada
2022-09-26 21:06 ` Nick Desaulniers
2022-09-28 8:32 ` Masahiro Yamada
2022-09-28 19:30 ` Nicolas Schier
2022-09-24 18:19 ` [PATCH v3 3/7] kbuild: move .vmlinux.objs rule to Makefile.modpost Masahiro Yamada
2022-09-28 19:35 ` Nicolas Schier
2022-09-24 18:19 ` [PATCH v3 4/7] kbuild: move vmlinux.o rule to the top Makefile Masahiro Yamada
2022-09-28 19:37 ` Nicolas Schier
2022-09-24 18:19 ` [PATCH v3 5/7] kbuild: unify two modpost invocations Masahiro Yamada
2022-09-28 19:59 ` Nicolas Schier
2022-09-28 21:05 ` Masahiro Yamada
2022-09-24 18:19 ` [PATCH v3 6/7] kbuild: use obj-y instead extra-y for objects placed at the head Masahiro Yamada
2022-09-28 20:15 ` Nicolas Schier
2022-10-24 18:24 ` Jiri Slaby
2022-10-25 12:26 ` Michael Matz
2022-10-26 8:35 ` Jiri Slaby
2022-10-26 11:20 ` Ard Biesheuvel
2022-10-26 16:29 ` Masahiro Yamada
2022-10-26 17:09 ` Michael Matz
2022-09-24 18:19 ` [PATCH v3 7/7] kbuild: remove head-y syntax Masahiro Yamada
2022-09-29 15:21 ` Nicolas Schier
2022-10-18 8:16 ` Jiri Slaby
2022-10-18 9:12 ` Masahiro Yamada
2022-09-26 21:39 ` Nick Desaulniers [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAKwvOdkREP8kFuE2UqMV4a+CRykKMO44X-12N6zcQoBbRYdJ4g@mail.gmail.com \
--to=ndesaulniers@google.com \
--cc=linux-arch@vger.kernel.org \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=masahiroy@kernel.org \
--cc=nathan@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).