[ovs-git] [openvswitch/ovs] 691d20: other-config: Add tc-policy switch to control tc f...

Thu Jun 15 10:11:16 UTC 2017

  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 691d20cbdcf3f56502ce7a43979a3c8c20203c25
      https://github.com/openvswitch/ovs/commit/691d20cbdcf3f56502ce7a43979a3c8c20203c25
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

  Changed paths:
    M lib/netdev.c
    M lib/tc.c
    M lib/tc.h
    M vswitchd/vswitch.xml

  Log Message:
  -----------
  other-config: Add tc-policy switch to control tc flower flag

Add a new configuration tc-policy option that controls tc
flower flag. Possible options are none, skip_sw, skip_hw.
The default is none which is to insert the rule both to sw and hw.
This option is only relevant if hw-offload is enabled.

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: 32b77c316d9982b42a0ab10cf183974de4b2f3ae
      https://github.com/openvswitch/ovs/commit/32b77c316d9982b42a0ab10cf183974de4b2f3ae
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

  Changed paths:
    M lib/dpif.c
    M lib/dpif.h
    M lib/netdev.c
    M lib/netdev.h

  Log Message:
  -----------
  dpif: Save added ports in a port map for netdev flow api use

To use netdev flow offloading api, dpifs needs to iterate over
added ports. This addition inserts the added dpif ports in a hash map,
The map will also be used to translate dpif ports to netdevs.

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: f7dde6df70c21f9846a00667b8d25e7bc75e0a34
      https://github.com/openvswitch/ovs/commit/f7dde6df70c21f9846a00667b8d25e7bc75e0a34
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

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

  Log Message:
  -----------
  dpif-netlink: Flush added ports using netdev flow api

If netdev flow offloading is enabled, flush all
added ports using netdev flow api.

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: 8140a5ff687489f6c35bf3b5c00d39db18c8ed73
      https://github.com/openvswitch/ovs/commit/8140a5ff687489f6c35bf3b5c00d39db18c8ed73
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

  Changed paths:
    M lib/netdev-tc-offloads.c

  Log Message:
  -----------
  netdev-tc-offloads: Implement netdev flow flush using tc interface

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: f2280b419860fdbbfed6c1bfb08994e0944a6672
      https://github.com/openvswitch/ovs/commit/f2280b419860fdbbfed6c1bfb08994e0944a6672
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

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

  Log Message:
  -----------
  dpif-netlink: Dump netdevs flows on flow dump

While dumping flows, dump flows that were offloaded to
netdev and parse them back to dpif flow.

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: 9116730dbb86c84e7128ed1c8af4a489a28b74da
      https://github.com/openvswitch/ovs/commit/9116730dbb86c84e7128ed1c8af4a489a28b74da
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

  Changed paths:
    M lib/netdev-tc-offloads.c

  Log Message:
  -----------
  netdev-tc-offloads: Add ufid to tc/netdev map

Flows offloaded to tc are identified by priority
and handle pair while OVS flows are identified by ufid.
Added a hash map to convert between the two for later
retrieval and deleting of offloaded flows.

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: 3def36a83252a52a502b518010fcb191029dc112
      https://github.com/openvswitch/ovs/commit/3def36a83252a52a502b518010fcb191029dc112
  Author: Roi Dayan <roid at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

  Changed paths:
    M include/openvswitch/match.h
    M lib/match.c

  Log Message:
  -----------
  match: Add helper function to set tunnel tp_dst

Add help function match_set_tun_tp_dst().
Will be used in the next commit.
This patch doesn't change any functionality.

Signed-off-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: 8f7620e6a406df7212400ba5d25ef789d1df33e6
      https://github.com/openvswitch/ovs/commit/8f7620e6a406df7212400ba5d25ef789d1df33e6
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

  Changed paths:
    M lib/netdev-tc-offloads.c

  Log Message:
  -----------
  netdev-tc-offloads: Implement netdev flow dump api using tc interface

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: 8b668ee3f0cc297783449619a34dbde275c9cc70
      https://github.com/openvswitch/ovs/commit/8b668ee3f0cc297783449619a34dbde275c9cc70
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

  Changed paths:
    M lib/dpif-netlink.c
    M lib/odp-util.c
    M lib/odp-util.h

  Log Message:
  -----------
  dpif-netlink: Use netdev flow put api to insert a flow

