HW offload support for set actions

Bug #1735248 reported by Simon Horman
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openvswitch (Ubuntu)
Wishlist
Unassigned

Bug Description

Description:

As of v2.8 Open vSwitch may program flows into the kernel and hardware using TC-Flower. This is a mechanism to allow acceleration of OvS using hardware such as Netronome SmartNICs.

The scope of this bug is to add set-action support to TC-based offload of Open vSwitch.

Status:

Accepted upstream for inclusion in v2.9.

Upstream patches:

a7ce5b852911 tc: Send csum action only if we need to update csum
fbaf1bf9ef69 tc: Fix wrong struct variable order
518bbe9971e4 netdev-tc-offloads: Remove redundant loop handling ovs action set
e13bbbab3cf8 netdev-tc-offloads: Remove redundant brackets
d6118e628988 netdev-tc-offloads: Verify csum flags on dump from tc
408671c4655e netdev-tc-offloads: Fix accidental skipping of extended pedit keys
6e1a6bfd9977 netdev-tc-offloads: Fix travis compilation error

1ae83bb20677 netdev-tc-offloads: Add support for action set
8ada482bbe19 tc: Add header rewrite using tc pedit action
0aee3827559b odp-util: Expose ovs flow key attr len table for reuse
e5b1657e6f67 compat: Add act_pedit compatibility for old kernels

description: updated
description: updated
description: updated
Revision history for this message
Simon Horman (horms-netronome) wrote :
Revision history for this message
Simon Horman (horms-netronome) wrote :

This is a feature request for Bionic

Revision history for this message
James Page (james-page) wrote :

Simon; if these patches will be included in ovs 2.9.x we're all good for Ubuntu Bionic; 2.9.0 should be the 18.04 release version of ovs (based on currently plans for both Ubuntu and OVS).

I'd rather not pull them into Bionic as distro patches ontop of 2.8, but we could pull in a 2.9.x snapshot as soon as OVS create a branch.

Changed in openvswitch (Ubuntu):
status: New → Triaged
importance: Undecided → Wishlist
milestone: none → ubuntu-18.04
Revision history for this message
Simon Horman (horms-netronome) wrote :

Hi James,

I confirm that the patches have been accepted for inclusion in v2.9.x and if Bionic ships with v2.9.0 then I believe we will be in good shape.

Revision history for this message
James Page (james-page) wrote :

Uploaded with snapshot for branch-2.9 into Bionic

Changed in openvswitch (Ubuntu):
status: Triaged → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package openvswitch - 2.9.0~git20180205.5a39582ca-0ubuntu1

---------------
openvswitch (2.9.0~git20180205.5a39582ca-0ubuntu1) bionic; urgency=medium

  * Snapshot from branch-2.9 (LP: #1735248, LP: #1686406, LP: #1709272):
    - d/p/dpif-kernel-gre-mtu-workaround.patch,
      d/p/dpif-netlink-rtnl-Use-65000-instead-of-65535-as-tunnel-MTU.patch,
      d/p/ovs-netdev-dpdk-17.11-upgrade.patch: Drop, included upstream.
  * d/control: Switch BD's to python{3}-all-dev to enable json native
    extensions (LP: #1672522).
  * d/control,rules,openvswitch-doc.*: Add documentation (LP: #965283).
  * Misc tidy of packaging:
    - d/*: wrap-and-sort -bast.
    - d/control: Priority optional->extra.
    - d/openvswitch-doc.doc-base: Update Section.
    - d/copyright: Pick machine formatted version from Debian
      (thanks Thomas).
    - d/control: Add missing depends on lsb-release.

 -- James Page <email address hidden> Mon, 05 Feb 2018 17:12:07 +0000

Changed in openvswitch (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers