From: Ethan Jackson <ethan@nicira.com>
Date: Sat, 7 May 2011 00:02:02 +0000 (-0700)
Subject: bridge: Don't configure QoS without Queues.
X-Git-Tag: v1.1.1~2
X-Git-Url: http://git.onelab.eu/?p=sliver-openvswitch.git;a=commitdiff_plain;h=0f50458b0a3054566c4f5cd70e0bc92ca7dafe49

bridge: Don't configure QoS without Queues.

It doesn't make sense to create a QoS object without any queues.
Before this patch, OVS would configure the QoS object and as a
result drop all traffic going through the affected interface.  With
this patch, OVS will simply clear QoS configuration on the
interface.

Bug #5583.
---

diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c
index f304ac014..e43af3c90 100644
--- a/vswitchd/bridge.c
+++ b/vswitchd/bridge.c
@@ -4632,7 +4632,7 @@ iface_update_carrier(struct iface *iface)
 static void
 iface_update_qos(struct iface *iface, const struct ovsrec_qos *qos)
 {
-    if (!qos || qos->type[0] == '\0') {
+    if (!qos || qos->type[0] == '\0' || qos->n_queues < 1) {
         netdev_set_qos(iface->netdev, NULL, NULL);
     } else {
         struct iface_delete_queues_cbdata cbdata;