Re: [PATCH v11 3/5] modules: Allow extended modversions without basic MODVERSIONS

[Masahiro Yamada]

On Tue, Dec 24, 2024 at 5:13 AM Matthew Maurer <mmaurer@xxxxxxxxxx> wrote:
>
> If you know that your kernel modules will only ever be loaded by a newer
> kernel, you can disable BASIC_MODVERSIONS to save space. This also
> allows easy creation of test modules to seee how tooling will respond to


seee -> see


> modules that only have the new format.
>
> Signed-off-by: Matthew Maurer <mmaurer@xxxxxxxxxx>
> ---
>  kernel/module/Kconfig    | 15 +++++++++++++++
>  scripts/Makefile.modpost |  1 +
>  scripts/mod/modpost.c    | 19 +++++++++++--------
>  3 files changed, 27 insertions(+), 8 deletions(-)
>
> diff --git a/kernel/module/Kconfig b/kernel/module/Kconfig
> index 9568b629a03ce8289d3f3597eefc66fc96445720..5f701391ab12c7d1b3ca496c9b3443c5a76a8745 100644
> --- a/kernel/module/Kconfig
> +++ b/kernel/module/Kconfig
> @@ -217,6 +217,21 @@ config EXTENDED_MODVERSIONS
>           The most likely reason you would enable this is to enable Rust
>           support. If unsure, say N.
>
> +config BASIC_MODVERSIONS
> +       bool "Basic Module Versioning Support"
> +       depends on MODVERSIONS
> +       default MODVERSIONS

Nit:

            default y

is simpler.


--
Best Regards

Masahiro Yamada