DPDK patches and discussions
 help / color / mirror / Atom feed
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>
Subject: [dpdk-dev] [PATCH 0/2] net/enic: a couple new features
Date: Tue, 10 Sep 2019 02:59:35 -0700	[thread overview]
Message-ID: <20190910095937.9731-1-hyonkim@cisco.com> (raw)

The first patch enables Geneve offload that supports Geneve header
options. Previous Geneve offload does not work when options are
present.

The second patch adds the new flow implementation based on Flow
Manager. So the driver now has two implementations: one based on the
old filter API, and this new one based on Flow Manager. Flow Manager
is newer firmware interface with more actions and match items than the
filter API. enic_fm_flow.c is quite large but self contained. We tried
to break it up into multiple patches, but it did not really make sense
in the end. So sending the whole file in one patch.

checkpatches complains about these errors. We checked and found it
safe to ignore them (false positives).

CHECK:CAMELCASE: Avoid CamelCase: <PRIx64>
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses

Thanks.
-Hyong

Hyong Youb Kim (2):
  net/enic: enable Geneve with options offload
  net/enic: add flow implementation based on Flow Manager API

 doc/guides/nics/enic.rst               |   21 +
 doc/guides/rel_notes/release_19_11.rst |    5 +
 drivers/net/enic/Makefile              |    1 +
 drivers/net/enic/base/vnic_dev.c       |   90 +-
 drivers/net/enic/base/vnic_dev.h       |    2 +
 drivers/net/enic/base/vnic_devcmd.h    |   19 +
 drivers/net/enic/base/vnic_flowman.h   |  364 ++++
 drivers/net/enic/enic.h                |   24 +-
 drivers/net/enic/enic_ethdev.c         |   25 +-
 drivers/net/enic/enic_fm_flow.c        | 2459 ++++++++++++++++++++++++
 drivers/net/enic/enic_main.c           |   30 +-
 drivers/net/enic/enic_res.c            |   13 +-
 drivers/net/enic/meson.build           |    1 +
 13 files changed, 3046 insertions(+), 8 deletions(-)
 create mode 100644 drivers/net/enic/base/vnic_flowman.h
 create mode 100644 drivers/net/enic/enic_fm_flow.c

-- 
2.22.0


             reply	other threads:[~2019-09-10 10:00 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-09-10  9:59 Hyong Youb Kim [this message]
2019-09-10  9:59 ` [dpdk-dev] [PATCH 1/2] net/enic: enable Geneve with options offload Hyong Youb Kim
2019-09-10  9:59 ` [dpdk-dev] [PATCH 2/2] net/enic: add flow implementation based on Flow Manager API Hyong Youb Kim

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190910095937.9731-1-hyonkim@cisco.com \
    --to=hyonkim@cisco.com \
    --cc=dev@dpdk.org \
    --cc=ferruh.yigit@intel.com \
    --cc=johndale@cisco.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).