Re: [PATCH v3 0/4] Miscellaneous cleanups

Luis Chamberlain
2022-02-23T00:27:27Z

On Tue, Feb 22, 2022 at 04:16:30PM +0100, Christophe Leroy wrote:
> This series applies on top of my series "Allocate module text and data separately" v5.
>
> It does some cleanup around the settling of modules's layout page flags:
> - Make module_enable_x() independent of CONFIG_ARCH_HAS_STRICT_MODULE_RWX
> - Reunify everything into strict_rwx.c
> - Change big hammer BUG_ON()s into WARN_ON()s
> - Change misleading debug_align() to strict_align()
>
> Changes in v2/v3:
> - Rebased on top of my series "Allocate module text and data separately" v5
> - Moved the Kconfig patch out of this series, it is an independent change
>
> Sorry for the mess in v2.

Thanks for all this work! Any chance you can actually submit this prior
to your allocate module series? This and the patch, "module: Move
module's Kconfig item in kernel/module/". Because *iff* we get enough
testing out of Aaron's changes and your cleanups, perhaps those might
be candidates for the next merge window. Then we can save all the new
functional changes for the other merge window.

Luis