[ovs-git] [openvswitch/ovs] a9f70f: system-traffic: Fix IPv6 frag vxlan check.

GitHub noreply at github.com
Fri Apr 22 21:54:46 UTC 2016


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: a9f70f3d99d68f2eefa3be6686cf773a4817ba53
      https://github.com/openvswitch/ovs/commit/a9f70f3d99d68f2eefa3be6686cf773a4817ba53
  Author: Joe Stringer <joe at ovn.org>
  Date:   2016-04-22 (Fri, 22 Apr 2016)

  Changed paths:
    M tests/system-traffic.at

  Log Message:
  -----------
  system-traffic: Fix IPv6 frag vxlan check.

This was missed before somehow, which would cause the test to fail
(rather than being skipped) if iproute2 didn't support setting the
vxlan dstport on the kernel tunnel device.

Signed-off-by: Joe Stringer <joe at ovn.org>
Acked-by: Daniele Di Proietto <diproiettod at vmware.com>


  Commit: 523256cc0123ea85aabba0b3bf902645363680c8
      https://github.com/openvswitch/ovs/commit/523256cc0123ea85aabba0b3bf902645363680c8
  Author: Joe Stringer <joe at ovn.org>
  Date:   2016-04-22 (Fri, 22 Apr 2016)

  Changed paths:
    M tests/system-common-macros.at
    M tests/system-traffic.at

  Log Message:
  -----------
  system-traffic: Add basic gre tunnel sanity test.

Signed-off-by: Joe Stringer <joe at ovn.org>
Acked-by: Daniele Di Proietto <diproiettod at vmware.com>


  Commit: 92b8af2c061eb6623b8a615fc9dbca5de092c167
      https://github.com/openvswitch/ovs/commit/92b8af2c061eb6623b8a615fc9dbca5de092c167
  Author: Joe Stringer <joe at ovn.org>
  Date:   2016-04-22 (Fri, 22 Apr 2016)

  Changed paths:
    M tests/system-common-macros.at
    M tests/system-traffic.at

  Log Message:
  -----------
  system-traffic: Add basic geneve tunnel sanity test.

Signed-off-by: Joe Stringer <joe at ovn.org>
Acked-by: Daniele Di Proietto <diproiettod at vmware.com>


  Commit: f2d105b5915f891c4cda62847ce9d0c31e2df5fe
      https://github.com/openvswitch/ovs/commit/f2d105b5915f891c4cda62847ce9d0c31e2df5fe
  Author: Joe Stringer <joe at ovn.org>
  Date:   2016-04-22 (Fri, 22 Apr 2016)

  Changed paths:
    M lib/util.h
    M ofproto/ofproto-dpif-xlate.c
    M tests/system-traffic.at

  Log Message:
  -----------
  ofproto-dpif-xlate: xlate ct_{mark, label} correctly.

When translating multiple ct actions in a row which include modification
of ct_mark or ct_labels, these fields could be incorrectly translated
into datapath actions, resulting in modification of these fields for
entries when the OpenFlow rules didn't actually specify the change.

For instance, the following OpenFlow actions:
ct(zone=1,commit,exec(set_field(1->ct_mark))),ct(zone=2,table=1),...

Would translate into the datapath actions:
ct(zone=1,commit,mark=1),ct(zone=2,mark=1),recirc(...),...

This commit fixes the issue by zeroing the wildcards for these fields
prior to performing nested actions translation (and restoring
afterwards). As such, these fields do not hold both the match and the
field modification values at the same time. As a result, the ct_mark and
ct_labels don't leak from one ct action to the next.

Fixes: 8e53fe8cf7a1 ("Add connection tracking mark support.")
Fixes: 9daf23484fb1 ("Add connection tracking label support.")
Signed-off-by: Joe Stringer <joe at ovn.org>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: 2a754f4a0a66c24fdc6f1997a789bcf1079aabb9
      https://github.com/openvswitch/ovs/commit/2a754f4a0a66c24fdc6f1997a789bcf1079aabb9
  Author: Joe Stringer <joe at ovn.org>
  Date:   2016-04-22 (Fri, 22 Apr 2016)

  Changed paths:
    M ofproto/ofproto-dpif-xlate.c

  Log Message:
  -----------
  ofproto-dpif-xlate: Tidy up ct_mark xlate code.

Make the ct_mark netlink serialization more consistent with the way that
ct_label is serialized.

Signed-off-by: Joe Stringer <joe at ovn.org>
Acked-by: Ben Pfaff <blp at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/455f38e00920...2a754f4a0a66


More information about the git mailing list