[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <2e58d7f5c493d6f158042fb39f299d24c7b60591.camel@redhat.com>
Date: Thu, 30 May 2024 15:58:05 +0200
From: Paolo Abeni <pabeni@...hat.com>
To: Breno Leitao <leitao@...ian.org>, Matteo Croce <technoboy85@...il.com>
Cc: netdev@...r.kernel.org, "David S. Miller" <davem@...emloft.net>, Eric
Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>, Shuah Khan
<shuah@...nel.org>, Shakeel Butt <shakeel.butt@...ux.dev>,
linux-kernel@...r.kernel.org, linux-kselftest@...r.kernel.org
Subject: Re: [PATCH net-next 2/2] selftests: net: tests
net.core.{r,w}mem_{default,max} sysctls in a netns
On Thu, 2024-05-30 at 05:10 -0700, Breno Leitao wrote:
> On Tue, May 28, 2024 at 02:11:39PM +0200, Matteo Croce wrote:
> > Add a selftest which checks that the sysctl is present in a netns,
> > that the value is read from the init one, and that it's readonly.
> >
> > Signed-off-by: Matteo Croce <teknoraver@...a.com>
> > ---
> > tools/testing/selftests/net/Makefile | 1 +
> > tools/testing/selftests/net/netns-sysctl.sh | 15 +++++++++++++++
> > 2 files changed, 16 insertions(+)
> > create mode 100755 tools/testing/selftests/net/netns-sysctl.sh
> >
> > diff --git a/tools/testing/selftests/net/Makefile b/tools/testing/selftests/net/Makefile
> > index bd01e4a0be2c..6da63d1831c1 100644
> > --- a/tools/testing/selftests/net/Makefile
> > +++ b/tools/testing/selftests/net/Makefile
> > @@ -53,6 +53,7 @@ TEST_PROGS += bind_bhash.sh
> > TEST_PROGS += ip_local_port_range.sh
> > TEST_PROGS += rps_default_mask.sh
> > TEST_PROGS += big_tcp.sh
> > +TEST_PROGS += netns-sysctl.sh
> > TEST_PROGS_EXTENDED := toeplitz_client.sh toeplitz.sh
> > TEST_GEN_FILES = socket nettest
> > TEST_GEN_FILES += psock_fanout psock_tpacket msg_zerocopy reuseport_addr_any
> > diff --git a/tools/testing/selftests/net/netns-sysctl.sh b/tools/testing/selftests/net/netns-sysctl.sh
> > new file mode 100755
> > index 000000000000..b948ba67b13a
> > --- /dev/null
> > +++ b/tools/testing/selftests/net/netns-sysctl.sh
> > @@ -0,0 +1,15 @@
> > +#!/bin/bash -e
>
> Don't you need to add the SPDX license header?
Yes, please!
Additionally, please handle explicitly the sysctl-related I/O errors so
that the script could output a human readable message in case of
failure.
Thanks!
Paolo
Powered by blists - more mailing lists