Most of coverage counters in netdev-linux are counting actual system calls
rather than reads from cached data.
Following patch fixes it by incrementing it after cache check.

Signed-off-by: Pravin B Shelar <pshe...@nicira.com>
---
 lib/netdev-linux.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/lib/netdev-linux.c b/lib/netdev-linux.c
index 5c322ca..8a8f912 100644
--- a/lib/netdev-linux.c
+++ b/lib/netdev-linux.c
@@ -1589,7 +1589,6 @@ netdev_linux_set_policing(struct netdev *netdev,
     const char *netdev_name = netdev_get_name(netdev);
     int error;
 
-    COVERAGE_INC(netdev_set_policing);
 
     kbits_burst = (!kbits_rate ? 0       /* Force to 0 if no rate specified. */
                    : !kbits_burst ? 1000 /* Default to 1000 kbits if 0. */
@@ -1602,6 +1601,7 @@ netdev_linux_set_policing(struct netdev *netdev,
         return 0;
     }
 
+    COVERAGE_INC(netdev_set_policing);
     /* Remove any existing ingress qdisc. */
     error = tc_add_del_ingress_qdisc(netdev, false);
     if (error) {
-- 
1.7.1

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to