[ovs-dev] [PATCH v2 1/2] netdev-linux: Add coverage counters for netdev_set_policing when ingress tc-offload

Tonghao Zhang xiangxia.m.yue at gmail.com
Sun Apr 7 15:10:47 UTC 2019


When enable tc-offload, we should add coverage counters for netdev_set_policing.

Fixes: e7f6ba220e10 ("lib/tc: add ingress ratelimiting support for tc-offload")
Cc: Pieter Jansen van Vuuren <pieter.jansenvanvuuren at netronome.com>
Signed-off-by: Tonghao Zhang <xiangxia.m.yue at gmail.com>
---
 lib/netdev-linux.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/lib/netdev-linux.c b/lib/netdev-linux.c
index deedc69..0fce217 100644
--- a/lib/netdev-linux.c
+++ b/lib/netdev-linux.c
@@ -2474,6 +2474,8 @@ netdev_linux_set_policing(struct netdev *netdev_,
         goto out;
     }
 
+    COVERAGE_INC(netdev_set_policing);
+
     /* Use matchall for policing when offloadling ovs with tc-flower. */
     if (netdev_is_flow_api_enabled()) {
         error = tc_del_matchall_policer(netdev_);
@@ -2484,7 +2486,6 @@ netdev_linux_set_policing(struct netdev *netdev_,
         return error;
     }
 
-    COVERAGE_INC(netdev_set_policing);
     /* Remove any existing ingress qdisc. */
     error = tc_add_del_ingress_qdisc(ifindex, false, 0);
     if (error) {
-- 
1.8.3.1



More information about the dev mailing list