[PATCH 1/2] net/netvsc: fix link speed not updated

[Shrey Khandelwal]

hn_dev_link_update() returns without updating the cached link when
the link status has not changed. A speed change alone is never
reported, and if the link is down at port start, the cached speed
and duplex stay zero.

Always update the cached link, and only log when the status changes.

Fixes: 4e9c73e96e83 ("net/netvsc: add Hyper-V network device")
Cc: stable at dpdk.org

Signed-off-by: Shrey Khandelwal <s.khandelwal at f5.com>
---
 .mailmap                       | 1 +
 drivers/net/netvsc/hn_ethdev.c | 8 +++-----
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/.mailmap b/.mailmap
index 2e348c3bce..131b9f86cb 100644
--- a/.mailmap
+++ b/.mailmap
@@ -1558,6 +1558,7 @@ Shlomi Gridish <sgridish at marvell.com>
 Shougang Wang <shougangx.wang at intel.com>
 Shraddha Joshi <jshraddha at vmware.com>
 Shreesh Adiga <16567adigashreesh at gmail.com>
+Shrey Khandelwal <s.khandelwal at f5.com>
 Shreyansh Jain <shreyansh.jain at nxp.com>
 Shrikrishna Khare <skhare at vmware.com>
 Shu Shen <shu.shen at radisys.com>
diff --git a/drivers/net/netvsc/hn_ethdev.c b/drivers/net/netvsc/hn_ethdev.c
index e4fdf89873..5dc91f9503 100644
--- a/drivers/net/netvsc/hn_ethdev.c
+++ b/drivers/net/netvsc/hn_ethdev.c
@@ -298,11 +298,9 @@ hn_dev_link_update(struct rte_eth_dev *dev,
 	else
 		link.link_status = RTE_ETH_LINK_DOWN;
 
-	if (old.link_status == link.link_status)
-		return 0;
-
-	PMD_INIT_LOG(DEBUG, "Port %d is %s", dev->data->port_id,
-		     (link.link_status == RTE_ETH_LINK_UP) ? "up" : "down");
+	if (old.link_status != link.link_status)
+		PMD_INIT_LOG(DEBUG, "Port %d is %s", dev->data->port_id,
+			     (link.link_status == RTE_ETH_LINK_UP) ? "up" : "down");
 
 	return rte_eth_linkstatus_set(dev, &link);
 }
-- 
2.55.0