On 1/13/25 10:31, Arulpandiyan Vadivel wrote:
> Commit c7ff693fa2094ba0a9d0a20feb4ab1658eff9c33 ("module: Split
> modules_install compression and in-kernel decompression") removed the
> MODULE_COMPRESS_NONE, but left it loadpin's Kconfig, and removing it
> 
> Signed-off-by: Arulpandiyan Vadivel <arulpandiyan.vadi...@siemens.com>

Please use a Fixes tag to record the problematic commit:

Fixes: c7ff693fa209 ("module: Split modules_install compression and in-kernel 
decompression")

> ---
>  security/loadpin/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/security/loadpin/Kconfig b/security/loadpin/Kconfig
> index 848f8b4a60190..94348e2831db9 100644
> --- a/security/loadpin/Kconfig
> +++ b/security/loadpin/Kconfig
> @@ -16,7 +16,7 @@ config SECURITY_LOADPIN_ENFORCE
>       depends on SECURITY_LOADPIN
>       # Module compression breaks LoadPin unless modules are decompressed in
>       # the kernel.
> -     depends on !MODULES || (MODULE_COMPRESS_NONE || MODULE_DECOMPRESS)
> +     depends on !MODULES || MODULE_DECOMPRESS
>       help
>         If selected, LoadPin will enforce pinning at boot. If not
>         selected, it can be enabled at boot with the kernel parameter

I think this should be updated to:

        depends on !MODULES || (!MODULE_COMPRESS || MODULE_DECOMPRESS)

-- 
Thanks,
Petr

Reply via email to