From: Adrien Mazarguil <adrien.mazarguil@6wind.com>
To: Ferruh Yigit <ferruh.yigit@intel.com>
Cc: dev@dpdk.org
Subject: [PATCH v3 0/7] ethdev: add flow API object converter
Date: Fri, 31 Aug 2018 11:00:57 +0200 [thread overview]
Message-ID: <20180831085337.21419-1-adrien.mazarguil@6wind.com> (raw)
In-Reply-To: <20180803132032.29038-1-adrien.mazarguil@6wind.com>
This is a follow up to the "Flow API helpers enhancements" series submitted
almost a year ago [1]. The new title is due to the reduced scope of this
version.
rte_flow_conv() is a flexible replacement to rte_flow_copy(), itself a
temporary solution pending something better [2]. It replaces a lot of
duplicated code found in testpmd and removes some of the maintenance burden
that developers tend to forget (me included) when modifying pattern
items or actions (updating app/test-pmd/config.c to be clear).
This series was unearthed in order to complete the implementation of
RTE_FLOW_ACTION_TYPE_ENCAP_(VXLAN|NVGRE) in testpmd [3] without having to
duplicate existing code once again.
See individual patches for specific changes in this version.
v3 changes:
- Marked rte_flow_conv() as experimental, modified net/bonding accordingly.
- Fixed compilation issue on ARM.
- Removed deprecation notice.
v2 changes:
- rte_flow_copy() is kept, albeit deprecated, no API/ABI impact.
- Updated bonding PMD.
- No more automatic generation of rte_flow_conv.h.
[1] https://mails.dpdk.org/archives/dev/2017-October/077551.html
[2] https://mails.dpdk.org/archives/dev/2017-July/070492.html
[3] Currently the command-line parser (cmdline_flow.c) is aware of these
actions, however config.c isn't. Flow rules with such actions cannot
be created and cannot be validated with PMDs that implement them.
Adrien Mazarguil (7):
ethdev: add flow API object converter
ethdev: add flow API item/action name conversion
app/testpmd: rely on flow API conversion function
net/failsafe: switch to flow API object conversion function
net/bonding: switch to flow API object conversion function
ethdev: add missing items/actions to flow object converter
ethdev: deprecate rte_flow_copy function
app/test-pmd/config.c | 407 +++------------
app/test-pmd/testpmd.h | 7 +-
doc/guides/prog_guide/rte_flow.rst | 20 +
doc/guides/rel_notes/deprecation.rst | 7 -
drivers/net/bonding/Makefile | 1 +
drivers/net/bonding/meson.build | 1 +
drivers/net/bonding/rte_eth_bond_api.c | 6 +-
drivers/net/bonding/rte_eth_bond_flow.c | 31 +-
drivers/net/bonding/rte_eth_bond_private.h | 5 +-
drivers/net/failsafe/failsafe_ether.c | 6 +-
drivers/net/failsafe/failsafe_flow.c | 31 +-
drivers/net/failsafe/failsafe_private.h | 5 +-
lib/librte_ethdev/rte_ethdev_version.map | 1 +
lib/librte_ethdev/rte_flow.c | 666 ++++++++++++++++++------
lib/librte_ethdev/rte_flow.h | 231 +++++++-
15 files changed, 886 insertions(+), 539 deletions(-)
--
2.11.0
next prev parent reply other threads:[~2018-08-31 9:01 UTC|newest]
Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-05 9:49 [PATCH v1 0/7] Flow API helpers enhancements Adrien Mazarguil
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 ` Adrien Mazarguil [this message]
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=20180831085337.21419-1-adrien.mazarguil@6wind.com \
--to=adrien.mazarguil@6wind.com \
--cc=dev@dpdk.org \
--cc=ferruh.yigit@intel.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.