DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH 0/5] fixes for hns3 PMD
@ 2021-04-20  8:59 Min Hu (Connor)
  2021-04-20  8:59 ` [dpdk-dev] [PATCH 1/5] net/hns3: delete unused macro Min Hu (Connor)
                   ` (5 more replies)
  0 siblings, 6 replies; 8+ messages in thread
From: Min Hu (Connor) @ 2021-04-20  8:59 UTC (permalink / raw)
  To: dev; +Cc: ferruh.yigit

This patch contains three coding fixes and two doc fixes.

Min Hu (Connor) (5):
  net/hns3: delete unused macro
  net/hns3: fix enum variable used as boolean
  net/hns3: fix unchecked function call
  doc: update hns3 feature list
  doc: fix Rx burst function doc

 doc/guides/nics/features/hns3.ini | 1 +
 doc/guides/nics/hns3.rst          | 2 +-
 drivers/net/hns3/hns3_ethdev.c    | 6 +++---
 drivers/net/hns3/hns3_rxtx.h      | 1 -
 drivers/net/hns3/hns3_tm.c        | 2 +-
 5 files changed, 6 insertions(+), 6 deletions(-)

-- 
2.7.4


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [dpdk-dev] [PATCH 1/5] net/hns3: delete unused macro
  2021-04-20  8:59 [dpdk-dev] [PATCH 0/5] fixes for hns3 PMD Min Hu (Connor)
@ 2021-04-20  8:59 ` Min Hu (Connor)
  2021-04-20  8:59 ` [dpdk-dev] [PATCH 2/5] net/hns3: fix enum variable used as boolean Min Hu (Connor)
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 8+ messages in thread
From: Min Hu (Connor) @ 2021-04-20  8:59 UTC (permalink / raw)
  To: dev; +Cc: ferruh.yigit

'HNS3_RXD_LKBK_B' was defined in previous versions but no used.
This patch deleted it.

Fixes: bba636698316 ("net/hns3: support Rx/Tx and related operations")
Cc: stable@dpdk.org

Signed-off-by: Min Hu (Connor) <humin29@huawei.com>
---
 drivers/net/hns3/hns3_rxtx.h | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/net/hns3/hns3_rxtx.h b/drivers/net/hns3/hns3_rxtx.h
index f7b457b..703c4b7 100644
--- a/drivers/net/hns3/hns3_rxtx.h
+++ b/drivers/net/hns3/hns3_rxtx.h
@@ -106,7 +106,6 @@
 #define HNS3_RXD_L3L4P_B			11
 
 #define HNS3_RXD_TS_VLD_B			14
-#define HNS3_RXD_LKBK_B				15
 #define HNS3_RXD_GRO_SIZE_S			16
 #define HNS3_RXD_GRO_SIZE_M			(0x3fff << HNS3_RXD_GRO_SIZE_S)
 
