net/ixgbe: fix RSS RETA be reset after port start

PATCH Superseded Murphy Yang 2 participants 2 messages Mar 29, 2021 Network drivers

Share & tools

Murphy Yang

author

2021-03-29 08:18 UTC #0 lore raw

PATCH v2 1/1 net/ixgbe: fix RSS RETA be reset after port start

If one calls ‘rte_eth_dev_rss_reta_update’ with ixgbe before starting the device (but after setting everything else), then RSS RETA configuration be zero after starting the device.

This patch gives a notification if the port not started.

Bugzilla ID: 664

Fixes 249358424eab ("ixgbe: RSS RETA configuration")

Signed-off-by Murphy Yang <[email protected]>

---
v2:
- tune the return value

 drivers/net/ixgbe/ixgbe_ethdev.c | 8 ++++++++
 1 file changed, 8 insertions(+)

drivers/net/ixgbe/ixgbe_ethdev.c +8 -0

index 8a9a21e7c2..6aebf9c11e 100644
@@ -5015,11 +5015,19 @@ ixgbe_dev_rss_reta_update(struct rte_eth_dev *dev,
 	uint32_t reta, r;
 	uint16_t idx, shift;
 	struct ixgbe_adapter *adapter = dev->data->dev_private;
+	struct rte_eth_dev_data *dev_data = dev->data;
 	struct ixgbe_hw *hw = IXGBE_DEV_PRIVATE_TO_HW(dev->data->dev_private);
 	uint32_t reta_reg;
 
 	PMD_INIT_FUNC_TRACE();
 
+	if (!dev_data->dev_started) {
+		PMD_DRV_LOG(ERR,
+			"port %d must be started before configuration",
+			 dev_data->port_id);
+		return -EIO;
+	}
+
 	if (!ixgbe_rss_update_sp(hw->mac.type)) {
 		PMD_DRV_LOG(ERR, "RSS reta update is not supported on this "
 			"NIC.");

Signature · 1 line

2.17.1

Wang, Haiyue

Acked-by on v2 1/1

2021-03-29 08:36 UTC #1 lore raw

Quoted 18 lines

-----Original Message-----
From: Murphy Yang <[email protected]>
Sent: Monday, March 29, 2021 16:11
To: [email protected]
Cc: Yang, Qiming <[email protected]>; Wang, Haiyue <[email protected]>; Guo, Jia <[email protected]>; Yang, SteveX <[email protected]>; Zhang, RobinX <[email protected]>; Yang, MurphyX <[email protected]>
Subject: [PATCH v2] net/ixgbe: fix RSS RETA be reset after port start
If one calls ‘rte_eth_dev_rss_reta_update’ with ixgbe before starting the device (but after setting everything else), then RSS RETA configuration be zero after starting the device.
This patch gives a notification if the port not started.
Bugzilla ID: 664
Fixes 249358424eab ("ixgbe: RSS RETA configuration")

Quoted 7 lines

Signed-off-by Murphy Yang <[email protected]>
---
v2:
- tune the return value
 drivers/net/ixgbe/ixgbe_ethdev.c | 8 ++++++++
 1 file changed, 8 insertions(+)

Acked-by Haiyue Wang <[email protected]>

Quoted 2 lines

--
2.17.1