Skip to contentSkip to conversation

net/iavf: fix return value of stats get function

PATCH Applied [email protected] 2 participants 2 messages Apr 14, 2020 – Apr 22, 2020 Network drivers

lore
Share & tools
lore

[email protected]

author
#0 lore raw

PATCH 1/1 net/iavf: fix return value of stats get function

The iavf_dev_stats_get function should return ret instead of -eio.

Signed-off-by Peng Cheng <[email protected]>
---
 drivers/net/iavf/iavf_ethdev.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/iavf/iavf_ethdev.c +1 -1
index 34913f9..93ab121 100644
@@ -1074,7 +1074,7 @@ iavf_dev_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats)
 	} else {
 		PMD_DRV_LOG(ERR, "Get statistics failed");
 	}
-	return -EIO;
+	return ret;
 }
 
 static int
Signature · 1 line
1.8.3.1
Acked-by on v1 1/1 Applied
#1 lore raw

Hi,

Thanks for the fix.

Quoted 2 lines · On 04/14, [email protected] wrote:

The iavf_dev_stats_get function should return ret instead of -eio.

Fixes f4a41a6953af ("net/avf: support stats")
Cc [email protected]
Quoted 17 lines
Signed-off-by Peng Cheng <[email protected]>

---
drivers/net/iavf/iavf_ethdev.c | 2 +-

1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/iavf/iavf_ethdev.c +1 -1
index 34913f9..93ab121 100644
@@ -1074,7 +1074,7 @@ iavf_dev_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats)
 	} else {
 		PMD_DRV_LOG(ERR, "Get statistics failed");
 	}
-	return -EIO;
+	return ret;
 }
Quoted 3 lines

static int

Signature · 1 line
1.8.3.1
Acked-by Xiaolong Ye <[email protected]>

Applied to dpdk-next-net-intel with fixes tag and cc table, Thanks.