-- 
2.7.4


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [dpdk-dev] [PATCH 2/5] net/hns3: fix enum variable used as boolean
  2021-04-20  8:59 [dpdk-dev] [PATCH 0/5] fixes for hns3 PMD Min Hu (Connor)
  2021-04-20  8:59 ` [dpdk-dev] [PATCH 1/5] net/hns3: delete unused macro Min Hu (Connor)
@ 2021-04-20  8:59 ` Min Hu (Connor)
  2021-04-20  8:59 ` [dpdk-dev] [PATCH 3/5] net/hns3: fix unchecked function call Min Hu (Connor)
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 8+ messages in thread
From: Min Hu (Connor) @ 2021-04-20  8:59 UTC (permalink / raw)
  To: dev; +Cc: ferruh.yigit

params->leaf.cman has enum type which is not isomorphic with boolean
type, however it is used as a boolean expression.

This patch fixed it.

Fixes: c09c7847d892 ("net/hns3: support traffic management")

Signed-off-by: Min Hu (Connor) <humin29@huawei.com>
---
 drivers/net/hns3/hns3_tm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/hns3/hns3_tm.c b/drivers/net/hns3/hns3_tm.c
index 165d1fb..aae4970 100644
--- a/drivers/net/hns3/hns3_tm.c
+++ b/drivers/net/hns3/hns3_tm.c
@@ -385,7 +385,7 @@ hns3_tm_leaf_node_param_check(struct rte_eth_dev *dev __rte_unused,
 		return -EINVAL;
 	}
 
-	if (params->leaf.cman) {
+	if (params->leaf.cman != RTE_TM_CMAN_TAIL_DROP) {
 		error->type = RTE_TM_ERROR_TYPE_NODE_PARAMS_CMAN;
 		error->message = "congestion management not supported";
 		return -EINVAL;
-- 
2.7.4


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [dpdk-dev] [PATCH 3/5] net/hns3: fix unchecked function call
  2021-04-20  8:59 [dpdk-dev] [PATCH 0/5] fixes for hns3 PMD Min Hu (Connor)
  2021-04-20  8:59 ` [dpdk-dev] [PATCH 1/5] net/hns3: delete unused macro Min Hu (Connor)
  2021-04-20  8:59 ` [dpdk-dev] [PATCH 2/5] net/hns3: fix enum variable used as boolean Min Hu (Connor)
@ 2021-04-20  8:59 ` Min Hu (Connor)
  2021-04-20  8:59 ` [dpdk-dev] [PATCH 4/5] doc: update hns3 feature list Min Hu (Connor)
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 8+ messages in thread
From: Min Hu (Connor) @ 2021-04-20  8:59 UTC (permalink / raw)
  To: dev; +Cc: ferruh.yigit

In hns3 PMD, as the handler always return 0, the return value
of a function 'rte_kvargs_process' no need to be checked. But
the API definition has return value, so 'void' could be used
to ignore that.

Fixes: a124f9e9591b ("net/hns3: add runtime config to select IO burst function")

Signed-off-by: Min Hu (Connor) <humin29@huawei.com>
---
 drivers/net/hns3/hns3_ethdev.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/net/hns3/hns3_ethdev.c b/drivers/net/hns3/hns3_ethdev.c
index b29aab5..60267e1 100644
--- a/drivers/net/hns3/hns3_ethdev.c
+++ b/drivers/net/hns3/hns3_ethdev.c
@@ -7268,11 +7268,11 @@ hns3_parse_devargs(struct rte_eth_dev *dev)
 	if (!kvlist)
 		return;
 
-	rte_kvargs_process(kvlist, HNS3_DEVARG_RX_FUNC_HINT,
+	(void)rte_kvargs_process(kvlist, HNS3_DEVARG_RX_FUNC_HINT,
 			   &hns3_parse_io_hint_func, &rx_func_hint);
-	rte_kvargs_process(kvlist, HNS3_DEVARG_TX_FUNC_HINT,
+	(void)rte_kvargs_process(kvlist, HNS3_DEVARG_TX_FUNC_HINT,
 			   &hns3_parse_io_hint_func, &tx_func_hint);
-	rte_kvargs_process(kvlist, HNS3_DEVARG_DEV_CAPS_MASK,
+	(void)rte_kvargs_process(kvlist, HNS3_DEVARG_DEV_CAPS_MASK,
 			   &hns3_parse_dev_caps_mask, &dev_caps_mask);
 	rte_kvargs_free(kvlist);
 
-- 
2.7.4


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [dpdk-dev] [PATCH 4/5] doc: update hns3 feature list
  2021-04-20  8:59 [dpdk-dev] [PATCH 0/5] fixes for hns3 PMD Min Hu (Connor)
                   ` (2 preceding siblings ...)
  2021-04-20  8:59 ` [dpdk-dev] [PATCH 3/5] net/hns3: fix unchecked function call Min Hu (Connor)
@ 2021-04-20  8:59 ` Min Hu (Connor)
  2021-04-20 10:34   ` Ferruh Yigit
  2021-04-20  8:59 ` [dpdk-dev] [PATCH 5/5] doc: fix Rx burst function doc Min Hu (Connor)
  2021-04-20 10:56 ` [dpdk-dev] [PATCH 0/5] fixes for hns3 PMD Ferruh Yigit
  5 siblings, 1 reply; 8+ messages in thread
From: Min Hu (Connor) @ 2021-04-20  8:59 UTC (permalink / raw)
  To: dev; +Cc: ferruh.yigit

Hns3 PMD has supported SR-IOV in 19.11 version.
This patch added feature description in hns3.ini.

Signed-off-by: Min Hu (Connor) <humin29@huawei.com>
---
 doc/guides/nics/features/hns3.ini | 1 +
 1 file changed, 1 insertion(+)

diff --git a/doc/guides/nics/features/hns3.ini b/doc/guides/nics/features/hns3.ini
index 0f89c60..2a46dde 100644
--- a/doc/guides/nics/features/hns3.ini
+++ b/doc/guides/nics/features/hns3.ini
@@ -25,6 +25,7 @@ Multicast MAC filter = Y
 RSS hash             = Y
 RSS key update       = Y
 RSS reta update      = Y
+SR-IOV               = Y
 DCB                  = Y
 VLAN filter          = Y
 Flow control         = Y
-- 
2.7.4


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [dpdk-dev] [PATCH 5/5] doc: fix Rx burst function doc
  2021-04-20  8:59 [dpdk-dev] [PATCH 0/5] fixes for hns3 PMD Min Hu (Connor)
                   ` (3 preceding siblings ...)
  2021-04-20  8:59 ` [dpdk-dev] [PATCH 4/5] doc: update hns3 feature list Min Hu (Connor)
@ 2021-04-20  8:59 ` Min Hu (Connor)
  2021-04-20 10:56 ` [dpdk-dev] [PATCH 0/5] fixes for hns3 PMD Ferruh Yigit
  5 siblings, 0 replies; 8+ messages in thread
From: Min Hu (Connor) @ 2021-04-20  8:59 UTC (permalink / raw)
  To: dev; +Cc: ferruh.yigit

The patch 'net/hns3: rename Rx burst function' changed `simple'
Rx function name from 'scalar' to 'scalar simple', but doc
ignored that.

