[PATCH v2] net/ixgbe: fix ixgbe firmware version inconsistency
Shiyang He
shiyangx.he at intel.com
Mon Jan 9 07:53:09 CET 2023
This patch follows the code of ixgbe kernel driver so that it keeps the
firmware version obtained by dpdk-ethtool consistent with that obtained
by linux-ethtool.
Fixes: 8b0b56574269 ("net/ixgbe: add firmware version get")
Cc: stable at dpdk.org
Signed-off-by: Shiyang He <shiyangx.he at intel.com>
v2: follow the code of ixgbe kernel driver
---
drivers/net/ixgbe/ixgbe_ethdev.c | 31 ++++++++++++++++++++-----------
1 file changed, 20 insertions(+), 11 deletions(-)
diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c b/drivers/net/ixgbe/ixgbe_ethdev.c
index ae9f65b334..65655b9212 100644
--- a/drivers/net/ixgbe/ixgbe_ethdev.c
+++ b/drivers/net/ixgbe/ixgbe_ethdev.c
@@ -3852,23 +3852,32 @@ static int
ixgbe_fw_version_get(struct rte_eth_dev *dev, char *fw_version, size_t fw_size)
{
struct ixgbe_hw *hw = IXGBE_DEV_PRIVATE_TO_HW(dev->data->dev_private);
- u16 eeprom_verh, eeprom_verl;
- u32 etrack_id;
+ struct ixgbe_nvm_version nvm_ver;
int ret;
- ixgbe_read_eeprom(hw, 0x2e, &eeprom_verh);
- ixgbe_read_eeprom(hw, 0x2d, &eeprom_verl);
+ ixgbe_get_oem_prod_version(hw, &nvm_ver);
+ if (nvm_ver.oem_valid) {
+ snprintf(fw_version, fw_size, "%x.%x.%x",
+ nvm_ver.oem_major, nvm_ver.oem_minor,
+ nvm_ver.oem_release);
+ return 0;
+ }
+
+ ixgbe_get_etk_id(hw, &nvm_ver);
+ ixgbe_get_orom_version(hw, &nvm_ver);
- etrack_id = (eeprom_verh << 16) | eeprom_verl;
- ret = snprintf(fw_version, fw_size, "0x%08x", etrack_id);
+ if (nvm_ver.or_valid) {
+ snprintf(fw_version, fw_size, "0x%08x, %d.%d.%d",
+ nvm_ver.etk_id, nvm_ver.or_major,
+ nvm_ver.or_build, nvm_ver.or_patch);
+ return 0;
+ }
+
+ ret = snprintf(fw_version, fw_size, "0x%08x", nvm_ver.etk_id);
if (ret < 0)
return -EINVAL;
- ret += 1; /* add the size of '\0' */
- if (fw_size < (size_t)ret)
- return ret;
- else
- return 0;
+ return (fw_size < (size_t)ret++) ? ret : 0;
}
static int
--
2.34.1
More information about the stable
mailing list