public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
* [LTP] [PATCH v1] mount08.c: Add min_kver for test case
@ 2025-09-29 11:29 Wei Gao via ltp
  2025-09-29 12:05 ` Cyril Hrubis
  0 siblings, 1 reply; 6+ messages in thread
From: Wei Gao via ltp @ 2025-09-29 11:29 UTC (permalink / raw)
  To: ltp

Signed-off-by: Wei Gao <wegao@suse.com>
---
 testcases/kernel/syscalls/mount/mount08.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/testcases/kernel/syscalls/mount/mount08.c b/testcases/kernel/syscalls/mount/mount08.c
index fb2b28736..1c454cab4 100644
--- a/testcases/kernel/syscalls/mount/mount08.c
+++ b/testcases/kernel/syscalls/mount/mount08.c
@@ -63,6 +63,7 @@ static struct tst_test test = {
 	.test_all = run,
 	.needs_root = 1,
 	.mntpoint = MNTPOINT,
+	.min_kver = "6.12",
 	.tags = (const struct tst_tag[]) {
 		{"linux-git", "d80b065bb172"},
 		{}
-- 
2.51.0


-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [LTP] [PATCH v1] mount08.c: Add min_kver for test case
  2025-09-29 11:29 [LTP] [PATCH v1] mount08.c: Add min_kver for test case Wei Gao via ltp
@ 2025-09-29 12:05 ` Cyril Hrubis
  2025-09-29 12:09   ` Martin Doucha
  0 siblings, 1 reply; 6+ messages in thread
From: Cyril Hrubis @ 2025-09-29 12:05 UTC (permalink / raw)
  To: Wei Gao; +Cc: ltp

Hi!
Again, there has to be explanation here why this is needed...

But given that this is a regression test for a security hardening that
was added into 6.12 it's okay if the test fails on older kernel,
moreover hiding the failure may make the system more vunerable.

-- 
Cyril Hrubis
chrubis@suse.cz

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [LTP] [PATCH v1] mount08.c: Add min_kver for test case
  2025-09-29 12:05 ` Cyril Hrubis
@ 2025-09-29 12:09   ` Martin Doucha
  2025-09-29 18:11     ` Petr Vorel
  0 siblings, 1 reply; 6+ messages in thread
From: Martin Doucha @ 2025-09-29 12:09 UTC (permalink / raw)
  To: Cyril Hrubis, Wei Gao; +Cc: ltp

On 9/29/25 14:05, Cyril Hrubis wrote:
> Hi!
> Again, there has to be explanation here why this is needed...
> 
> But given that this is a regression test for a security hardening that
> was added into 6.12 it's okay if the test fails on older kernel,
> moreover hiding the failure may make the system more vunerable.

Hi,
I agree with rejecting this patch. If a kernel is not patched, the test 
should fail regardless of kernel version.

-- 
Martin Doucha   mdoucha@suse.cz
SW Quality Engineer
SUSE LINUX, s.r.o.
CORSO IIa
Krizikova 148/34
186 00 Prague 8
Czech Republic

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [LTP] [PATCH v1] mount08.c: Add min_kver for test case
  2025-09-29 12:09   ` Martin Doucha
@ 2025-09-29 18:11     ` Petr Vorel
  2025-09-30  8:58       ` Petr Vorel
  0 siblings, 1 reply; 6+ messages in thread
From: Petr Vorel @ 2025-09-29 18:11 UTC (permalink / raw)
  To: Martin Doucha; +Cc: ltp

Hi all,

> On 9/29/25 14:05, Cyril Hrubis wrote:
> > Hi!
> > Again, there has to be explanation here why this is needed...

> > But given that this is a regression test for a security hardening that
> > was added into 6.12 it's okay if the test fails on older kernel,
> > moreover hiding the failure may make the system more vunerable.

> Hi,
> I agree with rejecting this patch. If a kernel is not patched, the test
> should fail regardless of kernel version.

For a record, d80b065bb172 is a merge commit, the fix contains actually 7
commits. IMHO none of them has been backported to any stable / LTS kernel.  I'll
ask Christian and Greg if it makes sense to backport it.

Kind regards,
Petr

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [LTP] [PATCH v1] mount08.c: Add min_kver for test case
  2025-09-29 18:11     ` Petr Vorel
@ 2025-09-30  8:58       ` Petr Vorel
  2025-10-01 10:47         ` Petr Vorel
  0 siblings, 1 reply; 6+ messages in thread
From: Petr Vorel @ 2025-09-30  8:58 UTC (permalink / raw)
  To: Martin Doucha, ltp

Hi all,

...
> For a record, d80b065bb172 is a merge commit, the fix contains actually 7
> commits. IMHO none of them has been backported to any stable / LTS kernel.  I'll
> ask Christian and Greg if it makes sense to backport it.

asked [1]. (unfortunately I forgot to delete message-id and From: from the
original message).

Kind regards,
Petr

[1] https://lore.kernel.org/all/20240806-work-procfs-v1-0-fb04e1d09f0c@kernel.org/

> Kind regards,
> Petr

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [LTP] [PATCH v1] mount08.c: Add min_kver for test case
  2025-09-30  8:58       ` Petr Vorel
@ 2025-10-01 10:47         ` Petr Vorel
  0 siblings, 0 replies; 6+ messages in thread
From: Petr Vorel @ 2025-10-01 10:47 UTC (permalink / raw)
  To: Martin Doucha, Cyril Hrubis, Wei Gao; +Cc: ltp

Hi all,

...
> ...
> > For a record, d80b065bb172 is a merge commit, the fix contains actually 7
> > commits. IMHO none of them has been backported to any stable / LTS kernel.  I'll
> > ask Christian and Greg if it makes sense to backport it.

> asked [1]. (unfortunately I forgot to delete message-id and From: from the
> original message).

FYI answer from Aleksa [2] suggests it will not be backported, but let's wait
if somebody else give his opinion. Anyway, I'm not advocating for this patch
(to add .min_kver = "6.12"), although it wouldn't harm to document it in the
test that unlike kernel folks which consider it as a "new feature" we consider
it as a "security fix".

Kind regards,
Petr

> Kind regards,
> Petr

> [1] https://lore.kernel.org/all/20240806-work-procfs-v1-0-fb04e1d09f0c@kernel.org/
[2] https://lore.kernel.org/linux-fsdevel/2025-09-30-emerald-unsure-pillow-prism-nKVGLB@cyphar.com/

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2025-10-01 10:47 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-09-29 11:29 [LTP] [PATCH v1] mount08.c: Add min_kver for test case Wei Gao via ltp
2025-09-29 12:05 ` Cyril Hrubis
2025-09-29 12:09   ` Martin Doucha
2025-09-29 18:11     ` Petr Vorel
2025-09-30  8:58       ` Petr Vorel
2025-10-01 10:47         ` Petr Vorel

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox