patches for DPDK stable branches
 help / color / mirror / Atom feed
* [PATCH] net/nfp: advertise no support for keeping flow rules
@ 2023-02-03  1:36 Chaoyong He
  2023-02-03  1:59 ` [PATCH v2] " Chaoyong He
  0 siblings, 1 reply; 3+ messages in thread
From: Chaoyong He @ 2023-02-03  1:36 UTC (permalink / raw)
  To: dev
  Cc: oss-drivers, niklas.soderlund, Peng Zhang, chaoyong.he, stable,
	Louis Peens

From: Peng Zhang <peng.zhang@corigine.com>

Explicitly clear the RTE_ETH_DEV_CAPA_FLOW_RULE_KEEP capability
bit to match behavior of other PMDs. This was missed when first
adding flow support for the NFP PMD.

Fixes: 30ecce522732 ("net/nfp: support flow API")
Cc: chaoyong.he@corigine.com
Cc: stable@dpdk.org

Signed-off-by: Peng Zhang <peng.zhang@corigine.com>
Reviewed-by: Chaoyong He <chaoyong.he@corigine.com>
Reviewed-by: Louis Peens <louis.peens@corigine.com>
---
 drivers/net/nfp/nfp_common.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/net/nfp/nfp_common.c b/drivers/net/nfp/nfp_common.c
index 71711bfa22..5c191b01ef 100644
--- a/drivers/net/nfp/nfp_common.c
+++ b/drivers/net/nfp/nfp_common.c
@@ -794,6 +794,7 @@ nfp_net_infos_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info)
 	dev_info->speed_capa = RTE_ETH_LINK_SPEED_1G | RTE_ETH_LINK_SPEED_10G |
 			       RTE_ETH_LINK_SPEED_25G | RTE_ETH_LINK_SPEED_40G |
 			       RTE_ETH_LINK_SPEED_50G | RTE_ETH_LINK_SPEED_100G;
+	dev_info->dev_capa &= ~RTE_ETH_DEV_CAPA_FLOW_RULE_KEEP;
 
 	return 0;
 }
-- 
2.29.3


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

* [PATCH v2] net/nfp: advertise no support for keeping flow rules
  2023-02-03  1:36 [PATCH] net/nfp: advertise no support for keeping flow rules Chaoyong He
@ 2023-02-03  1:59 ` Chaoyong He
  2023-02-10 20:28   ` Ferruh Yigit
  0 siblings, 1 reply; 3+ messages in thread
From: Chaoyong He @ 2023-02-03  1:59 UTC (permalink / raw)
  To: dev; +Cc: oss-drivers, niklas.soderlund, Peng Zhang, chaoyong.he, stable

From: Peng Zhang <peng.zhang@corigine.com>

Explicitly clear the RTE_ETH_DEV_CAPA_FLOW_RULE_KEEP capability
bit to match behavior of other PMDs. This was missed when first
adding flow support for the NFP PMD.

Fixes: 30ecce522732 ("net/nfp: support flow API")
Cc: chaoyong.he@corigine.com
Cc: stable@dpdk.org

Signed-off-by: Peng Zhang <peng.zhang@corigine.com>
Reviewed-by: Chaoyong He <chaoyong.he@corigine.com>
Reviewed-by: Niklas Söderlund <niklas.soderlund@corigine.com>
---
v2:
* Fix the wrongly 'Reviewed-by' tag.
---
 drivers/net/nfp/nfp_common.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/net/nfp/nfp_common.c b/drivers/net/nfp/nfp_common.c
index 71711bfa22..5c191b01ef 100644
--- a/drivers/net/nfp/nfp_common.c
+++ b/drivers/net/nfp/nfp_common.c
@@ -794,6 +794,7 @@ nfp_net_infos_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info)
 	dev_info->speed_capa = RTE_ETH_LINK_SPEED_1G | RTE_ETH_LINK_SPEED_10G |
 			       RTE_ETH_LINK_SPEED_25G | RTE_ETH_LINK_SPEED_40G |
 			       RTE_ETH_LINK_SPEED_50G | RTE_ETH_LINK_SPEED_100G;
+	dev_info->dev_capa &= ~RTE_ETH_DEV_CAPA_FLOW_RULE_KEEP;
 
 	return 0;
 }
-- 
2.29.3


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

* Re: [PATCH v2] net/nfp: advertise no support for keeping flow rules
  2023-02-03  1:59 ` [PATCH v2] " Chaoyong He
@ 2023-02-10 20:28   ` Ferruh Yigit
  0 siblings, 0 replies; 3+ messages in thread
From: Ferruh Yigit @ 2023-02-10 20:28 UTC (permalink / raw)
  To: Chaoyong He, dev; +Cc: oss-drivers, niklas.soderlund, Peng Zhang, stable

On 2/3/2023 1:59 AM, Chaoyong He wrote:
> From: Peng Zhang <peng.zhang@corigine.com>
> 
> Explicitly clear the RTE_ETH_DEV_CAPA_FLOW_RULE_KEEP capability
> bit to match behavior of other PMDs. This was missed when first
> adding flow support for the NFP PMD.
> 
> Fixes: 30ecce522732 ("net/nfp: support flow API")
> Cc: chaoyong.he@corigine.com
> Cc: stable@dpdk.org
> 
> Signed-off-by: Peng Zhang <peng.zhang@corigine.com>
> Reviewed-by: Chaoyong He <chaoyong.he@corigine.com>
> Reviewed-by: Niklas Söderlund <niklas.soderlund@corigine.com>
> ---
> v2:
> * Fix the wrongly 'Reviewed-by' tag.
> ---
>  drivers/net/nfp/nfp_common.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/net/nfp/nfp_common.c b/drivers/net/nfp/nfp_common.c
> index 71711bfa22..5c191b01ef 100644
> --- a/drivers/net/nfp/nfp_common.c
> +++ b/drivers/net/nfp/nfp_common.c
> @@ -794,6 +794,7 @@ nfp_net_infos_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info)
>  	dev_info->speed_capa = RTE_ETH_LINK_SPEED_1G | RTE_ETH_LINK_SPEED_10G |
>  			       RTE_ETH_LINK_SPEED_25G | RTE_ETH_LINK_SPEED_40G |
>  			       RTE_ETH_LINK_SPEED_50G | RTE_ETH_LINK_SPEED_100G;
> +	dev_info->dev_capa &= ~RTE_ETH_DEV_CAPA_FLOW_RULE_KEEP;
>  
>  	return 0;
>  }

Nack

Above code has no impact, it has been added to drivers to highlight this
feature to driver maintainers.

This feature is added to ethdev layer and requires driver support, but
driver maintainer may miss that this feature exists at all. With above
change driver maintainer can chose to remove that line meaning they are
aware but not prefer to support this feature or properly add support it.

For your case, no need to add it explicitly, if driver doesn't support
RTE_ETH_DEV_CAPA_FLOW_RULE_KEEP, no action is needed.

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

end of thread, other threads:[~2023-02-10 20:29 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-03  1:36 [PATCH] net/nfp: advertise no support for keeping flow rules Chaoyong He
2023-02-03  1:59 ` [PATCH v2] " Chaoyong He
2023-02-10 20:28   ` 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).