This patch fixed it.

Fixes: aa5baf47e1a3 ("net/hns3: rename Rx burst function")

Signed-off-by: Min Hu (Connor) <humin29@huawei.com>
---
 doc/guides/nics/hns3.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/guides/nics/hns3.rst b/doc/guides/nics/hns3.rst
index fa45721..52d6718 100644
--- a/doc/guides/nics/hns3.rst
+++ b/doc/guides/nics/hns3.rst
@@ -59,7 +59,7 @@ Runtime Config Options
   ``sve``, if supported use the ``sve`` Rx function which indicates the
   sve algorithm.
   ``simple``, if supported use the ``simple`` Rx function which indicates
-  the scalar algorithm.
+  the scalar simple algorithm.
   ``common``, if supported use the ``common`` Rx function which indicates
   the scalar scattered algorithm.
 
-- 
2.7.4


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [dpdk-dev] [PATCH 4/5] doc: update hns3 feature list
  2021-04-20  8:59 ` [dpdk-dev] [PATCH 4/5] doc: update hns3 feature list Min Hu (Connor)
@ 2021-04-20 10:34   ` Ferruh Yigit
  0 siblings, 0 replies; 8+ messages in thread
From: Ferruh Yigit @ 2021-04-20 10:34 UTC (permalink / raw)
  To: Min Hu (Connor), dev

On 4/20/2021 9:59 AM, Min Hu (Connor) wrote:
> Hns3 PMD has supported SR-IOV in 19.11 version.
> This patch added feature description in hns3.ini.
> 

Can you please add the commit id of the patch that adds the actual support?
It can be added as fixes tag.

> Signed-off-by: Min Hu (Connor) <humin29@huawei.com>
> ---
>   doc/guides/nics/features/hns3.ini | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/doc/guides/nics/features/hns3.ini b/doc/guides/nics/features/hns3.ini
> index 0f89c60..2a46dde 100644
> --- a/doc/guides/nics/features/hns3.ini
> +++ b/doc/guides/nics/features/hns3.ini
> @@ -25,6 +25,7 @@ Multicast MAC filter = Y
>   RSS hash             = Y
>   RSS key update       = Y
>   RSS reta update      = Y
> +SR-IOV               = Y
>   DCB                  = Y
>   VLAN filter          = Y
>   Flow control         = Y
> 


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [dpdk-dev] [PATCH 0/5] fixes for hns3 PMD
  2021-04-20  8:59 [dpdk-dev] [PATCH 0/5] fixes for hns3 PMD Min Hu (Connor)
                   ` (4 preceding siblings ...)
  2021-04-20  8:59 ` [dpdk-dev] [PATCH 5/5] doc: fix Rx burst function doc Min Hu (Connor)
@ 2021-04-20 10:56 ` Ferruh Yigit
  5 siblings, 0 replies; 8+ messages in thread
From: Ferruh Yigit @ 2021-04-20 10:56 UTC (permalink / raw)
  To: Min Hu (Connor), dev

On 4/20/2021 9:59 AM, Min Hu (Connor) wrote:
> This patch contains three coding fixes and two doc fixes.
> 
> Min Hu (Connor) (5):
>    net/hns3: delete unused macro
>    net/hns3: fix enum variable used as boolean
>    net/hns3: fix unchecked function call
>    doc: update hns3 feature list
>    doc: fix Rx burst function doc

Except from 4/5,
Series applied to dpdk-next-net/main, thanks.

4/5 can be sent separately with commit log update.

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2021-04-20 10:57 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-20  8:59 [dpdk-dev] [PATCH 0/5] fixes for hns3 PMD Min Hu (Connor)
2021-04-20  8:59 ` [dpdk-dev] [PATCH 1/5] net/hns3: delete unused macro Min Hu (Connor)
2021-04-20  8:59 ` [dpdk-dev] [PATCH 2/5] net/hns3: fix enum variable used as boolean Min Hu (Connor)
2021-04-20  8:59 ` [dpdk-dev] [PATCH 3/5] net/hns3: fix unchecked function call Min Hu (Connor)
2021-04-20  8:59 ` [dpdk-dev] [PATCH 4/5] doc: update hns3 feature list Min Hu (Connor)
2021-04-20 10:34   ` Ferruh Yigit
2021-04-20  8:59 ` [dpdk-dev] [PATCH 5/5] doc: fix Rx burst function doc Min Hu (Connor)
2021-04-20 10:56 ` [dpdk-dev] [PATCH 0/5] fixes for hns3 PMD Ferruh Yigit

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