selftests: net: add missing config for GENEVE

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

Posted by Matthias May 1 year, 10 months ago

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 <matthias.may@westermo.com>
---
 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

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

Posted by Jakub Kicinski 1 year, 10 months ago

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 <matthias.may@westermo.com>

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

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

Posted by Matthias May 1 year, 10 months ago

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 <matthias.may@westermo.com>
> 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