All of lore.kernel.org
 help / color / mirror / Atom feed
From: Adrien Mazarguil <adrien.mazarguil@6wind.com>
To: Ferruh Yigit <ferruh.yigit@intel.com>,
	Gaetan Rivet <gaetan.rivet@6wind.com>
Cc: dev@dpdk.org
Subject: [PATCH v1 0/7] Flow API helpers enhancements
Date: Thu,  5 Oct 2017 11:49:01 +0200	[thread overview]
Message-ID: <cover.1507193185.git.adrien.mazarguil@6wind.com> (raw)

This series brings enhancements to various rte_flow helpers:

- Allow applications to use rte_flow_error_set() by making it part of the
  public interface and documenting it as such.

- Address rte_flow_copy()'s limitations by replacing it with the more
  versatile rte_flow_conv(). This new function allows retrieving other
  properties such as item/action names, enabling testpmd to finally use it
  and get rid of duplicated code.

- Add a script (gen-rte_flow_conv-h.sh) to help with generating the
  resources used by rte_flow_conv(). Developers should run it when adding or
  modifying pattern items or actions (done as part of this series to add the
  missing "fuzzy" pattern item).

- Future plans for rte_flow_conv() include translating error codes to
  human-readable messages, so applications do not have to make their own.

All these changes address concerns raised a couple of months ago [1]. Work
on these patches actually started at the time but I was unable to complete
and clean them up until recently.

[1] http://dpdk.org/ml/archives/dev/2017-July/070492.html

Adrien Mazarguil (7):
  ethdev: expose flow API error helper
  ethdev: replace flow API object copy function
  ethdev: add flow API item/action name conversion
  app/testpmd: rely on flow API conversion function
  ethdev: enhance flow API item/action descriptions
  ethdev: generate flow API conversion header
  ethdev: update flow API conversion header

 MAINTAINERS                             |   1 +
 app/test-pmd/config.c                   | 293 ++++-----------
 app/test-pmd/testpmd.h                  |   7 +-
 buildtools/gen-rte_flow_conv-h.sh       | 264 +++++++++++++
 doc/guides/prog_guide/rte_flow.rst      |  43 ++-
 drivers/net/failsafe/failsafe_ether.c   |   6 +-
 drivers/net/failsafe/failsafe_flow.c    |  29 +-
 drivers/net/failsafe/failsafe_private.h |   4 +-
 drivers/net/mlx4/mlx4_flow.c            |   6 +-
 drivers/net/tap/tap_flow.c              |   2 +-
 lib/librte_ether/Makefile               |  10 +
 lib/librte_ether/rte_ethdev_version.map |   1 +
 lib/librte_ether/rte_flow.c             | 539 ++++++++++++++++-----------
 lib/librte_ether/rte_flow.h             | 275 ++++++++++++--
 lib/librte_ether/rte_flow_conv.h        | 345 +++++++++++++++++
 lib/librte_ether/rte_flow_driver.h      |  38 --
 16 files changed, 1320 insertions(+), 543 deletions(-)
 create mode 100755 buildtools/gen-rte_flow_conv-h.sh
 create mode 100644 lib/librte_ether/rte_flow_conv.h

