diff mbox series

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

Message ID 20210120062238.18721-1-hemant.agrawal@nxp.com
State New
Headers show
Series [1/2] ethdev: add MPLS RSS offload type | expand

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(-)

-- 
2.17.1

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.