* [PATCH net] selftests: drv-net: add configs for the TSO test
@ 2025-06-02 23:16 Jakub Kicinski
2025-06-03 11:17 ` Paolo Abeni
0 siblings, 1 reply; 3+ messages in thread
From: Jakub Kicinski @ 2025-06-02 23:16 UTC (permalink / raw)
To: davem
Cc: netdev, edumazet, pabeni, andrew+netdev, horms, Jakub Kicinski,
shuah, willemb, linux-kselftest
Add missing config options for the tso.py test, specifically
to make sure the kernel is built with vxlan and gre tunnels.
I noticed this while adding a TSO-capable device QEMU to the CI.
Previously we only run virtio tests and it doesn't report LSO
stats on the QEMU we have.
Fixes: 0d0f4174f6c8 ("selftests: drv-net: add a simple TSO test")
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
---
CC: shuah@kernel.org
CC: willemb@google.com
CC: linux-kselftest@vger.kernel.org
---
tools/testing/selftests/drivers/net/hw/config | 6 ++++++
1 file changed, 6 insertions(+)
create mode 100644 tools/testing/selftests/drivers/net/hw/config
diff --git a/tools/testing/selftests/drivers/net/hw/config b/tools/testing/selftests/drivers/net/hw/config
new file mode 100644
index 000000000000..ea4b70d71563
--- /dev/null
+++ b/tools/testing/selftests/drivers/net/hw/config
@@ -0,0 +1,6 @@
+CONFIG_IPV6=y
+CONFIG_IPV6_GRE=y
+CONFIG_NET_IP_TUNNEL=y
+CONFIG_NET_IPGRE=y
+CONFIG_NET_IPGRE_DEMUX=y
+CONFIG_VXLAN=y
--
2.49.0
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH net] selftests: drv-net: add configs for the TSO test
2025-06-02 23:16 [PATCH net] selftests: drv-net: add configs for the TSO test Jakub Kicinski
@ 2025-06-03 11:17 ` Paolo Abeni
2025-06-03 13:55 ` Jakub Kicinski
0 siblings, 1 reply; 3+ messages in thread
From: Paolo Abeni @ 2025-06-03 11:17 UTC (permalink / raw)
To: Jakub Kicinski, davem
Cc: netdev, edumazet, andrew+netdev, horms, shuah, willemb,
linux-kselftest
On 6/3/25 1:16 AM, Jakub Kicinski wrote:
> diff --git a/tools/testing/selftests/drivers/net/hw/config b/tools/testing/selftests/drivers/net/hw/config
> new file mode 100644
> index 000000000000..ea4b70d71563
> --- /dev/null
> +++ b/tools/testing/selftests/drivers/net/hw/config
> @@ -0,0 +1,6 @@
> +CONFIG_IPV6=y
> +CONFIG_IPV6_GRE=y
> +CONFIG_NET_IP_TUNNEL=y
I think the above it's not strictly needed, because it's selected by gre
and vxlan (indirectly), but it should not hurt, nor I think we have
minimal non redundant knobs list in other config files.
/P
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH net] selftests: drv-net: add configs for the TSO test
2025-06-03 11:17 ` Paolo Abeni
@ 2025-06-03 13:55 ` Jakub Kicinski
0 siblings, 0 replies; 3+ messages in thread
From: Jakub Kicinski @ 2025-06-03 13:55 UTC (permalink / raw)
To: Paolo Abeni
Cc: davem, netdev, edumazet, andrew+netdev, horms, shuah, willemb,
linux-kselftest
On Tue, 3 Jun 2025 13:17:25 +0200 Paolo Abeni wrote:
> On 6/3/25 1:16 AM, Jakub Kicinski wrote:
> > diff --git a/tools/testing/selftests/drivers/net/hw/config b/tools/testing/selftests/drivers/net/hw/config
> > new file mode 100644
> > index 000000000000..ea4b70d71563
> > --- /dev/null
> > +++ b/tools/testing/selftests/drivers/net/hw/config
> > @@ -0,0 +1,6 @@
> > +CONFIG_IPV6=y
> > +CONFIG_IPV6_GRE=y
> > +CONFIG_NET_IP_TUNNEL=y
>
> I think the above it's not strictly needed, because it's selected by gre
> and vxlan (indirectly), but it should not hurt, nor I think we have
> minimal non redundant knobs list in other config files.
Oops, you're right. I'll respin after
netlink: specs: rt-link: decode ip6gre
gets thru CI, this change alone fixed TSO but broke the stats test :S
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-06-03 13:55 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-06-02 23:16 [PATCH net] selftests: drv-net: add configs for the TSO test Jakub Kicinski
2025-06-03 11:17 ` Paolo Abeni
2025-06-03 13:55 ` Jakub Kicinski
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).