-- 
2.1.4

             reply	other threads:[~2017-10-05  9:49 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-05  9:49 Adrien Mazarguil [this message]
2017-10-05  9:49 ` [PATCH v1 1/7] ethdev: expose flow API error helper Adrien Mazarguil
2017-10-11  9:23   ` Thomas Monjalon
2017-10-11 11:56     ` Adrien Mazarguil
2017-10-11 19:05       ` Aaron Conole
2017-10-12 13:37         ` Neil Horman
2017-10-12 14:02         ` Adrien Mazarguil
2017-10-05  9:49 ` [PATCH v1 2/7] ethdev: replace flow API object copy function Adrien Mazarguil
2017-10-05  9:49 ` [PATCH v1 3/7] ethdev: add flow API item/action name conversion Adrien Mazarguil
2017-10-05  9:49 ` [PATCH v1 4/7] app/testpmd: rely on flow API conversion function Adrien Mazarguil
2017-10-05  9:49 ` [PATCH v1 5/7] ethdev: enhance flow API item/action descriptions Adrien Mazarguil
2017-10-05  9:49 ` [PATCH v1 6/7] ethdev: generate flow API conversion header Adrien Mazarguil
2017-10-05  9:49 ` [PATCH v1 7/7] ethdev: update " Adrien Mazarguil
2017-10-06  1:13 ` [PATCH v1 0/7] Flow API helpers enhancements Ferruh Yigit
2017-10-06  8:05   ` Adrien Mazarguil
2017-10-10 18:05     ` Ferruh Yigit
2017-10-11  9:57       ` Adrien Mazarguil
2017-10-11 18:07         ` Ferruh Yigit
2017-10-12 12:53           ` Adrien Mazarguil
2017-10-12 16:37             ` Ferruh Yigit
2017-10-13 10:42               ` Adrien Mazarguil
2018-08-03 13:36 ` [PATCH v2 0/7] ethdev: add flow API object converter Adrien Mazarguil
2018-08-03 13:36   ` [PATCH v2 1/7] " Adrien Mazarguil
2018-08-03 13:36   ` [PATCH v2 2/7] ethdev: add flow API item/action name conversion Adrien Mazarguil
2018-08-03 13:36   ` [PATCH v2 3/7] app/testpmd: rely on flow API conversion function Adrien Mazarguil
2018-08-03 13:36   ` [PATCH v2 4/7] net/failsafe: switch to flow API object " Adrien Mazarguil
2018-08-03 13:36   ` [PATCH v2 5/7] net/bonding: " Adrien Mazarguil
2018-08-03 13:36   ` [PATCH v2 6/7] ethdev: deprecate rte_flow_copy function Adrien Mazarguil
2018-08-03 13:36   ` [PATCH v2 7/7] ethdev: add missing item/actions to flow object converter Adrien Mazarguil
2018-08-03 14:06   ` [PATCH v2 0/7] ethdev: add flow API " Thomas Monjalon
2018-08-23 13:48   ` Ferruh Yigit
2018-08-27 15:14     ` Adrien Mazarguil
2018-08-24 10:58   ` Ferruh Yigit
2018-08-27 14:12     ` Adrien Mazarguil
2018-08-31  9:00   ` [PATCH v3 " Adrien Mazarguil
2018-08-31  9:01     ` [PATCH v3 1/7] " Adrien Mazarguil
2018-08-31  9:01     ` [PATCH v3 2/7] ethdev: add flow API item/action name conversion Adrien Mazarguil
2018-08-31  9:01     ` [PATCH v3 3/7] app/testpmd: rely on flow API conversion function Adrien Mazarguil
2018-08-31  9:01     ` [PATCH v3 4/7] net/failsafe: switch to flow API object " Adrien Mazarguil
2018-08-31  9:01     ` [PATCH v3 5/7] net/bonding: " Adrien Mazarguil
2018-08-31  9:01     ` [PATCH v3 6/7] ethdev: add missing items/actions to flow object converter Adrien Mazarguil
2018-08-31  9:01     ` [PATCH v3 7/7] ethdev: deprecate rte_flow_copy function Adrien Mazarguil
2018-10-04 14:21       ` Ferruh Yigit
2018-08-31 11:32     ` [PATCH v3 0/7] ethdev: add flow API object converter Nélio Laranjeiro
2018-10-03 20:31       ` Thomas Monjalon
2018-10-04 14:25         ` Ferruh Yigit

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=cover.1507193185.git.adrien.mazarguil@6wind.com \
    --to=adrien.mazarguil@6wind.com \
    --cc=dev@dpdk.org \
    --cc=ferruh.yigit@intel.com \
    --cc=gaetan.rivet@6wind.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.