From 0f50458b0a3054566c4f5cd70e0bc92ca7dafe49 Mon Sep 17 00:00:00 2001 From: Ethan Jackson Date: Fri, 6 May 2011 17:02:02 -0700 Subject: [PATCH] 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. --- vswitchd/bridge.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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; -- 2.43.0