netdev-linux: Fix off-by-one error dumping queue stats.
authorBen Pfaff <blp@nicira.com>
Thu, 16 Sep 2010 22:38:10 +0000 (15:38 -0700)
committerBen Pfaff <blp@nicira.com>
Fri, 1 Oct 2010 17:40:00 +0000 (10:40 -0700)
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.

lib/netdev-linux.c

index e1a3c8c46d9c41111aa6f4819566cd32646bb2db..2ea411dcdab3656ad0c17a5fd70f98da86561766 100644 (file)
@@ -2659,7 +2659,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;
 }