Re: [PATCH v2 0/4] Miscellaneous cleanups

Christophe Leroy
2022-02-22T15:14:33Z

Le 22/02/2022 à 16:12, Christophe Leroy a écrit :
> 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()

Seems like I made a big mess with v2.

I will resend it as v3.

Sorry for that.

Christophe

>
> Changes in v2:
> - 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
>
> Christophe Leroy (4):
> module: Make module_enable_x() independent of
> CONFIG_ARCH_HAS_STRICT_MODULE_RWX
> module: Move module_enable_x() and frob_text() in strict_rwx.c
> module: Rework layout alignment to avoid BUG_ON()s
> module: Rename debug_align() as strict_align()
>
> kernel/module/Makefile | 3 +-
> kernel/module/internal.h | 26 +++----------
> kernel/module/kallsyms.c | 4 +-
> kernel/module/main.c | 58 ++++++-----------------------
> kernel/module/strict_rwx.c | 76 +++++++++++++++++++++++++++++++++-----
> 5 files changed, 88 insertions(+), 79 deletions(-)
>