[ovs-git] [openvswitch/ovs] 7f5fe5: ofproto-dpif-xlate: Restore was_mpls after a patch...

GitHub noreply at github.com
Tue Mar 24 18:22:48 UTC 2015


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 7f5fe537de3357e36bbb5c385c0aab2fc2bf6a08
      https://github.com/openvswitch/ovs/commit/7f5fe537de3357e36bbb5c385c0aab2fc2bf6a08
  Author: Jarno Rajahalme <jrajahalme at nicira.com>
  Date:   2015-03-24 (Tue, 24 Mar 2015)

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

  Log Message:
  -----------
  ofproto-dpif-xlate: Restore was_mpls after a patch port.

The peer bridge popping MPLS should have no effect on the original
bridge.

Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>


  Commit: e93ef1c79617c67200835ee05f0605b5c949cc07
      https://github.com/openvswitch/ovs/commit/e93ef1c79617c67200835ee05f0605b5c949cc07
  Author: Jarno Rajahalme <jrajahalme at nicira.com>
  Date:   2015-03-24 (Tue, 24 Mar 2015)

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

  Log Message:
  -----------
  ofproto-dpif-xlate: Remove bond recirculation parameters from context.

The bond recirculation parameters in the translation context are short
lived and only carry parameters to immediate downstream functions.  As
such they are better served as normal function parameters.  This
simplifies the translation context, which needs to be essentially
restored after traversing patch ports.

Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>


  Commit: 5b09e56908a1b1e79d056e7be8e8a97783b1570d
      https://github.com/openvswitch/ovs/commit/5b09e56908a1b1e79d056e7be8e8a97783b1570d
  Author: Jarno Rajahalme <jrajahalme at nicira.com>
  Date:   2015-03-24 (Tue, 24 Mar 2015)

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

  Log Message:
  -----------
  ofproto-dpif-xlate: Roll back group bucket actions after every bucket.

We used to roll back group bucket changes only for 'all' and
'indirect' group types, but the expected semantics of all group types
is that any changes by the group bucket are not visible after the
group has been executed.

Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>


  Commit: 80e3509d72acecc44d4661f0da5f50a716e23e18
      https://github.com/openvswitch/ovs/commit/80e3509d72acecc44d4661f0da5f50a716e23e18
  Author: Jarno Rajahalme <jrajahalme at nicira.com>
  Date:   2015-03-24 (Tue, 24 Mar 2015)

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

  Log Message:
  -----------
  ofproto-dpif-xlate: More robust wildcarding for select group.

The flow key should be the same regardless of whether a live bucket is
found or not, as it would be confusing that the flow key would be
different (different mask bits) after the last group bucket goes dead.

In general, the megaflow algorithm expects the mask bits be set as
soon as we read the header bits, regardless of what happens
afterwards.

Also, use flow_mask_hash_fields() instead of individually setting mask
fields. This immediately brings in IPv6 support, and helps keeping
masks in sync with potential algorithm changes to flow hashing
functions.

Found by inspection.

Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>


Compare: https://github.com/openvswitch/ovs/compare/0c4b9393b6a5...80e3509d72ac


More information about the git mailing list