* [PATCH] loadpin: remove MODULE_COMPRESS_NONE as it is no longer supported
@ 2025-01-13 9:31 Arulpandiyan Vadivel
2025-01-13 10:19 ` Greg KH
2025-01-13 13:42 ` Petr Pavlu
0 siblings, 2 replies; 9+ messages in thread
From: Arulpandiyan Vadivel @ 2025-01-13 9:31 UTC (permalink / raw)
To: linux-security-module
Cc: linux-modules, stable, cedric.hombourger, srikanth.krishnakar,
Arulpandiyan Vadivel
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@siemens.com>
---
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
--
2.39.5
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] loadpin: remove MODULE_COMPRESS_NONE as it is no longer supported
2025-01-13 9:31 [PATCH] " Arulpandiyan Vadivel
@ 2025-01-13 10:19 ` Greg KH
2025-01-13 13:42 ` Petr Pavlu
1 sibling, 0 replies; 9+ messages in thread
From: Greg KH @ 2025-01-13 10:19 UTC (permalink / raw)
To: Arulpandiyan Vadivel
Cc: linux-security-module, linux-modules, stable, cedric.hombourger,
srikanth.krishnakar
On Mon, Jan 13, 2025 at 03:01:15PM +0530, 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@siemens.com>
> ---
> 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
> --
> 2.39.5
>
>
<formletter>
This is not the correct way to submit patches for inclusion in the
stable kernel tree. Please read:
https://www.kernel.org/doc/html/latest/process/stable-kernel-rules.html
for how to do this properly.
</formletter>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] loadpin: remove MODULE_COMPRESS_NONE as it is no longer supported
2025-01-13 9:31 [PATCH] " Arulpandiyan Vadivel
2025-01-13 10:19 ` Greg KH
@ 2025-01-13 13:42 ` Petr Pavlu
1 sibling, 0 replies; 9+ messages in thread
From: Petr Pavlu @ 2025-01-13 13:42 UTC (permalink / raw)
To: Arulpandiyan Vadivel
Cc: linux-security-module, linux-modules, stable, cedric.hombourger,
srikanth.krishnakar
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@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
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] loadpin: remove MODULE_COMPRESS_NONE as it is no longer supported
@ 2025-01-14 16:17 Arulpandiyan Vadivel
2025-01-15 3:20 ` Paul Moore
0 siblings, 1 reply; 9+ messages in thread
From: Arulpandiyan Vadivel @ 2025-01-14 16:17 UTC (permalink / raw)
To: linux-security-module
Cc: linux-modules, cedric.hombourger, srikanth.krishnakar,
Arulpandiyan Vadivel
Updated the MODULE_COMPRESS_NONE with MODULE_COMPRESS as it was no longer
available from kernel modules.
Fixes: c7ff693fa209 ("module: Split modules_install compression and in-kernel decompression")
Signed-off-by: Arulpandiyan Vadivel <arulpandiyan.vadivel@siemens.com>
---
Changes in v2:
Reword the commit message
Modify logic and add Fixes tag.
---
security/loadpin/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/security/loadpin/Kconfig b/security/loadpin/Kconfig
index 848f8b4a60190..d82bcdb34cc8a 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_COMPRESS || MODULE_DECOMPRESS)
help
If selected, LoadPin will enforce pinning at boot. If not
selected, it can be enabled at boot with the kernel parameter
--
2.39.5
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] loadpin: remove MODULE_COMPRESS_NONE as it is no longer supported
2025-01-14 16:17 [PATCH] loadpin: remove MODULE_COMPRESS_NONE as it is no longer supported Arulpandiyan Vadivel
@ 2025-01-15 3:20 ` Paul Moore
2025-02-27 0:17 ` Paul Moore
0 siblings, 1 reply; 9+ messages in thread
From: Paul Moore @ 2025-01-15 3:20 UTC (permalink / raw)
To: Arulpandiyan Vadivel
Cc: linux-security-module, linux-modules, cedric.hombourger,
srikanth.krishnakar
On Tue, Jan 14, 2025 at 11:16 AM Arulpandiyan Vadivel
<arulpandiyan.vadivel@siemens.com> wrote:
>
> Updated the MODULE_COMPRESS_NONE with MODULE_COMPRESS as it was no longer
> available from kernel modules.
>
> Fixes: c7ff693fa209 ("module: Split modules_install compression and in-kernel decompression")
> Signed-off-by: Arulpandiyan Vadivel <arulpandiyan.vadivel@siemens.com>
> ---
> Changes in v2:
> Reword the commit message
> Modify logic and add Fixes tag.
> ---
> security/loadpin/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
For what it's worth, this looks fine to me.
Reviewed-by: Paul Moore <paul@paul-moore.com>
> diff --git a/security/loadpin/Kconfig b/security/loadpin/Kconfig
> index 848f8b4a60190..d82bcdb34cc8a 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_COMPRESS || MODULE_DECOMPRESS)
> help
> If selected, LoadPin will enforce pinning at boot. If not
> selected, it can be enabled at boot with the kernel parameter
> --
> 2.39.5
--
paul-moore.com
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] loadpin: remove MODULE_COMPRESS_NONE as it is no longer supported
2025-01-15 3:20 ` Paul Moore
@ 2025-02-27 0:17 ` Paul Moore
2025-02-28 19:42 ` Kees Cook
0 siblings, 1 reply; 9+ messages in thread
From: Paul Moore @ 2025-02-27 0:17 UTC (permalink / raw)
To: Arulpandiyan Vadivel, Kees Cook
Cc: linux-security-module, linux-modules, cedric.hombourger,
srikanth.krishnakar
On Tue, Jan 14, 2025 at 10:20 PM Paul Moore <paul@paul-moore.com> wrote:
> On Tue, Jan 14, 2025 at 11:16 AM Arulpandiyan Vadivel
> <arulpandiyan.vadivel@siemens.com> wrote:
> >
> > Updated the MODULE_COMPRESS_NONE with MODULE_COMPRESS as it was no longer
> > available from kernel modules.
> >
> > Fixes: c7ff693fa209 ("module: Split modules_install compression and in-kernel decompression")
> > Signed-off-by: Arulpandiyan Vadivel <arulpandiyan.vadivel@siemens.com>
> > ---
> > Changes in v2:
> > Reword the commit message
> > Modify logic and add Fixes tag.
> > ---
> > security/loadpin/Kconfig | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
>
> For what it's worth, this looks fine to me.
>
> Reviewed-by: Paul Moore <paul@paul-moore.com>
Kees, did you ever merge this into your LoadPin tree?
> > diff --git a/security/loadpin/Kconfig b/security/loadpin/Kconfig
> > index 848f8b4a60190..d82bcdb34cc8a 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_COMPRESS || MODULE_DECOMPRESS)
> > help
> > If selected, LoadPin will enforce pinning at boot. If not
> > selected, it can be enabled at boot with the kernel parameter
> > --
> > 2.39.5
--
paul-moore.com
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] loadpin: remove MODULE_COMPRESS_NONE as it is no longer supported
2025-02-27 0:17 ` Paul Moore
@ 2025-02-28 19:42 ` Kees Cook
2025-03-02 10:38 ` [PATCH V3] " Arulpandiyan Vadivel
0 siblings, 1 reply; 9+ messages in thread
From: Kees Cook @ 2025-02-28 19:42 UTC (permalink / raw)
To: Paul Moore
Cc: Arulpandiyan Vadivel, linux-security-module, linux-modules,
cedric.hombourger, srikanth.krishnakar
On Wed, Feb 26, 2025 at 07:17:33PM -0500, Paul Moore wrote:
> On Tue, Jan 14, 2025 at 10:20 PM Paul Moore <paul@paul-moore.com> wrote:
> > On Tue, Jan 14, 2025 at 11:16 AM Arulpandiyan Vadivel
> > <arulpandiyan.vadivel@siemens.com> wrote:
> > >
> > > Updated the MODULE_COMPRESS_NONE with MODULE_COMPRESS as it was no longer
> > > available from kernel modules.
> > >
> > > Fixes: c7ff693fa209 ("module: Split modules_install compression and in-kernel decompression")
> > > Signed-off-by: Arulpandiyan Vadivel <arulpandiyan.vadivel@siemens.com>
> > > ---
> > > Changes in v2:
> > > Reword the commit message
> > > Modify logic and add Fixes tag.
> > > ---
> > > security/loadpin/Kconfig | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > For what it's worth, this looks fine to me.
> >
> > Reviewed-by: Paul Moore <paul@paul-moore.com>
>
> Kees, did you ever merge this into your LoadPin tree?
Hi! This slipped through the cracks.
>
> > > diff --git a/security/loadpin/Kconfig b/security/loadpin/Kconfig
> > > index 848f8b4a60190..d82bcdb34cc8a 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_COMPRESS || MODULE_DECOMPRESS)
MODULE_COMPRESS depends on MODULES, so I think this can be reduced to:
depends on !MODULE_COMPRESS || MODULE_DECOMPRESS
-Kees
--
Kees Cook
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH V3] loadpin: remove MODULE_COMPRESS_NONE as it is no longer supported
2025-02-28 19:42 ` Kees Cook
@ 2025-03-02 10:38 ` Arulpandiyan Vadivel
2025-03-03 17:33 ` Kees Cook
0 siblings, 1 reply; 9+ messages in thread
From: Arulpandiyan Vadivel @ 2025-03-02 10:38 UTC (permalink / raw)
To: linux-security-module
Cc: cedric.hombourger, srikanth.krishnakar, linux-modules, paul, kees,
Arulpandiyan Vadivel
Updated the MODULE_COMPRESS_NONE with MODULE_COMPRESS as it was no longer
available from kernel modules. As MODULE_COMPRESS and MODULE_DECOMPRESS
depends on MODULES removing MODULES as well.
Fixes: c7ff693fa209 ("module: Split modules_install compression and in-kernel decompression")
Signed-off-by: Arulpandiyan Vadivel <arulpandiyan.vadivel@siemens.com>
---
Changes in v3:
Remove MODULES as suggested
Changes in v2:
Reword the commit message
Modify logic and add Fixes tag.
---
security/loadpin/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/security/loadpin/Kconfig b/security/loadpin/Kconfig
index 848f8b4a60190..aef63d3e30dfa 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 !MODULE_COMPRESS || MODULE_DECOMPRESS
help
If selected, LoadPin will enforce pinning at boot. If not
selected, it can be enabled at boot with the kernel parameter
--
2.39.5
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH V3] loadpin: remove MODULE_COMPRESS_NONE as it is no longer supported
2025-03-02 10:38 ` [PATCH V3] " Arulpandiyan Vadivel
@ 2025-03-03 17:33 ` Kees Cook
0 siblings, 0 replies; 9+ messages in thread
From: Kees Cook @ 2025-03-03 17:33 UTC (permalink / raw)
To: linux-security-module, Arulpandiyan Vadivel
Cc: Kees Cook, cedric.hombourger, srikanth.krishnakar, linux-modules,
paul
On Sun, 02 Mar 2025 16:08:31 +0530, Arulpandiyan Vadivel wrote:
> Updated the MODULE_COMPRESS_NONE with MODULE_COMPRESS as it was no longer
> available from kernel modules. As MODULE_COMPRESS and MODULE_DECOMPRESS
> depends on MODULES removing MODULES as well.
>
>
Applied to for-next/hardening, thanks!
[1/1] loadpin: remove MODULE_COMPRESS_NONE as it is no longer supported
https://git.kernel.org/kees/c/fdafb7e70cca
Take care,
--
Kees Cook
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2025-03-03 17:33 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-01-14 16:17 [PATCH] loadpin: remove MODULE_COMPRESS_NONE as it is no longer supported Arulpandiyan Vadivel
2025-01-15 3:20 ` Paul Moore
2025-02-27 0:17 ` Paul Moore
2025-02-28 19:42 ` Kees Cook
2025-03-02 10:38 ` [PATCH V3] " Arulpandiyan Vadivel
2025-03-03 17:33 ` Kees Cook
-- strict thread matches above, loose matches on Subject: below --
2025-01-13 9:31 [PATCH] " Arulpandiyan Vadivel
2025-01-13 10:19 ` Greg KH
2025-01-13 13:42 ` Petr Pavlu
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).