Using the new netdev flow api operate will now try and
offload flows to the relevant netdev of the input port.
Other operate methods flows will come in later patches.

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: d86eea7cd98b099448cde9d776d778b8684390f8
      https://github.com/openvswitch/ovs/commit/d86eea7cd98b099448cde9d776d778b8684390f8
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

  Changed paths:
    M lib/netdev-tc-offloads.c

  Log Message:
  -----------
  netdev-tc-offloads: Add flower mask to priority map

Flower classifer requires a different priority per mask,
so we hash the mask and generate a new priority for
each new mask used.

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: 8f283af892980cc5f96c2f878ae699f64de45e64
      https://github.com/openvswitch/ovs/commit/8f283af892980cc5f96c2f878ae699f64de45e64
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

  Changed paths:
    M lib/netdev-tc-offloads.c

  Log Message:
  -----------
  netdev-tc-offloads: Implement netdev flow put using tc interface

Currently only tunnel offload is supported.

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: 0335a89ced351eab3b9d0496652011b7ca3eb8a3
      https://github.com/openvswitch/ovs/commit/0335a89ced351eab3b9d0496652011b7ca3eb8a3
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

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

  Log Message:
  -----------
  dpif-netlink: Use netdev flow del api to delete a flow

If a flow was offloaded to a netdev we delete it using netdev
flow api.

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: 30b6b047260bb64281f9db802ec7a2a5196e088a
      https://github.com/openvswitch/ovs/commit/30b6b047260bb64281f9db802ec7a2a5196e088a
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

  Changed paths:
    M lib/netdev-tc-offloads.c

  Log Message:
  -----------
  netdev-tc-offloads: Implement netdev flow del using tc interface

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: 6c343984809c1e02c3e86991a71755705950d405
      https://github.com/openvswitch/ovs/commit/6c343984809c1e02c3e86991a71755705950d405
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

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

  Log Message:
  -----------
  dpif-netlink: Use netdev flow get api to query a flow

Search all datapath added netdevs for a given flow
using netdev flow api and parse it back to dpif flow.

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: 7ecdef27d4ad08575354a99519a4f40e7affe5a7
      https://github.com/openvswitch/ovs/commit/7ecdef27d4ad08575354a99519a4f40e7affe5a7
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

  Changed paths:
    M lib/netdev-tc-offloads.c

  Log Message:
  -----------
  netdev-tc-offloads: Implement flow get using tc interface

Search the requested ufid for a offloaded flow, and if found,
dump and parse it back to required format.

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Reviewed-by: Simon Horman <simon.horman at netronome.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: d5ae4a60261f7c7d64bf0690199ddac3faa09a84
      https://github.com/openvswitch/ovs/commit/d5ae4a60261f7c7d64bf0690199ddac3faa09a84
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

  Changed paths:
    M lib/netdev-linux.c

  Log Message:
  -----------
  netdev-linux: Disallow setting policing when configured with hw offload

Notify as not supported. Otherwise the ingress qdisc is being removed and
offload rules will be removed.

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: 01b257860c89680da0cce94d640c71a592327342
      https://github.com/openvswitch/ovs/commit/01b257860c89680da0cce94d640c71a592327342
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

  Changed paths:
    M lib/netdev-linux.c
    M lib/netdev-linux.h
    M lib/netdev-vport.c

  Log Message:
  -----------
  netdev-vport: Use common offloads interface

netdev vports are backed by actualy netdev at the kernel
level, so they can use the common netdev-tc offloads interface
for flow offloading (if enabled).

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: adbbe97f5e476cdb8993a50149efef2590e587dc
      https://github.com/openvswitch/ovs/commit/adbbe97f5e476cdb8993a50149efef2590e587dc
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

  Changed paths:
    M lib/netdev-tc-offloads.c

  Log Message:
  -----------
  netdev-tc-offloads: Add ingress on netdev flow api init

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: 9e9d7651a70a4edc788e0d10620a811a379b68c4
      https://github.com/openvswitch/ovs/commit/9e9d7651a70a4edc788e0d10620a811a379b68c4
  Author: Roi Dayan <roid at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

  Changed paths:
    M lib/dpctl.c

  Log Message:
  -----------
  dpctl: Add filter arg to dump-flows command info

