patch 'net/nfp: notify flower firmware about PF speed' has been queued to stable release 23.11.3

[Xueming Li]

Hi,

FYI, your patch has been queued to stable release 23.11.3

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 11/30/24. So please
shout if anyone has objections.

Also note that after the patch there's a diff of the upstream commit vs the
patch applied to the branch. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.

Queued patches are on a temporary branch at:
https://git.dpdk.org/dpdk-stable/log/?h=23.11-staging

This queued commit can be viewed at:
https://git.dpdk.org/dpdk-stable/commit/?h=23.11-staging&id=ebec3137a58cbd31c0cf8483586527b1d48ae779

Thanks.

Xueming Li <xuemingl at nvidia.com>

---
>From ebec3137a58cbd31c0cf8483586527b1d48ae779 Mon Sep 17 00:00:00 2001
From: Zerun Fu <zerun.fu at corigine.com>
Date: Mon, 14 Oct 2024 10:43:55 +0800
Subject: [PATCH] net/nfp: notify flower firmware about PF speed
Cc: Xueming Li <xuemingl at nvidia.com>

[ upstream commit 2254813795099aa6c05caed5e8c0dcc7a8f03b4e ]

When using flower firmware, the VF speed is obtained from the
firmware and the firmware get the VF speed from the PF.

But the previous logic does not notify the firmware about PF speed,
and this cause VF speed to be unavailable.

Fix this by add the logic of notify firmware about PF speed.

Fixes: e1124c4f8a45 ("net/nfp: add flower representor framework")

Signed-off-by: Zerun Fu <zerun.fu at corigine.com>
Reviewed-by: Chaoyong He <chaoyong.he at corigine.com>
Reviewed-by: Long Wu <long.wu at corigine.com>
Reviewed-by: Peng Zhang <peng.zhang at corigine.com>
---
 drivers/net/nfp/flower/nfp_flower_representor.c | 3 +++
 drivers/net/nfp/nfp_net_common.c                | 2 +-
 drivers/net/nfp/nfp_net_common.h                | 2 ++
 3 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/drivers/net/nfp/flower/nfp_flower_representor.c b/drivers/net/nfp/flower/nfp_flower_representor.c
index 23709acbba..ada28d07c6 100644
--- a/drivers/net/nfp/flower/nfp_flower_representor.c
+++ b/drivers/net/nfp/flower/nfp_flower_representor.c
@@ -33,6 +33,9 @@ nfp_flower_repr_link_update(struct rte_eth_dev *dev,
 	pf_hw = repr->app_fw_flower->pf_hw;
 	ret = nfp_net_link_update_common(dev, pf_hw, link, link->link_status);

+	if (repr->repr_type == NFP_REPR_TYPE_PF)
+		nfp_net_notify_port_speed(repr->app_fw_flower->pf_hw, link);
+
 	return ret;
 }

diff --git a/drivers/net/nfp/nfp_net_common.c b/drivers/net/nfp/nfp_net_common.c
index 134a9b807e..bf44373b26 100644
--- a/drivers/net/nfp/nfp_net_common.c
+++ b/drivers/net/nfp/nfp_net_common.c
@@ -166,7 +166,7 @@ nfp_net_link_speed_rte2nfp(uint32_t speed)
 	return NFP_NET_CFG_STS_LINK_RATE_UNKNOWN;
 }

-static void
+void
 nfp_net_notify_port_speed(struct nfp_net_hw *hw,
 		struct rte_eth_link *link)
 {
diff --git a/drivers/net/nfp/nfp_net_common.h b/drivers/net/nfp/nfp_net_common.h
index 41d59bfa99..72286ab5c9 100644
--- a/drivers/net/nfp/nfp_net_common.h
+++ b/drivers/net/nfp/nfp_net_common.h
@@ -282,6 +282,8 @@ int nfp_net_flow_ctrl_set(struct rte_eth_dev *dev,
 void nfp_pf_uninit(struct nfp_pf_dev *pf_dev);
 uint32_t nfp_net_get_port_num(struct nfp_pf_dev *pf_dev,
 		struct nfp_eth_table *nfp_eth_table);
+void nfp_net_notify_port_speed(struct nfp_net_hw *hw,
+		struct rte_eth_link *link);

 #define NFP_PRIV_TO_APP_FW_NIC(app_fw_priv)\
 	((struct nfp_app_fw_nic *)app_fw_priv)
--
2.34.1

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2024-11-11 14:23:10.471975632 +0800
+++ 0115-net-nfp-notify-flower-firmware-about-PF-speed.patch	2024-11-11 14:23:05.352192835 +0800
@@ -1 +1 @@
-From 2254813795099aa6c05caed5e8c0dcc7a8f03b4e Mon Sep 17 00:00:00 2001
+From ebec3137a58cbd31c0cf8483586527b1d48ae779 Mon Sep 17 00:00:00 2001
@@ -4,0 +5,3 @@
+Cc: Xueming Li <xuemingl at nvidia.com>
+
+[ upstream commit 2254813795099aa6c05caed5e8c0dcc7a8f03b4e ]
@@ -15 +17,0 @@
-Cc: stable at dpdk.org
@@ -28 +30 @@
-index eb0a02874b..eae6ba39e1 100644
+index 23709acbba..ada28d07c6 100644
@@ -31,3 +33,3 @@
-@@ -37,6 +37,9 @@ nfp_flower_repr_link_update(struct rte_eth_dev *dev,
-
- 	ret = nfp_net_link_update_common(dev, link, link->link_status);
+@@ -33,6 +33,9 @@ nfp_flower_repr_link_update(struct rte_eth_dev *dev,
+ 	pf_hw = repr->app_fw_flower->pf_hw;
+ 	ret = nfp_net_link_update_common(dev, pf_hw, link, link->link_status);
@@ -42 +44 @@
-index b986ed4622..f76d5a6895 100644
+index 134a9b807e..bf44373b26 100644
@@ -45,2 +47,2 @@
-@@ -184,7 +184,7 @@ nfp_net_link_speed_nfp2rte_check(uint32_t speed)
- 	return RTE_ETH_SPEED_NUM_NONE;
+@@ -166,7 +166,7 @@ nfp_net_link_speed_rte2nfp(uint32_t speed)
+ 	return NFP_NET_CFG_STS_LINK_RATE_UNKNOWN;
@@ -55 +57 @@
-index 8429db68f0..d4fe8338b9 100644
+index 41d59bfa99..72286ab5c9 100644
@@ -58,4 +60,4 @@
-@@ -383,6 +383,8 @@ int nfp_net_vf_config_app_init(struct nfp_net_hw *net_hw,
- bool nfp_net_version_check(struct nfp_hw *hw,
- 		struct nfp_pf_dev *pf_dev);
- void nfp_net_ctrl_bar_size_set(struct nfp_pf_dev *pf_dev);
+@@ -282,6 +282,8 @@ int nfp_net_flow_ctrl_set(struct rte_eth_dev *dev,
+ void nfp_pf_uninit(struct nfp_pf_dev *pf_dev);
+ uint32_t nfp_net_get_port_num(struct nfp_pf_dev *pf_dev,
+ 		struct nfp_eth_table *nfp_eth_table);