[ovs-git] [openvswitch/ovs] e82898: ovn-controller: Fix memory leak and bug in setup_q...

[GitHub]

  Branch: refs/heads/branch-2.7
  Home:   https://github.com/openvswitch/ovs
  Commit: e828989078db7142e89fd0d013ab04d0fe2469d9
      https://github.com/openvswitch/ovs/commit/e828989078db7142e89fd0d013ab04d0fe2469d9
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2017-02-01 (Wed, 01 Feb 2017)

  Changed paths:
    M ovn/controller/binding.c

  Log Message:
  -----------
  ovn-controller: Fix memory leak and bug in setup_qos().

The caller of netdev_get_qos() is responsible for freeing its 'details'
smap.

Also, the previous qdisc details, for some arbitrary previous type of QoS,
aren't relevant for changing the type of QoS.

Signed-off-by: Ben Pfaff <blp at ovn.org>


  Commit: f2cbc936f62cd4348325b743ab411dd3e5cad13e
      https://github.com/openvswitch/ovs/commit/f2cbc936f62cd4348325b743ab411dd3e5cad13e
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2017-02-01 (Wed, 01 Feb 2017)

  Changed paths:
    M ovn/controller/binding.c

  Log Message:
  -----------
  ovn-controller: Configure interface QoS only if it would actually be used.

Until now, ovn-controller has unconditionally configured linux-htb on
physical interfaces.  QoS is pretty much always trouble, but it's even more
trouble if we set it up for no good reason.  We received a bug report, in
particular, that doing this disrupts connectivity in Docker.

This commit attempts to make that less likely, by making ovn-controller
only configure a qdisc if QoS support has in turn been configured in OVN.
The same problems as before will recur if QoS support is actually
configured, but at least now there's some purpose, and possibly a symptom
that the user can better diagnose ("I turned on QoS and OVN stopped
working" is at least a cause-and-effect chain that makes some sense).

Reported-by: Ritesh Rekhi <ritesh.rekhi at nutanix.com>
Reported-by: Hexin Wang <hexin.wang at nutanix.com>
Reported-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2017-February/043564.html
Tested-by: Hexin Wang <hexin.wang at nutanix.com>
Tested-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2017-February/043575.html
Signed-off-by: Ben Pfaff <blp at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/bf2dee0e41f5...f2cbc936f62c