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/?p=sliver-openvswitch.git;a=commitdiff_plain;h=19f997675dd321ab6b04e0d48e643aac53fd302d

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;
 }