public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
* [LTP] [PATCH] listmount04.c: update struct mnt_id_req support to kernel >= 6.17
@ 2026-01-02 11:51 Po-Hsu Lin via ltp
  2026-01-05  6:54 ` Andrea Cervesato via ltp
  2026-01-06 12:04 ` Petr Vorel
  0 siblings, 2 replies; 5+ messages in thread
From: Po-Hsu Lin via ltp @ 2026-01-02 11:51 UTC (permalink / raw)
  To: ltp; +Cc: po-hsu.lin

The kernel change 78f0e33cd6c93 ("fs/namespace: correctly handle errors
returned by grab_requested_mnt_ns") from 6.18 has been ported to upstream
6.17.9 [1][2].

Therefore change the expectation from >= 6.18 to >= 6.17 accodingly.

[1] https://lwn.net/Articles/1047684/
[2] https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?h=v6.17.9&id=8ff97ade

Signed-off-by: Po-Hsu Lin <po-hsu.lin@canonical.com>
---
 testcases/kernel/syscalls/listmount/listmount04.c | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/testcases/kernel/syscalls/listmount/listmount04.c b/testcases/kernel/syscalls/listmount/listmount04.c
index 2b8b49101..34789353b 100644
--- a/testcases/kernel/syscalls/listmount/listmount04.c
+++ b/testcases/kernel/syscalls/listmount/listmount04.c
@@ -20,8 +20,8 @@
 #include "lapi/syscalls.h"
 
 #define MNT_SIZE 32
-#define BEFORE_6_18 1
-#define AFTER_6_18 2
+#define BEFORE_6_17 1
+#define AFTER_6_17 2
 
 static mnt_id_req *request;
 static uint64_t mnt_ids[MNT_SIZE];
@@ -84,7 +84,7 @@ static struct tcase {
 		.nr_mnt_ids = MNT_SIZE,
 		.exp_errno = EINVAL,
 		.msg = "invalid mnt_id_req.spare",
-		.kver = BEFORE_6_18,
+		.kver = BEFORE_6_17,
 	},
 	{
 		.req_usage = 1,
@@ -95,7 +95,7 @@ static struct tcase {
 		.nr_mnt_ids = MNT_SIZE,
 		.exp_errno = EBADF,
 		.msg = "invalid mnt_id_req.mnt_ns_fd",
-		.kver = AFTER_6_18,
+		.kver = AFTER_6_17,
 	},
 	{
 		.req_usage = 1,
@@ -154,10 +154,10 @@ static void run(unsigned int n)
 
 static void setup(void)
 {
-	if (tst_kvercmp(6, 18, 0) >= 0)
-		kver = AFTER_6_18;
+	if (tst_kvercmp(6, 17, 0) >= 0)
+		kver = AFTER_6_17;
 	else
-		kver = BEFORE_6_18;
+		kver = BEFORE_6_17;
 }
 
 static struct tst_test test = {
-- 
2.43.0


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

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

* Re: [LTP] [PATCH] listmount04.c: update struct mnt_id_req support to kernel >= 6.17
  2026-01-02 11:51 [LTP] [PATCH] listmount04.c: update struct mnt_id_req support to kernel >= 6.17 Po-Hsu Lin via ltp
@ 2026-01-05  6:54 ` Andrea Cervesato via ltp
  2026-01-06 12:04 ` Petr Vorel
  1 sibling, 0 replies; 5+ messages in thread
From: Andrea Cervesato via ltp @ 2026-01-05  6:54 UTC (permalink / raw)
  To: Po-Hsu Lin, ltp

Hi!

Reviewed-by: Andrea Cervesato <andrea.cervesato@suse.com>

-- 
Andrea Cervesato
SUSE QE Automation Engineer Linux
andrea.cervesato@suse.com


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

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

* Re: [LTP] [PATCH] listmount04.c: update struct mnt_id_req support to kernel >= 6.17
  2026-01-02 11:51 [LTP] [PATCH] listmount04.c: update struct mnt_id_req support to kernel >= 6.17 Po-Hsu Lin via ltp
  2026-01-05  6:54 ` Andrea Cervesato via ltp
@ 2026-01-06 12:04 ` Petr Vorel
  2026-01-07 10:27   ` Po-Hsu Lin via ltp
  1 sibling, 1 reply; 5+ messages in thread
From: Petr Vorel @ 2026-01-06 12:04 UTC (permalink / raw)
  To: Po-Hsu Lin; +Cc: ltp

> The kernel change 78f0e33cd6c93 ("fs/namespace: correctly handle errors
> returned by grab_requested_mnt_ns") from 6.18 has been ported to upstream
> 6.17.9 [1][2].

Good catch.

> Therefore change the expectation from >= 6.18 to >= 6.17 accodingly.

> [1] https://lwn.net/Articles/1047684/
> [2] https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?h=v6.17.9&id=8ff97ade

...
>  static void setup(void)
>  {
> -	if (tst_kvercmp(6, 18, 0) >= 0)
> -		kver = AFTER_6_18;
> +	if (tst_kvercmp(6, 17, 0) >= 0)

Shouldn't we match 6.17.9 then?
It's not about the fix but about a kernel change.

Kind regards,
Petr

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

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

* Re: [LTP] [PATCH] listmount04.c: update struct mnt_id_req support to kernel >= 6.17
  2026-01-06 12:04 ` Petr Vorel
@ 2026-01-07 10:27   ` Po-Hsu Lin via ltp
  2026-01-07 10:38     ` Petr Vorel
  0 siblings, 1 reply; 5+ messages in thread
From: Po-Hsu Lin via ltp @ 2026-01-07 10:27 UTC (permalink / raw)
  To: Petr Vorel; +Cc: ltp

On Tue, 6 Jan 2026 at 20:04, Petr Vorel <pvorel@suse.cz> wrote:

> > The kernel change 78f0e33cd6c93 ("fs/namespace: correctly handle errors
> > returned by grab_requested_mnt_ns") from 6.18 has been ported to upstream
> > 6.17.9 [1][2].
>
> Good catch.
>
> > Therefore change the expectation from >= 6.18 to >= 6.17 accodingly.
>
> > [1] https://lwn.net/Articles/1047684/
> > [2]
> https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?h=v6.17.9&id=8ff97ade
>
> ...
> >  static void setup(void)
> >  {
> > -     if (tst_kvercmp(6, 18, 0) >= 0)
> > -             kver = AFTER_6_18;
> > +     if (tst_kvercmp(6, 17, 0) >= 0)
>
> Shouldn't we match 6.17.9 then?
> It's not about the fix but about a kernel change.
>
> Hi Petr,
thanks for the feedback,
in this case should I also modify the macros to BEFORE_6_17_9 and
AFTER_6_17_9?


> Kind regards,
> Petr
>

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

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

* Re: [LTP] [PATCH] listmount04.c: update struct mnt_id_req support to kernel >= 6.17
  2026-01-07 10:27   ` Po-Hsu Lin via ltp
@ 2026-01-07 10:38     ` Petr Vorel
  0 siblings, 0 replies; 5+ messages in thread
From: Petr Vorel @ 2026-01-07 10:38 UTC (permalink / raw)
  To: Po-Hsu Lin; +Cc: ltp

...
> > > -     if (tst_kvercmp(6, 18, 0) >= 0)
> > > -             kver = AFTER_6_18;
> > > +     if (tst_kvercmp(6, 17, 0) >= 0)

> > Shouldn't we match 6.17.9 then?
> > It's not about the fix but about a kernel change.

> > Hi Petr,
> thanks for the feedback,
> in this case should I also modify the macros to BEFORE_6_17_9 and
> AFTER_6_17_9?

Yes, probably the best will be to use BEFORE_6_17_9 and put a comment that
78f0e33cd6c93 ("fs/namespace: correctly handle errors > > > returned by grab_requested_mnt_ns")
from 6.18 was backported to 6.17.9.

Thank you!

Kind regards,
Petr

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

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

end of thread, other threads:[~2026-01-07 10:38 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-01-02 11:51 [LTP] [PATCH] listmount04.c: update struct mnt_id_req support to kernel >= 6.17 Po-Hsu Lin via ltp
2026-01-05  6:54 ` Andrea Cervesato via ltp
2026-01-06 12:04 ` Petr Vorel
2026-01-07 10:27   ` Po-Hsu Lin via ltp
2026-01-07 10:38     ` Petr Vorel

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