From: Nathan Chancellor <nathan@kernel.org>
To: Masahiro Yamada <masahiroy@kernel.org>
Cc: linux-kbuild@vger.kernel.org, clang-built-linux@googlegroups.com,
linux-kernel@vger.kernel.org,
Nicolas Schier a <nicolas@fjasle.eu>,
Ard Biesheuvel <ardb@kernel.org>,
Luis Chamberlain <mcgrof@kernel.org>,
Peter Zijlstra <peterz@infradead.org>,
linuxppc-dev@lists.ozlabs.org, linux-um@lists.infradead.org,
linux-s390@vger.kernel.org,
Nick Desaulniers <ndesaulniers@google.com>,
Sami Tolvanen <samitolvanen@google.com>,
Kees Cook <keescook@chromium.org>
Subject: Re: [PATCH v3 00/15] kbuild: yet another series of cleanups (modpost, LTO, MODULE_REL_CRCS)
Date: Fri, 6 May 2022 15:45:28 -0700 [thread overview]
Message-ID: <YnWlCH2tfr5YMb1/@dev-arch.thelio-3990X> (raw)
In-Reply-To: <20220505072244.1155033-1-masahiroy@kernel.org>
Hi Masahiro,
On Thu, May 05, 2022 at 04:22:29PM +0900, Masahiro Yamada wrote:
>
> This is the third batch of cleanups in this development cycle.
>
> Major changes in v3:
>
> - Generate symbol CRCs as C code, and remove CONFIG_MODULE_REL_CRCS.
>
> Major changes in v2:
>
> - V1 did not work with CONFIG_MODULE_REL_CRCS.
> I fixed this for v2.
>
> - Reflect some review comments in v1
>
> - Refactor the code more
>
> - Avoid too long argument error
>
>
> Masahiro Yamada (15):
> modpost: mitigate false-negatives for static EXPORT_SYMBOL checks
> modpost: change the license of EXPORT_SYMBOL to bool type
> modpost: merge add_{intree_flag,retpoline,staging_flag} to add_header
> modpost: move *.mod.c generation to write_mod_c_files()
> kbuild: generate a list of objects in vmlinux
> kbuild: record symbol versions in *.cmd files
> modpost: extract symbol versions from *.cmd files
> kbuild: link symbol CRCs at final link, removing
> CONFIG_MODULE_REL_CRCS
> kbuild: stop merging *.symversions
> genksyms: adjust the output format to modpost
> kbuild: do not create *.prelink.o for Clang LTO or IBT
> modpost: simplify the ->is_static initialization
> modpost: use hlist for hash table implementation
> kbuild: make built-in.a rule robust against too long argument error
> kbuild: make *.mod rule robust against too long argument error
I merged this series into mainline and tested an Arch Linux
x86_64 configuration and Fedora aarch64 configuration with ThinLTO and
saw no new warnings or issues. Modules loaded just fine in QEMU for Arch
Linux and I did not notice any boot issues or warnings.
Tested-by: Nathan Chancellor <nathan@kernel.org>
Cheers,
Nathan
WARNING: multiple messages have this Message-ID (diff)
From: Nathan Chancellor <nathan@kernel.org>
To: Masahiro Yamada <masahiroy@kernel.org>
Cc: linux-s390@vger.kernel.org, Nicolas Schier a <nicolas@fjasle.eu>,
linux-kbuild@vger.kernel.org,
Peter Zijlstra <peterz@infradead.org>,
Nick Desaulniers <ndesaulniers@google.com>,
linux-um@lists.infradead.org, linux-kernel@vger.kernel.org,
clang-built-linux@googlegroups.com,
Luis Chamberlain <mcgrof@kernel.org>,
Sami Tolvanen <samitolvanen@google.com>,
linuxppc-dev@lists.ozlabs.org, Ard Biesheuvel <ardb@kernel.org>,
Kees Cook <keescook@chromium.org>
Subject: Re: [PATCH v3 00/15] kbuild: yet another series of cleanups (modpost, LTO, MODULE_REL_CRCS)
Date: Fri, 6 May 2022 15:45:28 -0700 [thread overview]
Message-ID: <YnWlCH2tfr5YMb1/@dev-arch.thelio-3990X> (raw)
In-Reply-To: <20220505072244.1155033-1-masahiroy@kernel.org>
Hi Masahiro,
On Thu, May 05, 2022 at 04:22:29PM +0900, Masahiro Yamada wrote:
>
> This is the third batch of cleanups in this development cycle.
>
> Major changes in v3:
>
> - Generate symbol CRCs as C code, and remove CONFIG_MODULE_REL_CRCS.
>
> Major changes in v2:
>
> - V1 did not work with CONFIG_MODULE_REL_CRCS.
> I fixed this for v2.
>
> - Reflect some review comments in v1
>
> - Refactor the code more
>
> - Avoid too long argument error
>
>
> Masahiro Yamada (15):
> modpost: mitigate false-negatives for static EXPORT_SYMBOL checks
> modpost: change the license of EXPORT_SYMBOL to bool type
> modpost: merge add_{intree_flag,retpoline,staging_flag} to add_header
> modpost: move *.mod.c generation to write_mod_c_files()
> kbuild: generate a list of objects in vmlinux
> kbuild: record symbol versions in *.cmd files
> modpost: extract symbol versions from *.cmd files
> kbuild: link symbol CRCs at final link, removing
> CONFIG_MODULE_REL_CRCS
> kbuild: stop merging *.symversions
> genksyms: adjust the output format to modpost
> kbuild: do not create *.prelink.o for Clang LTO or IBT
> modpost: simplify the ->is_static initialization
> modpost: use hlist for hash table implementation
> kbuild: make built-in.a rule robust against too long argument error
> kbuild: make *.mod rule robust against too long argument error
I merged this series into mainline and tested an Arch Linux
x86_64 configuration and Fedora aarch64 configuration with ThinLTO and
saw no new warnings or issues. Modules loaded just fine in QEMU for Arch
Linux and I did not notice any boot issues or warnings.
Tested-by: Nathan Chancellor <nathan@kernel.org>
Cheers,
Nathan
WARNING: multiple messages have this Message-ID (diff)
From: Nathan Chancellor <nathan@kernel.org>
To: Masahiro Yamada <masahiroy@kernel.org>
Cc: linux-kbuild@vger.kernel.org, clang-built-linux@googlegroups.com,
linux-kernel@vger.kernel.org,
Nicolas Schier a <nicolas@fjasle.eu>,
Ard Biesheuvel <ardb@kernel.org>,
Luis Chamberlain <mcgrof@kernel.org>,
Peter Zijlstra <peterz@infradead.org>,
linuxppc-dev@lists.ozlabs.org, linux-um@lists.infradead.org,
linux-s390@vger.kernel.org,
Nick Desaulniers <ndesaulniers@google.com>,
Sami Tolvanen <samitolvanen@google.com>,
Kees Cook <keescook@chromium.org>
Subject: Re: [PATCH v3 00/15] kbuild: yet another series of cleanups (modpost, LTO, MODULE_REL_CRCS)
Date: Fri, 6 May 2022 15:45:28 -0700 [thread overview]
Message-ID: <YnWlCH2tfr5YMb1/@dev-arch.thelio-3990X> (raw)
In-Reply-To: <20220505072244.1155033-1-masahiroy@kernel.org>
Hi Masahiro,
On Thu, May 05, 2022 at 04:22:29PM +0900, Masahiro Yamada wrote:
>
> This is the third batch of cleanups in this development cycle.
>
> Major changes in v3:
>
> - Generate symbol CRCs as C code, and remove CONFIG_MODULE_REL_CRCS.
>
> Major changes in v2:
>
> - V1 did not work with CONFIG_MODULE_REL_CRCS.
> I fixed this for v2.
>
> - Reflect some review comments in v1
>
> - Refactor the code more
>
> - Avoid too long argument error
>
>
> Masahiro Yamada (15):
> modpost: mitigate false-negatives for static EXPORT_SYMBOL checks
> modpost: change the license of EXPORT_SYMBOL to bool type
> modpost: merge add_{intree_flag,retpoline,staging_flag} to add_header
> modpost: move *.mod.c generation to write_mod_c_files()
> kbuild: generate a list of objects in vmlinux
> kbuild: record symbol versions in *.cmd files
> modpost: extract symbol versions from *.cmd files
> kbuild: link symbol CRCs at final link, removing
> CONFIG_MODULE_REL_CRCS
> kbuild: stop merging *.symversions
> genksyms: adjust the output format to modpost
> kbuild: do not create *.prelink.o for Clang LTO or IBT
> modpost: simplify the ->is_static initialization
> modpost: use hlist for hash table implementation
> kbuild: make built-in.a rule robust against too long argument error
> kbuild: make *.mod rule robust against too long argument error
I merged this series into mainline and tested an Arch Linux
x86_64 configuration and Fedora aarch64 configuration with ThinLTO and
saw no new warnings or issues. Modules loaded just fine in QEMU for Arch
Linux and I did not notice any boot issues or warnings.
Tested-by: Nathan Chancellor <nathan@kernel.org>
Cheers,
Nathan
_______________________________________________
linux-um mailing list
linux-um@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-um
next prev parent reply other threads:[~2022-05-06 22:45 UTC|newest]
Thread overview: 93+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-05-05 7:22 [PATCH v3 00/15] kbuild: yet another series of cleanups (modpost, LTO, MODULE_REL_CRCS) Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` [PATCH v3 01/15] modpost: mitigate false-negatives for static EXPORT_SYMBOL checks Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 19:25 ` Nicolas Schier
2022-05-05 19:25 ` Nicolas Schier
2022-05-05 19:25 ` Nicolas Schier
2022-05-05 7:22 ` [PATCH v3 02/15] modpost: change the license of EXPORT_SYMBOL to bool type Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 13:48 ` Masahiro Yamada
2022-05-05 13:48 ` Masahiro Yamada
2022-05-05 13:48 ` Masahiro Yamada
2022-05-05 19:53 ` Nicolas Schier
2022-05-05 19:53 ` Nicolas Schier
2022-05-05 19:53 ` Nicolas Schier
2022-05-05 7:22 ` [PATCH v3 03/15] modpost: merge add_{intree_flag,retpoline,staging_flag} to add_header Masahiro Yamada
2022-05-05 7:22 ` [PATCH v3 03/15] modpost: merge add_{intree_flag, retpoline, staging_flag} " Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 19:58 ` [PATCH v3 03/15] modpost: merge add_{intree_flag,retpoline,staging_flag} " Nicolas Schier
2022-05-05 19:58 ` Nicolas Schier
2022-05-05 19:58 ` Nicolas Schier
2022-05-05 7:22 ` [PATCH v3 04/15] modpost: move *.mod.c generation to write_mod_c_files() Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 20:06 ` Nicolas Schier
2022-05-05 20:06 ` Nicolas Schier
2022-05-05 20:06 ` Nicolas Schier
2022-05-05 7:22 ` [PATCH v3 05/15] kbuild: generate a list of objects in vmlinux Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` [PATCH v3 06/15] kbuild: record symbol versions in *.cmd files Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` [PATCH v3 07/15] modpost: extract symbol versions from " Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 20:09 ` Nicolas Schier
2022-05-05 20:09 ` Nicolas Schier
2022-05-05 20:09 ` Nicolas Schier
2022-05-05 7:22 ` [PATCH v3 08/15] kbuild: link symbol CRCs at final link, removing CONFIG_MODULE_REL_CRCS Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` [PATCH v3 09/15] kbuild: stop merging *.symversions Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 20:19 ` Nicolas Schier
2022-05-05 20:19 ` Nicolas Schier
2022-05-05 20:19 ` Nicolas Schier
2022-05-05 7:22 ` [PATCH v3 10/15] genksyms: adjust the output format to modpost Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 20:22 ` Nicolas Schier
2022-05-05 20:22 ` Nicolas Schier
2022-05-05 20:22 ` Nicolas Schier
2022-05-05 7:22 ` [PATCH v3 11/15] kbuild: do not create *.prelink.o for Clang LTO or IBT Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` [PATCH v3 12/15] modpost: simplify the ->is_static initialization Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 20:27 ` Nicolas Schier
2022-05-05 20:27 ` Nicolas Schier
2022-05-05 20:27 ` Nicolas Schier
2022-05-05 7:22 ` [PATCH v3 13/15] modpost: use hlist for hash table implementation Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` [PATCH v3 14/15] kbuild: make built-in.a rule robust against too long argument error Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 20:29 ` Nicolas Schier
2022-05-05 20:29 ` Nicolas Schier
2022-05-05 20:29 ` Nicolas Schier
2022-05-05 20:31 ` Nick Desaulniers
2022-05-05 20:31 ` Nick Desaulniers
2022-05-05 20:31 ` Nick Desaulniers
2022-05-05 7:22 ` [PATCH v3 15/15] kbuild: make *.mod " Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 7:22 ` Masahiro Yamada
2022-05-05 20:31 ` Nicolas Schier
2022-05-05 20:31 ` Nicolas Schier
2022-05-05 20:31 ` Nicolas Schier
2022-05-05 16:49 ` [PATCH v3 00/15] kbuild: yet another series of cleanups (modpost, LTO, MODULE_REL_CRCS) Masahiro Yamada
2022-05-05 16:49 ` Masahiro Yamada
2022-05-05 16:49 ` Masahiro Yamada
2022-05-06 22:45 ` Nathan Chancellor [this message]
2022-05-06 22:45 ` Nathan Chancellor
2022-05-06 22:45 ` Nathan Chancellor
2022-05-08 18:28 ` Masahiro Yamada
2022-05-08 18:28 ` Masahiro Yamada
2022-05-08 18:28 ` Masahiro Yamada
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=YnWlCH2tfr5YMb1/@dev-arch.thelio-3990X \
--to=nathan@kernel.org \
--cc=ardb@kernel.org \
--cc=clang-built-linux@googlegroups.com \
--cc=keescook@chromium.org \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-s390@vger.kernel.org \
--cc=linux-um@lists.infradead.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=masahiroy@kernel.org \
--cc=mcgrof@kernel.org \
--cc=ndesaulniers@google.com \
--cc=nicolas@fjasle.eu \
--cc=peterz@infradead.org \
--cc=samitolvanen@google.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.