tc does not include ethernet header length in packet byte count.
This fix will allow the packets that go trough tc to be 14 bytes less.
This difference in the TC implementation is already described in
tc-offload.rst.
Signed-off-by: Eelco Chaudron
Acked-by: Roi Dayan
---
tests/system-offloads.at |1 -
tests/system-traffic.at |2 +-
2 files changed, 1 insertion(+), 2 deletions(-)
diff --git a/tests/system-offloads.at b/tests/system-offloads.at
index a7076c2c2..cc45f662a 100644
--- a/tests/system-offloads.at
+++ b/tests/system-offloads.at
@@ -76,7 +76,6 @@ conntrack - multi-stage pipeline, local
conntrack - ICMP related with NAT
conntrack - DNAT load balancing
conntrack - DNAT load balancing with NC
-conntrack - Multiple ICMP traverse
conntrack - can match and clear ct_state from outside OVS
IGMP - flood under normal action"
echo "$ovs_test_skip_list" | sed "s// /g"])
diff --git a/tests/system-traffic.at b/tests/system-traffic.at
index 980b0bd70..07913a192 100644
--- a/tests/system-traffic.at
+++ b/tests/system-traffic.at
@@ -6929,7 +6929,7 @@ AT_CHECK([DPCTL_DUMP_CONNTRACK | FORMAT_CT(10.1.1)], [0],
[dnl
icmp,orig=(src=10.1.1.1,dst=10.1.1.2,id=,type=8,code=0),reply=(src=10.1.1.2,dst=10.1.1.1,id=,type=0,code=0)
])
-AT_CHECK([ovs-ofctl dump-flows br0 | grep table=2, |
OFPROTO_CLEAR_DURATION_IDLE],
+AT_CHECK([ovs-ofctl dump-flows br0 | grep table=2, |
OFPROTO_CLEAR_DURATION_IDLE | sed 's/n_bytes=70,/n_bytes=84,/'],
[0], [dnl
cookie=0x0, duration=, table=2, n_packets=2, n_bytes=84,
idle_age=, priority=10,ct_state=+new+trk,in_port=1 actions=drop
cookie=0x0, duration=, table=2, n_packets=0, n_bytes=0,
idle_age=, priority=10,ct_state=+est+trk actions=drop
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev