netdev-linux: Make netdev_set_policing coverage counter consistent with other counters.
authorPravin B Shelar <pshelar@nicira.com>
Mon, 5 Mar 2012 23:44:30 +0000 (15:44 -0800)
committerPravin B Shelar <pshelar@nicira.com>
Mon, 5 Mar 2012 23:44:30 +0000 (15:44 -0800)
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 <pshelar@nicira.com>
lib/netdev-linux.c

index 5c322ca..8a8f912 100644 (file)
@@ -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) {