From 2c999774c1758ebdb138bb73e89e8de01f134d26 Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Thu, 16 Feb 2012 16:33:14 -0800 Subject: [PATCH] vswitchd: Always configure a default queue for QoS. When an interface has QoS configured but no default queue (queue 0), OVS has until now installed that QoS configuration literally, which causes all packets destined for this default queue to be dropped. This behavior is usually both unexpected and undesirable. This commit changes behavior so that, when no default queue is configured, OVS uses a default "empty" configuration for the default queue. This behavior should be more acceptable when QoS is slightly misconfigured. I tested that, without this patch, configuring only queue 1 causes "tc class show" to show only queue 1 (handle 1:2) for linux-htb and linux-hfsc, and that with this patch it shows configurations for both queue 0 (handle 1:1) and queue 1. Bug #5583. Feature #7413. Signed-off-by: Ben Pfaff --- NEWS | 3 +++ vswitchd/bridge.c | 7 +++++-- vswitchd/vswitch.xml | 13 +++++++++++-- 3 files changed, 19 insertions(+), 4 deletions(-) diff --git a/NEWS b/NEWS index b1761309..c16649ec 100644 --- a/NEWS +++ b/NEWS @@ -20,6 +20,9 @@ post-v1.5.0 - Added "fin_timeout" support to "learn" action. - The default MAC learning timeout has been increased from 60 seconds to 300 seconds. The MAC learning timeout is now configurable. + - When QoS settings for an interface do not configure a default queue + (queue 0), Open vSwitch now uses a default configuration for that + queue, instead of dropping all packets as in previous versions. - Logging to console and file will have UTC timestamp as a default for all the daemons. An example of the default format is 2012-01-27T16:35:17Z. ovs-appctl can be used to change the default format as before. diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c index f6cf1196..e857077a 100644 --- a/vswitchd/bridge.c +++ b/vswitchd/bridge.c @@ -3198,9 +3198,12 @@ iface_configure_qos(struct iface *iface, const struct ovsrec_qos *qos) if (!queue_zero) { static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(1, 1); VLOG_WARN_RL(&rl, "interface %s: QoS configured without a default " - "queue (queue 0). Packets not directed to a " - "correctly configured queue may be dropped.", + "queue (queue 0). Using default configuration.", iface->name); + + shash_init(&details); + netdev_set_queue(iface->netdev, 0, &details); + shash_destroy(&details); } } diff --git a/vswitchd/vswitch.xml b/vswitchd/vswitch.xml index 5941d229..cd50f556 100644 --- a/vswitchd/vswitch.xml +++ b/vswitchd/vswitch.xml @@ -2057,8 +2057,17 @@ supported range of queue numbers depend on . The queue numbers are the same as the queue_id used in OpenFlow in struct ofp_action_enqueue and other - structures. Queue 0 is used by OpenFlow output actions that do not - specify a specific queue.

+ structures.

+ +

+ Queue 0 is the ``default queue.'' It is used by OpenFlow output + actions when no specific queue has been set. Ordinarily should include a configuration for queue 0. When no + configuration for queue 0 is present, a default configuration is used. + (Before version 1.6, Open vSwitch would leave queue 0 unconfigured in + this case. With some queuing disciplines, this dropped all packets + destined for the default queue.) +

-- 2.30.2