selftests: forwarding: mirror_lib: Tighten up VLAN capture
authorPetr Machata <petrm@mellanox.com>
Sun, 8 Jul 2018 17:58:55 +0000 (19:58 +0200)
committerDavid S. Miller <davem@davemloft.net>
Wed, 11 Jul 2018 05:58:10 +0000 (22:58 -0700)
commitdb560d1612f81c932df8d31e6f3bf7581c32ba91
tree0adfef0d3a32b31a311f44e21852561d1b007b8a
parent5025b99c96a509e46a299942eab8286bb6f0f50a
selftests: forwarding: mirror_lib: Tighten up VLAN capture

The function do_test_span_vlan_dir_ips() is used for testing whether
mirrored packets are VLAN-encapsulated. But since it only considers
VLAN encapsulation, it may end up matching unmirrored ARP traffic as
well. One consequence is a rare failure of mirror_gre_vlan_bridge_1q's
test_gretap_untagged_egress. Decreasing ping cadence in mirror_test()
makes the problem easily reproducible.

Therefore tighten up the match criterion to only count those 802.1q
packets where the next header is IP.

Signed-off-by: Petr Machata <petrm@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
tools/testing/selftests/net/forwarding/mirror_lib.sh