Skip to contentSkip to conversation

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

PATCH Applied Murphy Yang 3 participants 3 messages Mar 29, 2021 – Apr 1, 2021 1 message not in the archive Network drivers

lore
Share & tools
lore

A message that is not in the archive (sent off-list or not yet received): [email protected]

Murphy Yang

author
#1 lore raw

PATCH v3 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]>

---
v3:
- modify the notify message
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..d8ef618e5f 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 rss reta update",
+			 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
Acked-by on v3 1/1
#2 lore raw

drivers/net/ixgbe/ixgbe_ethdev.c

Quoted 31 lines

-----Original Message-----
From: Yang, MurphyX <[email protected]>
Sent: Monday, March 29, 2021 16:29
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 v3] 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]>

---
v3:
- modify the notify message
v2:
- tune the return value

 drivers/net/ixgbe/ixgbe_ethdev.c | 8 ++++++++
 1 file changed, 8 insertions(+)
drivers/net/ixgbe/ixgbe_ethdev.c +0 -0
index 8a9a21e7c2..d8ef618e5f 100644
Acked-by Haiyue Wang <[email protected]>
Quoted 2 lines

--
2.17.1

Applied
#3 lore raw
Quoted 40 lines

-----Original Message-----
From: dev <[email protected]> On Behalf Of Wang, Haiyue
Sent: Monday, March 29, 2021 8:41 PM
To: Yang, MurphyX <[email protected]>; [email protected]
Cc: Yang, Qiming <[email protected]>; Guo, Jia <[email protected]>; Yang, SteveX <[email protected]>; Zhang, RobinX <[email protected]>
Subject: Re: [dpdk-dev] [PATCH v3] net/ixgbe: fix RSS RETA be reset after port start

Quoted 28 lines

-----Original Message-----
From: Yang, MurphyX <[email protected]>
Sent: Monday, March 29, 2021 16:29
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 v3] 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]>

---
v3:
- modify the notify message
v2:
- tune the return value

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

drivers/net/ixgbe/ixgbe_ethdev.c

Quoted 5 lines
drivers/net/ixgbe/ixgbe_ethdev.c +0 -0
index 8a9a21e7c2..d8ef618e5f 100644
Acked-by Haiyue Wang <[email protected]>

Applied to dpdk-next-net-intel.

Thanks
Qi

Quoted 3 lines
Quoted 3 lines

--
2.17.1