From: Ben Pfaff <blp@nicira.com>
Date: Thu, 16 Sep 2010 22:38:10 +0000 (-0700)
Subject: netdev-linux: Fix off-by-one error dumping queue stats.
X-Git-Tag: v1.0.2~6
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=19f997675dd321ab6b04e0d48e643aac53fd302d;hp=59b380da47991502517e3a70b148deda7a2ecfca;p=sliver-openvswitch.git

netdev-linux: Fix off-by-one error dumping queue stats.

Linux kernel queue numbers are one greater than OpenFlow queue numbers, for
HTB anyhow.  The code to dump queues wasn't compensating for this, so this
commit fixes it up.
---

diff --git a/lib/netdev-linux.c b/lib/netdev-linux.c
index e6036bfc5..f2afc3520 100644
--- a/lib/netdev-linux.c
+++ b/lib/netdev-linux.c
@@ -2621,7 +2621,7 @@ htb_class_dump_stats(const struct netdev *netdev OVS_UNUSED,
     major = tc_get_major(handle);
     minor = tc_get_minor(handle);
     if (major == 1 && minor > 0 && minor <= HTB_N_QUEUES) {
-        (*cb)(tc_get_minor(handle), &stats, aux);
+        (*cb)(minor - 1, &stats, aux);
     }
     return 0;
 }