This is for it to appear in bash completion.

Signed-off-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: 7e8b71993826a41198d1ffec5dbf10b375e52bd9
      https://github.com/openvswitch/ovs/commit/7e8b71993826a41198d1ffec5dbf10b375e52bd9
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

  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
    M ofproto/ofproto-dpif-upcall.c
    M ofproto/ofproto-dpif.c

  Log Message:
  -----------
  dpctl: Add an option to dump only certain kinds of flows

Usage:
    # to dump all datapath flows (default):
    ovs-dpctl dump-flows

    # to dump only flows that in kernel datapath:
    ovs-dpctl dump-flows type=ovs

    # to dump only flows that are offloaded:
    ovs-dpctl dump-flows type=offloaded

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: 4742003c74aca5f19a971169aa2b83246b9f96f0
      https://github.com/openvswitch/ovs/commit/4742003c74aca5f19a971169aa2b83246b9f96f0
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

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

  Log Message:
  -----------
  dpctl: Indicate if flow is offloaded when dumping flows of all types

When verbosity is requested on dump-flows (-m) indicate which flows
are offloaded.

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: fe5ca4a7668c336f7c78625b48b25223204c3b15
      https://github.com/openvswitch/ovs/commit/fe5ca4a7668c336f7c78625b48b25223204c3b15
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

  Changed paths:
    M tests/.gitignore
    M tests/automake.mk
    M tests/ofproto-macros.at
    A tests/system-offloads-testsuite.at
    A tests/system-offloads-traffic.at

  Log Message:
  -----------
  tests: Add system-offloads-testsuite

The new system-offloads-testsuite, which can be launched via
`make check-offloads`, tests offloading capabilities
to makes sure that certian flows are actually offloaded.

The tests run on virtual netdevices (VETH).

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: 1067cf93ac04306e26d560b4d4516bc9ceecbdfa
      https://github.com/openvswitch/ovs/commit/1067cf93ac04306e26d560b4d4516bc9ceecbdfa
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

  Changed paths:
    M lib/netdev.c

  Log Message:
  -----------
  netdev: Init flow api on already added ports on offload enable

Ports already added to a switch are not being initialized for offloading
so when enabling offload we need to go over those ports.

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: eff1e5b091c422bbf20ef709bdb65b14690d3d0a
      https://github.com/openvswitch/ovs/commit/eff1e5b091c422bbf20ef709bdb65b14690d3d0a
  Author: Roi Dayan <roid at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

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

  Log Message:
  -----------
  dpif: Refactor flow logging functions to be used by other modules

To be reused by other modules.

Signed-off-by: Roi Dayan <roid at mellanox.com>
Reviewed-by: Paul Blakey <paulb at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: 3cd99886191e1d35c29a110ba67b437481d5fe83
      https://github.com/openvswitch/ovs/commit/3cd99886191e1d35c29a110ba67b437481d5fe83
  Author: Roi Dayan <roid at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

  Changed paths:
    M lib/dpif-netlink.c

  Log Message:
  -----------
  dpif-netlink: Use dpif logging functions

Remove redundant logging functions and reuse
the exposed dpif logging functions.

Signed-off-by: Roi Dayan <roid at mellanox.com>
Reviewed-by: Paul Blakey <paulb at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

  Commit: bf090264e68d160d0ae70ebc93d59bc09d34cc8b
      https://github.com/openvswitch/ovs/commit/bf090264e68d160d0ae70ebc93d59bc09d34cc8b
  Author: Roi Dayan <roid at mellanox.com>
  Date:   2017-06-15 (Thu, 15 Jun 2017)

  Changed paths:
    M NEWS

  Log Message:
  -----------
  NEWS: add a note about hw offloading

Signed-off-by: Roi Dayan <roid at mellanox.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Simon Horman <simon.horman at netronome.com>

Compare: https://github.com/openvswitch/ovs/compare/31b88c97512b...bf090264e68d