[ovs-git] [openvswitch/ovs] 058d3a: ofproto-dpif-xlate: translate action_set in clone ...

GitHub noreply at github.com
Thu Mar 8 20:19:41 UTC 2018


  Branch: refs/heads/branch-2.9
  Home:   https://github.com/openvswitch/ovs
  Commit: 058d3a4afddd4db6bf052c71ff431a59fa5a0335
      https://github.com/openvswitch/ovs/commit/058d3a4afddd4db6bf052c71ff431a59fa5a0335
  Author: Eric Garver <e at erig.me>
  Date:   2018-03-08 (Thu, 08 Mar 2018)

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

  Log Message:
  -----------
  ofproto-dpif-xlate: translate action_set in clone action

A clone action saves the action_set prior to performing the clone, then
restores it afterwards. However when xlating the actions it neglects to
consider the action_set so any write_action() inside a clone() are
ignored. Unfortunately patch ports are internally implemented via
clone(). So a frame traversing to a second bridge via patch port will
never be affected by write_action() in the second bridge's flow table.

Lets make clone() aware of the action_set.

Signed-off-by: Eric Garver <e at erig.me>
Signed-off-by: Ben Pfaff <blp at ovn.org>


  Commit: 54817b7207dc1fba78d3150f36c801bb60c75bfc
      https://github.com/openvswitch/ovs/commit/54817b7207dc1fba78d3150f36c801bb60c75bfc
  Author: Eric Garver <e at erig.me>
  Date:   2018-03-08 (Thu, 08 Mar 2018)

  Changed paths:
    M tests/ofproto-dpif.at

  Log Message:
  -----------
  tests/ofproto-dpif: New test for action_set after traversing patch port

Signed-off-by: Eric Garver <e at erig.me>
Signed-off-by: Ben Pfaff <blp at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/eb3bf7a834fc...54817b7207dc


More information about the git mailing list