* [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