From: Ori Kam <orika@mellanox.com>
To: Andrew Rybchenko <arybchenko@solarflare.com>,
"wenzhuo.lu@intel.com" <wenzhuo.lu@intel.com>,
"jingjing.wu@intel.com" <jingjing.wu@intel.com>,
"bernard.iremonger@intel.com" <bernard.iremonger@intel.com>,
"ferruh.yigit@intel.com" <ferruh.yigit@intel.com>,
"stephen@networkplumber.org" <stephen@networkplumber.org>,
Adrien Mazarguil <adrien.mazarguil@6wind.com>
Cc: "dev@dpdk.org" <dev@dpdk.org>,
"Dekel Peled" <dekelp@mellanox.com>,
"Thomas Monjalon" <thomas@monjalon.net>,
"Nélio Laranjeiro" <nelio.laranjeiro@6wind.com>,
"Yongseok Koh" <yskoh@mellanox.com>,
"Shahaf Shuler" <shahafs@mellanox.com>
Subject: Re: [dpdk-dev] [PATCH v4 1/3] ethdev: add raw encapsulation action
Date: Mon, 22 Oct 2018 13:32:02 +0000 [thread overview]
Message-ID: <AM4PR05MB34251B45054872AECF037589DBF40@AM4PR05MB3425.eurprd05.prod.outlook.com> (raw)
In-Reply-To: <4c172bc9-2213-089e-0061-34028f888613@solarflare.com>
> -----Original Message-----
> From: Andrew Rybchenko <arybchenko@solarflare.com>
> Sent: Monday, October 22, 2018 4:28 PM
> To: Ori Kam <orika@mellanox.com>; wenzhuo.lu@intel.com;
> jingjing.wu@intel.com; bernard.iremonger@intel.com; ferruh.yigit@intel.com;
> stephen@networkplumber.org; Adrien Mazarguil
> <adrien.mazarguil@6wind.com>
> Cc: dev@dpdk.org; Dekel Peled <dekelp@mellanox.com>; Thomas Monjalon
> <thomas@monjalon.net>; Nélio Laranjeiro <nelio.laranjeiro@6wind.com>;
> Yongseok Koh <yskoh@mellanox.com>; Shahaf Shuler
> <shahafs@mellanox.com>
> Subject: Re: [dpdk-dev] [PATCH v4 1/3] ethdev: add raw encapsulation action
>
> On 10/22/18 4:19 PM, Ori Kam wrote:
> >> -----Original Message-----
> >> From: Andrew Rybchenko <arybchenko@solarflare.com>
> >> Sent: Monday, October 22, 2018 4:06 PM
> >> To: Ori Kam <orika@mellanox.com>; wenzhuo.lu@intel.com;
> >> jingjing.wu@intel.com; bernard.iremonger@intel.com;
> ferruh.yigit@intel.com;
> >> stephen@networkplumber.org; Adrien Mazarguil
> >> <adrien.mazarguil@6wind.com>
> >> Cc: dev@dpdk.org; Dekel Peled <dekelp@mellanox.com>; Thomas Monjalon
> >> <thomas@monjalon.net>; Nélio Laranjeiro <nelio.laranjeiro@6wind.com>;
> >> Yongseok Koh <yskoh@mellanox.com>; Shahaf Shuler
> >> <shahafs@mellanox.com>
> >> Subject: Re: [dpdk-dev] [PATCH v4 1/3] ethdev: add raw encapsulation action
> >>
> >> On 10/17/18 11:43 AM, Ori Kam wrote:
> >>>> -----Original Message-----
> >>>> From: Andrew Rybchenko <arybchenko@solarflare.com>
> >>>> Sent: Wednesday, October 17, 2018 10:56 AM
> >>>> To: Ori Kam <orika@mellanox.com>; wenzhuo.lu@intel.com;
> >>>> jingjing.wu@intel.com; bernard.iremonger@intel.com;
> >> ferruh.yigit@intel.com;
> >>>> stephen@networkplumber.org; Adrien Mazarguil
> >>>> <adrien.mazarguil@6wind.com>
> >>>> Cc: dev@dpdk.org; Dekel Peled <dekelp@mellanox.com>; Thomas
> Monjalon
> >>>> <thomas@monjalon.net>; Nélio Laranjeiro
> <nelio.laranjeiro@6wind.com>;
> >>>> Yongseok Koh <yskoh@mellanox.com>; Shahaf Shuler
> >>>> <shahafs@mellanox.com>
> >>>> Subject: Re: [dpdk-dev] [PATCH v4 1/3] ethdev: add raw encapsulation
> action
> >>>>
> >>>> On 10/17/18 12:41 AM, Ori Kam wrote:
> >>>> Currenlty the encap/decap actions only support encapsulation
> >>>> of VXLAN and NVGRE L2 packets (L2 encapsulation is where
> >>>> the inner packet has a valid Ethernet header, while L3 encapsulation
> >>>> is where the inner packet doesn't have the Ethernet header).
> >>>> In addtion the parameter to to the encap action is a list of rte items,
> >>>> this results in 2 extra translation, between the application to the
> >>>> actioni and from the action to the NIC. This results in negetive impact
> >>>> on the insertion performance.
> >>>>
> >>>> Looking forward there are going to be a need to support many more
> tunnel
> >>>> encapsulations. For example MPLSoGRE, MPLSoUDP.
> >>>> Adding the new encapsulation will result in duplication of code.
> >>>> For example the code for handling NVGRE and VXLAN are exactly the
> same,
> >>>> and each new tunnel will have the same exact structure.
> >>>>
> >>>> This patch introduce a raw encapsulation that can support L2 tunnel types
> >>>> and L3 tunnel types. In addtion the new
> >>>> encapsulations commands are using raw buffer inorder to save the
> >>>> converstion time, both for the application and the PMD.
> >>>>
> >>>> In order to encapsulate L3 tunnel type there is a need to use both
> >>>> actions in the same rule: The decap to remove the L2 of the original
> >>>> packet, and then encap command to encapsulate the packet with the
> >>>> tunnel.
> >>>> For decap L3 there is also a need to use both commands in the same flow
> >>>> first the decap command to remove the outer tunnel header and then
> encap
> >>>> to add the L2 header.
> >>>>
> >>>> Signed-off-by: Ori Kam mailto:orika@mellanox.com
> >> [...]
> >>
> >>>> +
> >>>> +This action modifies the payload of matched flows. The data supplied
> must
> >>>> +be a valid header, either holding layer 2 data in case of removing layer 2
> >>>> +before incapsulation of layer 3 tunnel (for example MPLSoGRE) or
> >> complete
> >>>> +tunnel definition starting from layer 2 and moving to the tunel item
> itself.
> >>>> +When applied to the original packet the resulting packet must be a
> >>>> +valid packet.
> >>>> +
> >>>> +.. _table_rte_flow_action_raw_decap:
> >>>> +
> >>>> +.. table:: RAW_DECAP
> >>>> +
> >>>> + +----------------+----------------------------------------+
> >>>> + | Field | Value |
> >>>> +
> +================+========================================+
> >>>> + | ``data`` | Decapsulation data |
> >>>>
> >>>> Sorry, I've missed the point why it is here. Is it used for matching?
> >>>> Why is the size insufficient?
> >>>>
> >>> No it is not used for matching this is only for the encapsulation data.
> >>> The data is for PMD that needs to validate that they can decapsulate
> >>> The packet, and on some PMD there might need the specify which headers
> >>> to remove and not just number of bytes.
> >> Sorry, but I still don't understand. How should PMD or HW use it?
> >> I guess the main problem here is that it is a generic action.
> >> If it is VXLAN_DECAP, it would not be a problem and neither
> >> size nor data would be required.
> >>
> > The data is buffer of the encap/decap headers, so the PMD can parse the this
> data
> > and check the validity and if the HW supports it.
> > Some NICs will not use this others can check if the tunnel request is valid.
> > For example let's assume that some tunnel encapsulation is supported on
> some FW version
> > and not supported in other version so the PMD can check the encapsulation
> data to see
> > what is the requested tunnel type and the FW capabilities to return success or
> fail.
>
> OK, I see. Could you improve the action description to make it clear.
> Right now the description says nothing about it.
>
Sure I will update it.
Ori
next prev parent reply other threads:[~2018-10-22 13:32 UTC|newest]
Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-16 16:53 [dpdk-dev] [PATCH 0/3] add generic L2/L3 tunnel encapsulation actions Ori Kam
2018-09-16 16:53 ` [dpdk-dev] [PATCH 1/3] ethdev: " Ori Kam
2018-09-16 16:53 ` [dpdk-dev] [PATCH 2/3] ethdev: convert testpmd encap commands to new API Ori Kam
2018-09-16 16:53 ` [dpdk-dev] [PATCH 3/3] ethdev: remove vxlan and nvgre encapsulation commands Ori Kam
2018-09-26 21:00 ` [dpdk-dev] [PATCH v2 0/3] ethdev: add generic L2/L3 tunnel encapsulation actions Ori Kam
2018-09-26 21:00 ` [dpdk-dev] [PATCH v2 1/3] " Ori Kam
2018-09-26 21:00 ` [dpdk-dev] [PATCH v2 2/3] app/testpmd: convert testpmd encap commands to new API Ori Kam
2018-09-26 21:00 ` [dpdk-dev] [PATCH v2 3/3] ethdev: remove vxlan and nvgre encapsulation commands Ori Kam
2018-10-05 12:59 ` Ferruh Yigit
2018-10-05 13:26 ` Awal, Mohammad Abdul
2018-10-05 13:27 ` Mohammad Abdul Awal
2018-10-03 20:38 ` [dpdk-dev] [PATCH v2 0/3] ethdev: add generic L2/L3 tunnel encapsulation actions Thomas Monjalon
2018-10-05 12:57 ` Ferruh Yigit
2018-10-05 14:00 ` Ori Kam
2018-10-07 12:57 ` [dpdk-dev] [PATCH v3 " Ori Kam
2018-10-07 12:57 ` [dpdk-dev] [PATCH v3 1/3] " Ori Kam
2018-10-07 12:57 ` [dpdk-dev] [PATCH v3 2/3] app/testpmd: convert testpmd encap commands to new API Ori Kam
2018-10-07 12:57 ` [dpdk-dev] [PATCH v3 3/3] ethdev: remove vxlan and nvgre encapsulation commands Ori Kam
2018-10-09 16:48 ` [dpdk-dev] [PATCH v3 0/3] ethdev: add generic L2/L3 tunnel encapsulation actions Ferruh Yigit
2018-10-10 6:45 ` Andrew Rybchenko
2018-10-10 9:00 ` Ori Kam
2018-10-10 9:30 ` Andrew Rybchenko
2018-10-10 9:38 ` Thomas Monjalon
2018-10-10 12:02 ` Adrien Mazarguil
2018-10-10 13:17 ` Ori Kam
2018-10-10 16:10 ` Adrien Mazarguil
2018-10-11 8:48 ` Ori Kam
2018-10-11 13:12 ` Adrien Mazarguil
2018-10-11 13:55 ` Ori Kam
2018-10-16 21:40 ` [dpdk-dev] [PATCH v4 0/3] ethdev: add generic raw " Ori Kam
2018-10-16 21:41 ` [dpdk-dev] [PATCH v4 1/3] ethdev: add raw encapsulation action Ori Kam
2018-10-17 7:56 ` Andrew Rybchenko
2018-10-17 8:43 ` Ori Kam
2018-10-22 13:06 ` Andrew Rybchenko
2018-10-22 13:19 ` Ori Kam
2018-10-22 13:27 ` Andrew Rybchenko
2018-10-22 13:32 ` Ori Kam [this message]
2018-10-16 21:41 ` [dpdk-dev] [PATCH v4 2/3] app/testpmd: add MPLSoUDP encapsulation Ori Kam
2018-10-16 21:41 ` [dpdk-dev] [PATCH v4 3/3] app/testpmd: add MPLSoGRE encapsulation Ori Kam
2018-10-17 17:07 ` [dpdk-dev] [PATCH v5 0/3] ethdev: add generic raw tunnel encapsulation actions Ori Kam
2018-10-17 17:07 ` [dpdk-dev] [PATCH v5 1/3] ethdev: add raw encapsulation action Ori Kam
2018-10-22 14:15 ` Andrew Rybchenko
2018-10-22 14:31 ` Ori Kam
2018-10-17 17:07 ` [dpdk-dev] [PATCH v5 2/3] app/testpmd: add MPLSoUDP encapsulation Ori Kam
2018-10-17 17:07 ` [dpdk-dev] [PATCH v5 3/3] app/testpmd: add MPLSoGRE encapsulation Ori Kam
2018-10-22 14:45 ` [dpdk-dev] [PATCH v5 0/3] ethdev: add generic raw tunnel encapsulation actions Ferruh Yigit
2018-10-22 17:38 ` [dpdk-dev] [PATCH v6 0/3] ethdev: add generic raw tunnel encapsulation Ori Kam
2018-10-22 17:38 ` [dpdk-dev] [PATCH v6 1/3] ethdev: add raw encapsulation action Ori Kam
2018-10-22 17:38 ` [dpdk-dev] [PATCH v6 2/3] app/testpmd: add MPLSoUDP encapsulation Ori Kam
2018-10-23 9:55 ` Ferruh Yigit
2018-10-22 17:38 ` [dpdk-dev] [PATCH v6 3/3] app/testpmd: add MPLSoGRE encapsulation Ori Kam
2018-10-23 9:56 ` Ferruh Yigit
2018-10-23 9:56 ` [dpdk-dev] [PATCH v6 0/3] ethdev: add generic raw tunnel encapsulation 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=AM4PR05MB34251B45054872AECF037589DBF40@AM4PR05MB3425.eurprd05.prod.outlook.com \
--to=orika@mellanox.com \
--cc=adrien.mazarguil@6wind.com \
--cc=arybchenko@solarflare.com \
--cc=bernard.iremonger@intel.com \
--cc=dekelp@mellanox.com \
--cc=dev@dpdk.org \
--cc=ferruh.yigit@intel.com \
--cc=jingjing.wu@intel.com \
--cc=nelio.laranjeiro@6wind.com \
--cc=shahafs@mellanox.com \
--cc=stephen@networkplumber.org \
--cc=thomas@monjalon.net \
--cc=wenzhuo.lu@intel.com \
--cc=yskoh@mellanox.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).