2024-01-30 10:42:44

by Matthias May

Subject: [PATCH net-next] selftests: net: add missing config for GENEVE

l2_tos_ttl_inherit.sh verifies the inheritance of tos and ttl
for GRETAP, VXLAN and GENEVE.
Before testing it checks if the required module is available
and if not skips the tests accordingly.
Currently only GRETAP and VXLAN are tested because the GENEVE
module is missing.

Signed-off-by: Matthias May <[email protected]>
---
tools/testing/selftests/net/config | 1 +
1 file changed, 1 insertion(+)

diff --git a/tools/testing/selftests/net/config b/tools/testing/selftests/net/config
index 19ff75051660..8d79c024bebf 100644
--- a/tools/testing/selftests/net/config
+++ b/tools/testing/selftests/net/config
@@ -76,6 +76,7 @@ CONFIG_CRYPTO_SM4_GENERIC=y
CONFIG_AMT=m
CONFIG_TUN=y
CONFIG_VXLAN=m
+CONFIG_GENEVE=m
CONFIG_IP_SCTP=m
CONFIG_NETFILTER_XT_MATCH_POLICY=m
CONFIG_CRYPTO_ARIA=y
--
2.39.2

2024-01-31 17:16:10

by Jakub Kicinski

Subject: Re: [PATCH net-next] selftests: net: add missing config for GENEVE

On Tue, 30 Jan 2024 10:12:18 +0000 Matthias May wrote:
> l2_tos_ttl_inherit.sh verifies the inheritance of tos and ttl
> for GRETAP, VXLAN and GENEVE.
> Before testing it checks if the required module is available
> and if not skips the tests accordingly.
> Currently only GRETAP and VXLAN are tested because the GENEVE
> module is missing.
>
> Signed-off-by: Matthias May <[email protected]>

I reshuffled the option to sort slightly more alphabetically,
added a fixes tag and applied to net, hopefully this will help
folks testing stable.

Thank you!
--
pw-bot: accept

2024-01-31 21:40:30

by Matthias May

Subject: Re: [PATCH net-next] selftests: net: add missing config for GENEVE

On 31/01/2024 17:51, Jakub Kicinski wrote:
> On Tue, 30 Jan 2024 10:12:18 +0000 Matthias May wrote:
>> l2_tos_ttl_inherit.sh verifies the inheritance of tos and ttl
>> for GRETAP, VXLAN and GENEVE.
>> Before testing it checks if the required module is available
>> and if not skips the tests accordingly.
>> Currently only GRETAP and VXLAN are tested because the GENEVE
>> module is missing.
>>
>> Signed-off-by: Matthias May <[email protected]>
> I reshuffled the option to sort slightly more alphabetically,
> added a fixes tag and applied to net, hopefully this will help
> folks testing stable.
>
> Thank you!
Thank you for the fixup.
I wasn't sure if it actually "fixes" something since it kind of never
worked.

BR
Matthias