diff mbox series

[1/2] ethdev: add MPLS RSS offload type

Message ID 20210120062238.18721-1-hemant.agrawal@nxp.com (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers show
Series [1/2] ethdev: add MPLS RSS offload type | expand

Checks

Context Check Description
ci/checkpatch success coding style OK

Commit Message

Hemant Agrawal Jan. 20, 2021, 6:22 a.m. UTC
This patch defines new RSS offload types for MPLS. The distribution
will on the basis of MPLS tag.

Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>
---
 app/test-pmd/cmdline.c                      | 4 +++-
 doc/guides/testpmd_app_ug/testpmd_funcs.rst | 2 +-
 lib/librte_ethdev/rte_ethdev.h              | 4 +++-
 3 files changed, 7 insertions(+), 3 deletions(-)

Comments

Ferruh Yigit Jan. 27, 2021, 11:59 a.m. UTC | #1
On 1/20/2021 6:22 AM, Hemant Agrawal wrote:
> This patch defines new RSS offload types for MPLS. The distribution
> will on the basis of MPLS tag.
> 
> Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>
> ---
>   app/test-pmd/cmdline.c                      | 4 +++-
>   doc/guides/testpmd_app_ug/testpmd_funcs.rst | 2 +-
>   lib/librte_ethdev/rte_ethdev.h              | 4 +++-
>   3 files changed, 7 insertions(+), 3 deletions(-)
> 
> diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c
> index 89034c8b72..01cc79b12c 100644
> --- a/app/test-pmd/cmdline.c
> +++ b/app/test-pmd/cmdline.c
> @@ -2220,6 +2220,8 @@ cmd_config_rss_parsed(void *parsed_result,
>   		rss_conf.rss_hf = ETH_RSS_GTPU;
>   	else if (!strcmp(res->value, "ecpri"))
>   		rss_conf.rss_hf = ETH_RSS_ECPRI;
> +	else if (!strcmp(res->value, "mpls"))
> +		rss_conf.rss_hf = ETH_RSS_MPLS;
>   	else if (!strcmp(res->value, "none"))
>   		rss_conf.rss_hf = 0;
>   	else if (!strcmp(res->value, "level-default")) {
> @@ -2290,7 +2292,7 @@ cmdline_parse_inst_t cmd_config_rss = {
>   	.data = NULL,
>   	.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|none|level-default|"
> +		"nvgre|vxlan-gpe|l2tpv3|esp|ah|pfcp|ecpri|mpls|none|level-default|"
>   		"level-outer|level-inner|<flowtype_id>",

It should be added to the help string too, in 'cmd_help_long_parsed()', I can do 
it in next-net, no new version required for this.

>   	.tokens = {
>   		(void *)&cmd_config_rss_port,
> diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> index 458f25bb63..3ccc9fc300 100644
> --- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> +++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> @@ -2270,7 +2270,7 @@ port config - RSS
>   
>   Set the RSS (Receive Side Scaling) mode on or off::
>   
> -   testpmd> port config all rss (all|default|eth|vlan|ip|tcp|udp|sctp|ether|port|vxlan|geneve|nvgre|vxlan-gpe|l2tpv3|esp|ah|pfcp|ecpri|none)
> +   testpmd> 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)
>   
>   RSS is on by default.
>   
> diff --git a/lib/librte_ethdev/rte_ethdev.h b/lib/librte_ethdev/rte_ethdev.h
> index eb299955d4..7c0b6eda8f 100644
> --- a/lib/librte_ethdev/rte_ethdev.h
> +++ b/lib/librte_ethdev/rte_ethdev.h
> @@ -527,6 +527,7 @@ struct rte_eth_rss_conf {
>   #define ETH_RSS_PFCP               (1ULL << 30)
>   #define ETH_RSS_PPPOE		   (1ULL << 31)
>   #define ETH_RSS_ECPRI		   (1ULL << 32)
> +#define ETH_RSS_MPLS		   (1ULL << 33)
>   
>   /*
>    * We use the following macros to combine with above ETH_RSS_* for
> @@ -758,7 +759,8 @@ rte_eth_rss_hf_refine(uint64_t rss_hf)
>   	ETH_RSS_PORT  | \
>   	ETH_RSS_VXLAN | \
>   	ETH_RSS_GENEVE | \
> -	ETH_RSS_NVGRE)
> +	ETH_RSS_NVGRE | \
> +	ETH_RSS_MPLS)
>   
>   /*
>    * Definitions used for redirection table entry size.
>
Ferruh Yigit Jan. 27, 2021, 12:26 p.m. UTC | #2
On 1/20/2021 6:22 AM, Hemant Agrawal wrote:
> This patch defines new RSS offload types for MPLS. The distribution
> will on the basis of MPLS tag.
> 
> Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>

Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>

Series applied to dpdk-next-net/main, thanks.

(testpmd cmd_help_long_parsed() updated while merging)
diff mbox series

Patch

diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c
index 89034c8b72..01cc79b12c 100644
--- a/app/test-pmd/cmdline.c
+++ b/app/test-pmd/cmdline.c
@@ -2220,6 +2220,8 @@  cmd_config_rss_parsed(void *parsed_result,
 		rss_conf.rss_hf = ETH_RSS_GTPU;
 	else if (!strcmp(res->value, "ecpri"))
 		rss_conf.rss_hf = ETH_RSS_ECPRI;
+	else if (!strcmp(res->value, "mpls"))
+		rss_conf.rss_hf = ETH_RSS_MPLS;
 	else if (!strcmp(res->value, "none"))
 		rss_conf.rss_hf = 0;
 	else if (!strcmp(res->value, "level-default")) {
@@ -2290,7 +2292,7 @@  cmdline_parse_inst_t cmd_config_rss = {
 	.data = NULL,
 	.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|none|level-default|"
+		"nvgre|vxlan-gpe|l2tpv3|esp|ah|pfcp|ecpri|mpls|none|level-default|"
 		"level-outer|level-inner|<flowtype_id>",
 	.tokens = {
 		(void *)&cmd_config_rss_port,
diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
index 458f25bb63..3ccc9fc300 100644
--- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst
+++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
@@ -2270,7 +2270,7 @@  port config - RSS
 
 Set the RSS (Receive Side Scaling) mode on or off::
 
-   testpmd> port config all rss (all|default|eth|vlan|ip|tcp|udp|sctp|ether|port|vxlan|geneve|nvgre|vxlan-gpe|l2tpv3|esp|ah|pfcp|ecpri|none)
+   testpmd> 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)
 
 RSS is on by default.
 
diff --git a/lib/librte_ethdev/rte_ethdev.h b/lib/librte_ethdev/rte_ethdev.h
index eb299955d4..7c0b6eda8f 100644
--- a/lib/librte_ethdev/rte_ethdev.h
+++ b/lib/librte_ethdev/rte_ethdev.h
@@ -527,6 +527,7 @@  struct rte_eth_rss_conf {
 #define ETH_RSS_PFCP               (1ULL << 30)
 #define ETH_RSS_PPPOE		   (1ULL << 31)
 #define ETH_RSS_ECPRI		   (1ULL << 32)
+#define ETH_RSS_MPLS		   (1ULL << 33)
 
 /*
  * We use the following macros to combine with above ETH_RSS_* for
@@ -758,7 +759,8 @@  rte_eth_rss_hf_refine(uint64_t rss_hf)
 	ETH_RSS_PORT  | \
 	ETH_RSS_VXLAN | \
 	ETH_RSS_GENEVE | \
-	ETH_RSS_NVGRE)
+	ETH_RSS_NVGRE | \
+	ETH_RSS_MPLS)
 
 /*
  * Definitions used for redirection table entry size.