* [LTP] [PATCH 1/1] runtest/net.nfs: Run nfs02_06 on TCP only @ 2023-06-22 8:46 Petr Vorel 2023-07-12 7:51 ` Cyril Hrubis 2023-07-12 10:28 ` Jeff Layton 0 siblings, 2 replies; 5+ messages in thread From: Petr Vorel @ 2023-06-22 8:46 UTC (permalink / raw) To: ltp Cc: Olga Kornievskaia, Jeff Layton, linux-nfs, Trond Myklebust, Chuck Lever, Anna Schumaker UDP support disabled was on NFS server in kernel 5.6. Due that 2 of 3 nfs06.sh tests runs are being skipped on newer kernels. Therefore NFSv3 job in nfs02_06 test as TCP. This way all jobs in the test are TCP, thus test will not be skipped. This also bring NFSv3 testing also under TCP (previously it was tested only on UDP). Keep UDP in nfs01_06 jobs, so that NFSv3 on UDP is still covered for older kernels. Signed-off-by: Petr Vorel <pvorel@suse.cz> --- runtest/net.nfs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/runtest/net.nfs b/runtest/net.nfs index 72cf4b307..15a960017 100644 --- a/runtest/net.nfs +++ b/runtest/net.nfs @@ -58,7 +58,7 @@ nfs41_ipv6_05 nfs05.sh -6 -v 4.1 -t tcp nfs42_ipv6_05 nfs05.sh -6 -v 4.2 -t tcp nfs01_06 nfs06.sh -v "3,3,3,4,4,4" -t "udp,udp,tcp,tcp,tcp,tcp" -nfs02_06 nfs06.sh -v "3,4,4.1,4.2,4.2,4.2" -t "udp,tcp,tcp,tcp,tcp,tcp" +nfs02_06 nfs06.sh -v "3,4,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp" nfs03_ipv6_06 nfs06.sh -6 -v "4,4.1,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp" nfs3_07 nfs07.sh -v 3 -t udp -- 2.40.1 -- Mailing list info: https://lists.linux.it/listinfo/ltp ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [LTP] [PATCH 1/1] runtest/net.nfs: Run nfs02_06 on TCP only 2023-06-22 8:46 [LTP] [PATCH 1/1] runtest/net.nfs: Run nfs02_06 on TCP only Petr Vorel @ 2023-07-12 7:51 ` Cyril Hrubis 2023-07-12 8:37 ` Petr Vorel 2023-07-12 10:28 ` Jeff Layton 1 sibling, 1 reply; 5+ messages in thread From: Cyril Hrubis @ 2023-07-12 7:51 UTC (permalink / raw) To: Petr Vorel Cc: Olga Kornievskaia, Jeff Layton, linux-nfs, Anna Schumaker, Chuck Lever, Trond Myklebust, ltp Hi! Sounds reasonable. Acked-by: Cyril Hrubis <chrubis@suse.cz> -- Cyril Hrubis chrubis@suse.cz -- Mailing list info: https://lists.linux.it/listinfo/ltp ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [LTP] [PATCH 1/1] runtest/net.nfs: Run nfs02_06 on TCP only 2023-07-12 7:51 ` Cyril Hrubis @ 2023-07-12 8:37 ` Petr Vorel 0 siblings, 0 replies; 5+ messages in thread From: Petr Vorel @ 2023-07-12 8:37 UTC (permalink / raw) To: Cyril Hrubis Cc: Olga Kornievskaia, Jeff Layton, linux-nfs, Anna Schumaker, Chuck Lever, Trond Myklebust, ltp Hi Cyril, thanks a lot, merged! Kind regards, Petr -- Mailing list info: https://lists.linux.it/listinfo/ltp ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [LTP] [PATCH 1/1] runtest/net.nfs: Run nfs02_06 on TCP only 2023-06-22 8:46 [LTP] [PATCH 1/1] runtest/net.nfs: Run nfs02_06 on TCP only Petr Vorel 2023-07-12 7:51 ` Cyril Hrubis @ 2023-07-12 10:28 ` Jeff Layton 2023-07-12 11:18 ` Petr Vorel 1 sibling, 1 reply; 5+ messages in thread From: Jeff Layton @ 2023-07-12 10:28 UTC (permalink / raw) To: Petr Vorel, ltp Cc: Olga Kornievskaia, Chuck Lever, Anna Schumaker, linux-nfs, Trond Myklebust On Thu, 2023-06-22 at 10:46 +0200, Petr Vorel wrote: > UDP support disabled was on NFS server in kernel 5.6. > Due that 2 of 3 nfs06.sh tests runs are being skipped on newer kernels. > > Therefore NFSv3 job in nfs02_06 test as TCP. This way all jobs in the > test are TCP, thus test will not be skipped. This also bring NFSv3 > testing also under TCP (previously it was tested only on UDP). > > Keep UDP in nfs01_06 jobs, so that NFSv3 on UDP is still covered for > older kernels. > > Signed-off-by: Petr Vorel <pvorel@suse.cz> > --- > runtest/net.nfs | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/runtest/net.nfs b/runtest/net.nfs > index 72cf4b307..15a960017 100644 > --- a/runtest/net.nfs > +++ b/runtest/net.nfs > @@ -58,7 +58,7 @@ nfs41_ipv6_05 nfs05.sh -6 -v 4.1 -t tcp > nfs42_ipv6_05 nfs05.sh -6 -v 4.2 -t tcp > > nfs01_06 nfs06.sh -v "3,3,3,4,4,4" -t "udp,udp,tcp,tcp,tcp,tcp" > -nfs02_06 nfs06.sh -v "3,4,4.1,4.2,4.2,4.2" -t "udp,tcp,tcp,tcp,tcp,tcp" > +nfs02_06 nfs06.sh -v "3,4,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp" > nfs03_ipv6_06 nfs06.sh -6 -v "4,4.1,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp" > > nfs3_07 nfs07.sh -v 3 -t udp UDP support is more or less being deprecated at this point, so testing on tcp is preferred. Acked-by: Jeff Layton <jlayton@kernel.org> -- Mailing list info: https://lists.linux.it/listinfo/ltp ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [LTP] [PATCH 1/1] runtest/net.nfs: Run nfs02_06 on TCP only 2023-07-12 10:28 ` Jeff Layton @ 2023-07-12 11:18 ` Petr Vorel 0 siblings, 0 replies; 5+ messages in thread From: Petr Vorel @ 2023-07-12 11:18 UTC (permalink / raw) To: Jeff Layton Cc: Olga Kornievskaia, linux-nfs, Trond Myklebust, Chuck Lever, Anna Schumaker, ltp Hi Jeff, > On Thu, 2023-06-22 at 10:46 +0200, Petr Vorel wrote: > > UDP support disabled was on NFS server in kernel 5.6. > > Due that 2 of 3 nfs06.sh tests runs are being skipped on newer kernels. > > Therefore NFSv3 job in nfs02_06 test as TCP. This way all jobs in the > > test are TCP, thus test will not be skipped. This also bring NFSv3 > > testing also under TCP (previously it was tested only on UDP). > > Keep UDP in nfs01_06 jobs, so that NFSv3 on UDP is still covered for > > older kernels. > > Signed-off-by: Petr Vorel <pvorel@suse.cz> > > --- > > runtest/net.nfs | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/runtest/net.nfs b/runtest/net.nfs > > index 72cf4b307..15a960017 100644 > > --- a/runtest/net.nfs > > +++ b/runtest/net.nfs > > @@ -58,7 +58,7 @@ nfs41_ipv6_05 nfs05.sh -6 -v 4.1 -t tcp > > nfs42_ipv6_05 nfs05.sh -6 -v 4.2 -t tcp > > nfs01_06 nfs06.sh -v "3,3,3,4,4,4" -t "udp,udp,tcp,tcp,tcp,tcp" > > -nfs02_06 nfs06.sh -v "3,4,4.1,4.2,4.2,4.2" -t "udp,tcp,tcp,tcp,tcp,tcp" > > +nfs02_06 nfs06.sh -v "3,4,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp" > > nfs03_ipv6_06 nfs06.sh -6 -v "4,4.1,4.1,4.2,4.2,4.2" -t "tcp,tcp,tcp,tcp,tcp,tcp" > > nfs3_07 nfs07.sh -v 3 -t udp > UDP support is more or less being deprecated at this point, so testing > on tcp is preferred. > Acked-by: Jeff Layton <jlayton@kernel.org> too late to add the ack to the commit message (already merged), but thanks for your confirmation. FYI I kept some UDP testing because new LTP releases are sometimes used with *not so old* enterprise distros. But these UDP tests are single tests (there is one test run under TCP and one under UDP). nfs06.sh is specific that it combines 6 various NFS versions and TCP + UDP, and single UDP causes whole specific test run being cancelled. And again, I still kept nfs01_06 which uses older NFS (ver 3 and 4) with original UDP for older distros. Interesting question is when to drop UDP testing completely. I guess few years after kernel 5.6 was released, when all major enterprise distros are based on kernel >= 5.6 and there are no stable/LTS with older kernel (e.g. in 2026, when 5.4 and older LTS will be EOL). 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:[~2023-07-12 11:18 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2023-06-22 8:46 [LTP] [PATCH 1/1] runtest/net.nfs: Run nfs02_06 on TCP only Petr Vorel 2023-07-12 7:51 ` Cyril Hrubis 2023-07-12 8:37 ` Petr Vorel 2023-07-12 10:28 ` Jeff Layton 2023-07-12 11:18 ` Petr Vorel
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).