Acked-by: Mark Michelson <[email protected]> On 9/24/19 12:39 PM, Lorenzo Bianconi wrote:
OVN applies logical switch QoS settings to egress interfaces. It currently works by analyzing each br-int interface to see what the remote-ip is on it, and then adding qdiscs to the tunnel-egress-iface associated with this br-int interface.This doesn't work as well when working with VLAN networks. In VLAN networks, the interface on br-int associated with the localnet port on the logical switch will be a patch port. It's connected to another bridge, making it difficult to determine which interface (or interfaces) is the egress interface on that connected bridge. This series aims to mend this. On the bridge that is patched to br-int, interfaces can have the boolean external-ids:ovn-egress-iface set to true if this is an egress interface. This way, QoS can be applied to these interfaces the same as is applied to tunnel interfaces. Moreover rework qos code in order to add the capability to select automatically the queue_id used to identify the device queue Finally add the set_queue action to logical flows in Egress Port Security - L2 stage for localnet QoS capable ports in order to set the physical_interface qdisc id Changes since v1: - split build_lswitch_port_sec in build_lswitch_input_port_sec and build_lswitch_output_port_sec - removed unnecessary log messages Changes since RFC: - introduce build_lswitch_port_sec as a container for {in/out}_port_sec logical flow configuration - move ovn-egress-iface lookup in consider_localnet_port in order to avoid an unnecessary port_binding lookup Lorenzo Bianconi (3): Add egress QoS mapping for non-tunnel interfaces northd: add the possibility to define localnet as qos capable port northd: interoduce logical flow for localnet egress shaping controller/binding.c | 51 ++++++- controller/binding.h | 4 + controller/ovn-controller.c | 3 +- controller/patch.c | 76 +++++----- controller/patch.h | 4 + northd/ovn-northd.8.xml | 7 +- northd/ovn-northd.c | 276 ++++++++++++++++++++++-------------- 7 files changed, 276 insertions(+), 145 deletions(-)
_______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
