Message ID | 20190228070317.17002-1-hyonkim@cisco.com (mailing list archive) |
---|---|
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 03D0837B4; Thu, 28 Feb 2019 08:03:44 +0100 (CET) Received: from rcdn-iport-3.cisco.com (rcdn-iport-3.cisco.com [173.37.86.74]) by dpdk.org (Postfix) with ESMTP id 5C1C83798 for <dev@dpdk.org>; Thu, 28 Feb 2019 08:03:42 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=2156; q=dns/txt; s=iport; t=1551337422; x=1552547022; h=from:to:cc:subject:date:message-id; bh=D80F6qmqi4gr7tTwDY7tCC4+g/KAVFbHoRfZ3Fq6abs=; b=EaOh2OQ0CwruK/Vo4tZo9+WnTzenaVC6yl1vX1ySI8Z/ZxnJU0f2yb5h VtsQq4wUcOQGbExoGfNpK/U6ojKGJwOxaoXXt+vH6kyvB3aaxptwLlq+9 MgkSbbIoa/cdhhyGFRFs3OGl19mTzvEgQil4cX1+hql57tN4b6+PjZ1IV Q=; X-IronPort-AV: E=Sophos;i="5.58,422,1544486400"; d="scan'208";a="515535662" Received: from alln-core-11.cisco.com ([173.36.13.133]) by rcdn-iport-3.cisco.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 28 Feb 2019 07:03:40 +0000 Received: from cisco.com (savbu-usnic-a.cisco.com [10.193.184.48]) by alln-core-11.cisco.com (8.15.2/8.15.2) with ESMTP id x1S73e0R028926; Thu, 28 Feb 2019 07:03:40 GMT Received: by cisco.com (Postfix, from userid 508933) id 1745420F2001; Wed, 27 Feb 2019 23:03:40 -0800 (PST) From: Hyong Youb Kim <hyonkim@cisco.com> To: Ferruh Yigit <ferruh.yigit@intel.com> Cc: dev@dpdk.org, John Daley <johndale@cisco.com>, Hyong Youb Kim <hyonkim@cisco.com> Date: Wed, 27 Feb 2019 23:03:02 -0800 Message-Id: <20190228070317.17002-1-hyonkim@cisco.com> X-Mailer: git-send-email 2.16.2 X-Outbound-SMTP-Client: 10.193.184.48, savbu-usnic-a.cisco.com X-Outbound-Node: alln-core-11.cisco.com Subject: [dpdk-dev] [PATCH 00/15] net/enic: 19.05 updates X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions <dev.dpdk.org> List-Unsubscribe: <https://mails.dpdk.org/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://mails.dpdk.org/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <https://mails.dpdk.org/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Series |
net/enic: 19.05 updates
|
|
Message
Hyong Youb Kim (hyonkim)
Feb. 28, 2019, 7:03 a.m. UTC
This patch series fixes bugs in enic's implementation of flow API and adds very limited support for RAW, RSS, and PASSTHRU. Limited RSS and PASSTHRU are intended to support partial offloads in OVS-DPDK and VPP. These apps use MARK + default RSS and PASSTHRU + MARK to "mark packet and then receive normally". Cisco VIC can support these, even though general RSS and PASSTHRU are not possible. Intentionally removed Cc: stable from the last patch ("net/enic: fix several issues with inner packet matching") as it depends on a non-fix patch ("net/enic: move arguments into struct"). I will submit backport request for these separately, after rc1. Hyong Youb Kim (15): net/enic: remove unused code net/enic: fix flow director SCTP matching net/enic: fix SCTP match for flow API net/enic: allow flow mark ID 0 net/enic: check for unsupported flow item types net/enic: enable limited RSS flow action net/enic: enable limited PASSTHRU flow action net/enic: move arguments into struct net/enic: enable limited support for RAW flow item net/enic: initialize VXLAN port regardless of overlay offload net/enic: fix a couple issues with VXLAN match net/enic: fix an endian bug in VLAN match net/enic: fix several issues with inner packet matching doc: update enic guide doc: update release notes for enic doc/guides/nics/enic.rst | 16 +- doc/guides/rel_notes/release_19_05.rst | 5 + drivers/net/enic/Makefile | 1 - drivers/net/enic/base/vnic_rss.c | 23 - drivers/net/enic/base/vnic_rss.h | 5 - drivers/net/enic/enic.h | 8 +- drivers/net/enic/enic_clsf.c | 38 +- drivers/net/enic/enic_flow.c | 786 +++++++++++++++++++++------------ drivers/net/enic/enic_main.c | 9 +- drivers/net/enic/enic_res.c | 26 -- drivers/net/enic/enic_res.h | 2 - drivers/net/enic/enic_rxtx_common.h | 3 +- drivers/net/enic/meson.build | 1 - 13 files changed, 557 insertions(+), 366 deletions(-) delete mode 100644 drivers/net/enic/base/vnic_rss.c
Comments
Hi, > Hyong Youb Kim (15): > net/enic: remove unused code > net/enic: fix flow director SCTP matching > net/enic: fix SCTP match for flow API > net/enic: allow flow mark ID 0 > net/enic: check for unsupported flow item types > net/enic: enable limited RSS flow action > net/enic: enable limited PASSTHRU flow action > net/enic: move arguments into struct > net/enic: enable limited support for RAW flow item > net/enic: initialize VXLAN port regardless of overlay offload > net/enic: fix a couple issues with VXLAN match > net/enic: fix an endian bug in VLAN match > net/enic: fix several issues with inner packet matching > doc: update enic guide > doc: update release notes for enic It is better to update the doc in the patch of the code change. It would allow to find the code changes by looking at the doc history (with "git blame" or others). The other benefit is to avoid some "noise" in the git history for doc. Thanks
On Fri, Mar 01, 2019 at 03:26:24PM +0100, Thomas Monjalon wrote: > Hi, > > > Hyong Youb Kim (15): > > net/enic: remove unused code > > net/enic: fix flow director SCTP matching > > net/enic: fix SCTP match for flow API > > net/enic: allow flow mark ID 0 > > net/enic: check for unsupported flow item types > > net/enic: enable limited RSS flow action > > net/enic: enable limited PASSTHRU flow action > > net/enic: move arguments into struct > > net/enic: enable limited support for RAW flow item > > net/enic: initialize VXLAN port regardless of overlay offload > > net/enic: fix a couple issues with VXLAN match > > net/enic: fix an endian bug in VLAN match > > net/enic: fix several issues with inner packet matching > > doc: update enic guide > > doc: update release notes for enic > > It is better to update the doc in the patch of the code change. > It would allow to find the code changes by looking at the doc history > (with "git blame" or others). > The other benefit is to avoid some "noise" in the git history for doc. I will send a v2. Thanks. -Hyong