[ovs-git] [openvswitch/ovs] bb71c9: dpif: Refactor dpif_probe_feature()

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: bb71c96ef2395e29046df159c6b32c0f35014e91
      https://github.com/openvswitch/ovs/commit/bb71c96ef2395e29046df159c6b32c0f35014e91
  Author: Andy Zhou <azhou at ovn.org>
  Date:   2017-03-10 (Fri, 10 Mar 2017)

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

  Log Message:
  -----------
  dpif: Refactor dpif_probe_feature()

Allow actions to be part of the probe. No functional changes.
Future patch will make use this new API.

Signed-off-by: Andy Zhou <azhou at ovn.org>
Acked-by: Joe Stringer <joe at ovn.org>


  Commit: b52ac6592fd675b665f8a35acda664b377197ffe
      https://github.com/openvswitch/ovs/commit/b52ac6592fd675b665f8a35acda664b377197ffe
  Author: Andy Zhou <azhou at ovn.org>
  Date:   2017-03-10 (Fri, 10 Mar 2017)

  Changed paths:
    M include/openvswitch/flow.h
    M lib/dpif.c
    M ofproto/ofproto-dpif.c
    M ofproto/ofproto-dpif.h

  Log Message:
  -----------
  ofproto: Probe for sample nesting level.

Add logics to detect the max level of nesting allowed by the
sample action implemented in the datapath.

Future patch allows xlate code to generate different odp actions
based on this information.

Signed-off-by: Andy Zhou <azhou at ovn.org>
Acked-by: Joe Stringer <joe at ovn.org>


  Commit: 889ee380c87259aef06c44eb1fe47b6c289e4165
      https://github.com/openvswitch/ovs/commit/889ee380c87259aef06c44eb1fe47b6c289e4165
  Author: Andy Zhou <azhou at ovn.org>
  Date:   2017-03-10 (Fri, 10 Mar 2017)

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

  Log Message:
  -----------
  ofproto-dpif-xlate: Avoid using sample action when nesting level is low

When datapath sample action only allow a small number of nested actions
(i.e. less than 3), do not translate the OpenFlow's 'clone' action
into datapath 'sample' action, since such translation would cause
datapath to reject the flow, with 'EOVERFLOW', when OVS is used to
implement the OVN pipeline, or more generally, when deeper nested
clone are expected.

Reported-by: Numan Siddique <nusiddiq at redhat.com>
Reported-at: https://mail.openvswitch.org/pipermail/ovs-dev/2017-March/329586.html
Signed-off-by: Andy Zhou <azhou at ovn.org>
Tested-by: Numan Siddique <nusiddiq at redhat.com>
Acked-by: Joe Stringer <joe at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/489b23fe0e95...889ee380c872