]> git.baikalelectronics.ru Git - kernel.git/commit
selftests: fib offload: use sensible tos values
authorGuillaume Nault <gnault@redhat.com>
Wed, 2 Feb 2022 18:30:28 +0000 (19:30 +0100)
committerJakub Kicinski <kuba@kernel.org>
Fri, 4 Feb 2022 03:11:21 +0000 (19:11 -0800)
commitfe31c1c8ab84d0e82d5e16c6f3f0a92fbcc01878
treefa5a133619a8a467fdd7c0483a3a5aaffcc1bb21
parentea685587464159d81c6cfe781099b95d5b0ccbdc
selftests: fib offload: use sensible tos values

Although both iproute2 and the kernel accept 1 and 2 as tos values for
new routes, those are invalid. These values only set ECN bits, which
are ignored during IPv4 fib lookups. Therefore, no packet can actually
match such routes. This selftest therefore only succeeds because it
doesn't verify that the new routes do actually work in practice (it
just checks if the routes are offloaded or not).

It makes more sense to use tos values that don't conflict with ECN.
This way, the selftest won't be affected if we later decide to warn or
even reject invalid tos configurations for new routes.

Signed-off-by: Guillaume Nault <gnault@redhat.com>
Reviewed-by: Ido Schimmel <idosch@nvidia.com>
Link: https://lore.kernel.org/r/5e43b343720360a1c0e4f5947d9e917b26f30fbf.1643826556.git.gnault@redhat.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
tools/testing/selftests/net/forwarding/fib_offload_lib.sh