[ovs-dev] [PATCH 2/2] Revert "bridge: Fix ovs-appctl qos/show repeated queue information"
Eelco Chaudron
echaudro at redhat.com
Wed Oct 3 07:01:49 UTC 2018
On 3 Oct 2018, at 1:03, Ben Pfaff wrote:
> This reverts commit 6b4d0211e84a ("bridge: Fix ovs-appctl qos/show
> repeated queue information"), which is no longer necessary now that
> commit 65f3c34c7417 ("netdev: Properly clear 'details' when iterating
> in NETDEV_QOS_FOR_EACH.") has been applied. The former commit fixed
> a symptom of the root cause fixed by the latter.
>
> Signed-off-by: Ben Pfaff <blp at ovn.org>
Looks good!
Acked-by: Eelco Chaudron <echaudro at redhat.com>
> ---
> vswitchd/bridge.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c
> index 875bb607c603..706a07cc6fb9 100644
> --- a/vswitchd/bridge.c
> +++ b/vswitchd/bridge.c
> @@ -3235,7 +3235,6 @@ 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);
>
> --
> 2.16.1
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
More information about the dev
mailing list