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.vadivel@xxxxxxxxxxx>
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
