netdev-linux: Remove counter double-increments.
authorBen Pfaff <blp@nicira.com>
Mon, 1 Nov 2010 17:47:29 +0000 (10:47 -0700)
committerBen Pfaff <blp@nicira.com>
Tue, 30 Nov 2010 18:30:30 +0000 (10:30 -0800)
A few coverage counters were incremented both in netdev generic code and
in netdev_linux code.  This commit drops the increments from the
lower-level code.

(This is not an actual bug because these counters are used only for
logging.)

lib/netdev-linux.c

index 39e21bc6749b27091f764670052cd26bbea179bd..d04eb715a0e58a714e064e8130a7bb0636aa018a 100644 (file)
@@ -1059,8 +1059,6 @@ netdev_linux_get_stats(const struct netdev *netdev_,
     static int use_netlink_stats = -1;
     int error;
 
-    COVERAGE_INC(netdev_get_stats);
-
     if (netdev_dev->have_vport_stats ||
         !(netdev_dev->cache_valid & VALID_HAVE_VPORT_STATS)) {
 
@@ -1875,7 +1873,6 @@ netdev_linux_add_router(struct netdev *netdev OVS_UNUSED, struct in_addr router)
     make_in4_sockaddr(&rt.rt_gateway, router);
     make_in4_sockaddr(&rt.rt_genmask, any);
     rt.rt_flags = RTF_UP | RTF_GATEWAY;
-    COVERAGE_INC(netdev_add_router);
     error = ioctl(af_inet_sock, SIOCADDRT, &rt) < 0 ? errno : 0;
     if (error) {
         VLOG_WARN("ioctl(SIOCADDRT): %s", strerror(error));