diff options
author | Eelco Chaudron <echaudro@redhat.com> | 2018-10-02 08:51:05 -0400 |
---|---|---|
committer | Ben Pfaff <blp@ovn.org> | 2018-10-02 15:29:52 -0700 |
commit | 801491e0aff839453a4a4c51debb1ec48c4d2d4a (patch) | |
tree | 66ded7e844f7759ea2f2ac46a26ac5c0927b1b87 | |
parent | 1a727023086e862069f654269a53e27f4d4a6ed6 (diff) | |
download | openvswitch-801491e0aff839453a4a4c51debb1ec48c4d2d4a.tar.gz |
bridge: Fix ovs-appctl qos/show repeated queue information
The patch below would stop qos/show to repeat information from the previous queues.
See below an example before and after the fix:
Before:
$ ovs-appctl qos/show p5p2
QoS: p5p2 linux-htb
max-rate: 2428800
Default:
burst: 12512
min-rate: 12000
max-rate: 2428800
tx_packets: 0
tx_bytes: 0
tx_errors: 0
Queue 20:
burst: 12512
burst: 12512
min-rate: 12000
min-rate: 12000
max-rate: 607200
max-rate: 2428800
tx_packets: 28780
tx_bytes: 43572920
tx_errors: 17611
Queue 10:
burst: 12512
burst: 12512
burst: 12512
max-rate: 2428800
max-rate: 607200
max-rate: 2428800
min-rate: 12000
min-rate: 12000
min-rate: 12000
tx_packets: 71751
tx_bytes: 108631014
tx_errors: 18503
After:
$ ovs-appctl qos/show p5p2
QoS: p5p2 linux-htb
max-rate: 2428800
Default:
burst: 12512
min-rate: 12000
max-rate: 2428800
tx_packets: 0
tx_bytes: 0
tx_errors: 0
Queue 20:
burst: 12512
min-rate: 12000
max-rate: 607200
tx_packets: 28780
tx_bytes: 43572920
tx_errors: 17611
Queue 10:
burst: 12512
min-rate: 12000
max-rate: 2428800
tx_packets: 71751
tx_bytes: 108631014
tx_errors: 18503
Signed-off-by: Eelco Chaudron <echaudro@redhat.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
-rw-r--r-- | vswitchd/bridge.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c index 18c16fb83..aef38b543 100644 --- a/vswitchd/bridge.c +++ b/vswitchd/bridge.c @@ -3174,6 +3174,7 @@ qos_unixctl_show(struct unixctl_conn *conn, int argc OVS_UNUSED, smap_init(&details); NETDEV_QUEUE_FOR_EACH (&queue_id, &details, &dump, iface->netdev) { qos_unixctl_show_queue(queue_id, &details, iface, &ds); + smap_clear(&details); } smap_destroy(&details); |