[ovs-git] [openvswitch/ovs] 0d6b40: dpif-netdev: Initialize dpif_flow attrs

GitHub noreply at github.com
Thu Sep 13 14:56:46 UTC 2018


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 0d6b401cf60d10809e4644105cf47c44b4bdc4ab
      https://github.com/openvswitch/ovs/commit/0d6b401cf60d10809e4644105cf47c44b4bdc4ab
  Author: Gavi Teitz <gavi at mellanox.com>
  Date:   2018-09-13 (Thu, 13 Sep 2018)

  Changed paths:
    M lib/dpif-netdev.c

  Log Message:
  -----------
  dpif-netdev: Initialize dpif_flow attrs

In a previous commit, the dpif_flow struct was expanded, with the
'offloaded' field being moved into a new struct which also includes a
field for the dp layer the flow is handled on. The initialization of
these fields was only done in dpif-netlink.

This completes that commit, by initializing the fields in dpif-netdev
as well. As the 'offloaded' field was previously ignored by
dpif-netdev, the attrs are initialized to the default values of
'false' for the offloaded state, and 'ovs' for the dp layer.

Fixes: d63ca5329ff9 ("dpctl: Properly reflect a rule's offloaded to HW state")
Signed-off-by: Gavi Teitz <gavi at mellanox.com>
Acked-by: Roi Dayan <roid at mellanox.com>
Signed-off-by: Simon Horman <simon.horman at netronome.com>


  Commit: a692410af0f71f89ef3b4c4b89b039d23cdfedb3
      https://github.com/openvswitch/ovs/commit/a692410af0f71f89ef3b4c4b89b039d23cdfedb3
  Author: Gavi Teitz <gavi at mellanox.com>
  Date:   2018-09-13 (Thu, 13 Sep 2018)

  Changed paths:
    M lib/dpctl.c
    M lib/dpctl.man
    M lib/dpif-netdev.c
    M lib/dpif-netlink.c
    M lib/dpif-provider.h
    M lib/dpif.c
    M lib/dpif.h

  Log Message:
  -----------
  dpctl: Expand the flow dump type filter

Added new types to the flow dump filter, and allowed multiple filter
types to be passed at once, as a comma separated list. The new types
added are:
 * tc - specifies flows handled by the tc dp
 * non-offloaded - specifies flows not offloaded to the HW
 * all - specifies flows of all types

The type list is now fully parsed by the dpctl, and a new struct was
added to dpif which enables dpctl to define which types of dumps to
provide, rather than passing the type string and having dpif parse it.

Signed-off-by: Gavi Teitz <gavi at mellanox.com>
Acked-by: Roi Dayan <roid at mellanox.com>
Signed-off-by: Simon Horman <simon.horman at netronome.com>


Compare: https://github.com/openvswitch/ovs/compare/205639c253e6...a692410af0f7
      **NOTE:** This service has been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/

      Functionality will be removed from GitHub.com on January 31st, 2019.


More information about the git mailing list