[ovs-git] [openvswitch/ovs] ad386c: ovn-northd: Only peer router ports to other router...

GitHub noreply at github.com
Fri Jul 22 18:16:44 UTC 2016


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: ad386c3f31fea7c0e2e2bc91b0cd370132208811
      https://github.com/openvswitch/ovs/commit/ad386c3f31fea7c0e2e2bc91b0cd370132208811
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2016-07-22 (Fri, 22 Jul 2016)

  Changed paths:
    M ovn/northd/ovn-northd.c

  Log Message:
  -----------
  ovn-northd: Only peer router ports to other router ports.

A router port's "peer", if set, must point to another router port, but the
code as written also accepted switch ports.  This caused problems when
switch ports were actually specified.

Reported-by: Gurucharan Shetty <guru at ovn.org>
Reported-at: http://openvswitch.org/pipermail/dev/2016-July/075524.html
Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Gurucharan Shetty <guru at ovn.org>


  Commit: cd1508992ae93140c175a948bae9eff17b129321
      https://github.com/openvswitch/ovs/commit/cd1508992ae93140c175a948bae9eff17b129321
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2016-07-22 (Fri, 22 Jul 2016)

  Changed paths:
    M ovn/northd/ovn-northd.c

  Log Message:
  -----------
  ovn-northd: Avoid redundant lookup of logical router port peer.

An ovn_port keeps track of its peer in its 'peer' member, but the code
updated by this commit instead did a redundant lookup of the peer.

Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Gurucharan Shetty <guru at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/64839cf43251...cd1508992ae9


More information about the git mailing list