From: "Wang, Jie1X" <jie1x.wang@intel.com>
To: Ori Kam <orika@nvidia.com>, "dev@dpdk.org" <dev@dpdk.org>
Cc: "Yang, SteveX" <stevex.yang@intel.com>,
"Singh, Aman Deep" <aman.deep.singh@intel.com>,
"Yigit, Ferruh" <ferruh.yigit@intel.com>,
"NBU-Contact-Thomas Monjalon (EXTERNAL)" <thomas@monjalon.net>,
"andrew.rybchenko@oktetlabs.ru" <andrew.rybchenko@oktetlabs.ru>,
"Wu, Jingjing" <jingjing.wu@intel.com>,
"Xing, Beilei" <beilei.xing@intel.com>,
"Zhang, Qi Z" <qi.z.zhang@intel.com>
Subject: RE: [PATCH 1/6] ethdev: add L2TPv2 RSS offload type
Date: Tue, 25 Jan 2022 01:54:36 +0000 [thread overview]
Message-ID: <DM8PR11MB56392BD1DC39F91390A0C25CD15F9@DM8PR11MB5639.namprd11.prod.outlook.com> (raw)
In-Reply-To: <MW2PR12MB46666D86A654AE69D279967AD65E9@MW2PR12MB4666.namprd12.prod.outlook.com>
Hi Ori,
> -----Original Message-----
> From: Ori Kam <orika@nvidia.com>
> Sent: Tuesday, January 25, 2022 12:35 AM
> To: Wang, Jie1X <jie1x.wang@intel.com>; dev@dpdk.org
> Cc: Yang, SteveX <stevex.yang@intel.com>; Singh, Aman Deep
> <aman.deep.singh@intel.com>; Yigit, Ferruh <ferruh.yigit@intel.com>; NBU-
> Contact-Thomas Monjalon (EXTERNAL) <thomas@monjalon.net>;
> andrew.rybchenko@oktetlabs.ru; Wu, Jingjing <jingjing.wu@intel.com>; Xing,
> Beilei <beilei.xing@intel.com>; Zhang, Qi Z <qi.z.zhang@intel.com>
> Subject: RE: [PATCH 1/6] ethdev: add L2TPv2 RSS offload type
>
> Hi Jie
>
> > -----Original Message-----
> > From: Jie Wang <jie1x.wang@intel.com>
> > Sent: Monday, January 24, 2022 8:56 AM
> > Subject: [PATCH 1/6] ethdev: add L2TPv2 RSS offload type
> >
> > This patch defines new RSS offload type for L2TPv2, which are required
> > when users want to distribute packets based on the L2TPv2 field.
> >
> > Signed-off-by: Jie Wang <jie1x.wang@intel.com>
> > ---
> > app/test-pmd/cmdline.c | 10 ++++++----
> > app/test-pmd/config.c | 3 ++-
> > doc/guides/rel_notes/release_22_03.rst | 5 +++++
> > lib/ethdev/rte_ethdev.h | 3 +++
> > 4 files changed, 16 insertions(+), 5 deletions(-)
> >
> > diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c index
> > 6e10afeedd..3914fb9a75 100644
> > --- a/app/test-pmd/cmdline.c
> > +++ b/app/test-pmd/cmdline.c
> > @@ -2178,7 +2178,7 @@ cmd_config_rss_parsed(void *parsed_result,
> > RTE_ETH_RSS_TCP | RTE_ETH_RSS_UDP |
> RTE_ETH_RSS_SCTP |
> > RTE_ETH_RSS_L2_PAYLOAD | RTE_ETH_RSS_L2TPV3 |
> RTE_ETH_RSS_ESP |
> > RTE_ETH_RSS_AH | RTE_ETH_RSS_PFCP |
> RTE_ETH_RSS_GTPU |
> > - RTE_ETH_RSS_ECPRI;
> > + RTE_ETH_RSS_ECPRI | RTE_ETH_RSS_L2TPV2;
> > else if (!strcmp(res->value, "eth"))
> > rss_conf.rss_hf = RTE_ETH_RSS_ETH;
> > else if (!strcmp(res->value, "vlan")) @@ -2256,6 +2256,8 @@
> > cmd_config_rss_parsed(void *parsed_result,
> > rss_conf.rss_hf = (rss_hf | RTE_ETH_RSS_LEVEL_INNERMOST);
> > } else if (!strcmp(res->value, "default"))
> > use_default = 1;
> > + else if (!strcmp(res->value, "l2tpv2"))
> > + rss_conf.rss_hf = RTE_ETH_RSS_L2TPV2;
> > else if (isdigit(res->value[0]) && atoi(res->value) > 0 &&
> > atoi(res->value) < 64)
> > rss_conf.rss_hf = 1ULL << atoi(res->value); @@ -2314,7
> +2316,7 @@
> > cmdline_parse_inst_t cmd_config_rss = {
> > .help_str = "port config all rss "
> >
> "all|default|eth|vlan|ip|tcp|udp|sctp|ether|port|vxlan|geneve|"
> > "nvgre|vxlan-gpe|l2tpv3|esp|ah|pfcp|ecpri|mpls|none|level-
> default|"
> > - "level-outer|level-inner|ipv4-chksum|<flowtype_id>",
> > + "level-outer|level-inner|ipv4-chksum|l2tpv2|<flowtype_id>",
> > .tokens = {
> > (void *)&cmd_config_rss_port,
> > (void *)&cmd_config_rss_keyword,
> > @@ -2429,7 +2431,7 @@ cmdline_parse_token_string_t
> cmd_config_rss_hash_key_rss_type =
> > "ipv6-tcp-ex#ipv6-udp-ex#"
> > "l3-src-only#l3-dst-only#l4-src-only#l4-dst-
> only#"
> > "l2-src-only#l2-dst-only#s-vlan#c-vlan#"
> > -
> "l2tpv3#esp#ah#pfcp#pppoe#gtpu#ecpri#mpls");
> > +
> "l2tpv3#esp#ah#pfcp#pppoe#gtpu#ecpri#mpls#l2tpv2");
> > cmdline_parse_token_string_t cmd_config_rss_hash_key_value =
> > TOKEN_STRING_INITIALIZER(struct cmd_config_rss_hash_key, key,
> NULL);
> >
> > @@ -2442,7 +2444,7 @@ cmdline_parse_inst_t cmd_config_rss_hash_key = {
> > "l2-payload|ipv6-ex|ipv6-tcp-ex|ipv6-udp-ex|"
> > "l3-src-only|l3-dst-only|l4-src-only|l4-dst-only|"
> > "l2-src-only|l2-dst-only|s-vlan|c-vlan|"
> > - "l2tpv3|esp|ah|pfcp|pppoe|gtpu|ecpri|mpls "
> > + "l2tpv3|esp|ah|pfcp|pppoe|gtpu|ecpri|mpls|l2tpv2 "
> > "<string of hex digits (variable length, NIC dependent)>",
> > .tokens = {
> > (void *)&cmd_config_rss_hash_key_port, diff --git
> > a/app/test-pmd/config.c b/app/test-pmd/config.c index
> > 1722d6c8f8..ec922bd304 100644
> > --- a/app/test-pmd/config.c
> > +++ b/app/test-pmd/config.c
> > @@ -90,7 +90,7 @@ const struct rss_type_info rss_type_table[] = {
> > { "all", RTE_ETH_RSS_ETH | RTE_ETH_RSS_VLAN | RTE_ETH_RSS_IP |
> RTE_ETH_RSS_TCP |
> > RTE_ETH_RSS_UDP | RTE_ETH_RSS_SCTP |
> RTE_ETH_RSS_L2_PAYLOAD |
> > RTE_ETH_RSS_L2TPV3 | RTE_ETH_RSS_ESP | RTE_ETH_RSS_AH
> |
> > RTE_ETH_RSS_PFCP
> > |
> > - RTE_ETH_RSS_GTPU | RTE_ETH_RSS_ECPRI |
> RTE_ETH_RSS_MPLS},
> > + RTE_ETH_RSS_GTPU | RTE_ETH_RSS_ECPRI |
> RTE_ETH_RSS_MPLS |
> > RTE_ETH_RSS_L2TPV2},
> > { "none", 0 },
> > { "eth", RTE_ETH_RSS_ETH },
> > { "l2-src-only", RTE_ETH_RSS_L2_SRC_ONLY }, @@ -143,6 +143,7 @@
> > const struct rss_type_info rss_type_table[] = {
> > { "mpls", RTE_ETH_RSS_MPLS },
> > { "ipv4-chksum", RTE_ETH_RSS_IPV4_CHKSUM },
> > { "l4-chksum", RTE_ETH_RSS_L4_CHKSUM },
> > + { "l2tpv2", RTE_ETH_RSS_L2TPV2 },
> > { NULL, 0 },
> > };
> >
> > diff --git a/doc/guides/rel_notes/release_22_03.rst
> > b/doc/guides/rel_notes/release_22_03.rst
> > index 6d99d1eaa9..30695f11b7 100644
> > --- a/doc/guides/rel_notes/release_22_03.rst
> > +++ b/doc/guides/rel_notes/release_22_03.rst
> > @@ -55,6 +55,11 @@ New Features
> > Also, make sure to start the actual text at the margin.
> > =======================================================
> >
> > +* **Added new RSS offload types for L2TPv2 in RSS flow.**
> > +
> > + Added macros RTE_ETH_RSS_L2TPV2, now L2TPv2 field can be used as
> > + input set for RSS.
> > +
>
> Which fields exactly? I assume it is tunnel id but can it be something else?
>
It's session ID, I'll update it in v2.
> >
> > Removed Items
> > -------------
> > diff --git a/lib/ethdev/rte_ethdev.h b/lib/ethdev/rte_ethdev.h index
> > fa299c8ad7..43f582c90c 100644
> > --- a/lib/ethdev/rte_ethdev.h
> > +++ b/lib/ethdev/rte_ethdev.h
> > @@ -645,6 +645,9 @@ struct rte_eth_rss_conf {
> > #define RTE_ETH_RSS_L4_CHKSUM RTE_BIT64(35)
> > #define ETH_RSS_L4_CHKSUM RTE_ETH_RSS_L4_CHKSUM
> >
> > +#define RTE_ETH_RSS_L2TPV2 RTE_BIT64(36)
> > +#define ETH_RSS_L2TPV2 RTE_ETH_RSS_L2TPV2
> > +
>
> I know it was not done until now, but I think it is important to explain exactly on
> what fields this hash is being done.
>
Yes, I'll add some comments to explain it in v2.
>
> > /*
> > * We use the following macros to combine with above RTE_ETH_RSS_* for
> > * more specific input set selection. These bits are defined starting
> > --
> > 2.25.1
>
> Best,
> Ori
next prev parent reply other threads:[~2022-01-25 1:54 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-01-24 6:56 [PATCH 0/6] support L2TPv2 for AVF RSS hash and FDIR Jie Wang
2022-01-24 6:56 ` [PATCH 1/6] ethdev: add L2TPv2 RSS offload type Jie Wang
2022-01-24 16:35 ` Ori Kam
2022-01-25 1:54 ` Wang, Jie1X [this message]
2022-01-24 6:56 ` [PATCH 2/6] net: fix L2TPv2 common header Jie Wang
2022-01-24 6:56 ` [PATCH 3/6] app/testpmd: add 6 types of L2TPv2 message Jie Wang
2022-01-24 6:56 ` [PATCH 4/6] common/iavf: add session ID fields for L2TPv2 Jie Wang
2022-01-24 6:56 ` [PATCH 5/6] net/iavf: support L2TPv2 for AVF HASH Jie Wang
2022-01-24 6:56 ` [PATCH 6/6] net/iavf: support L2TPv2 for AVF FDIR Jie Wang
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=DM8PR11MB56392BD1DC39F91390A0C25CD15F9@DM8PR11MB5639.namprd11.prod.outlook.com \
--to=jie1x.wang@intel.com \
--cc=aman.deep.singh@intel.com \
--cc=andrew.rybchenko@oktetlabs.ru \
--cc=beilei.xing@intel.com \
--cc=dev@dpdk.org \
--cc=ferruh.yigit@intel.com \
--cc=jingjing.wu@intel.com \
--cc=orika@nvidia.com \
--cc=qi.z.zhang@intel.com \
--cc=stevex.yang@intel.com \
--cc=thomas@monjalon.net \
/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).