net/memif: check status of getting link info
authorAndrew Rybchenko <arybchenko@solarflare.com>
Tue, 10 Sep 2019 08:25:46 +0000 (09:25 +0100)
committerFerruh Yigit <ferruh.yigit@intel.com>
Mon, 7 Oct 2019 13:00:54 +0000 (15:00 +0200)
The return value of rte_eth_link_get() was changed from void to int.
Update the usage of the function according to the new return type.

Just log error if something goes wrong.

Signed-off-by: Andrew Rybchenko <arybchenko@solarflare.com>
drivers/net/memif/rte_eth_memif.c

index af260f7..a347e27 100644 (file)
@@ -266,7 +266,10 @@ eth_memif_rx(void *queue, struct rte_mbuf **bufs, uint16_t nb_pkts)
                return 0;
        if (unlikely(ring == NULL)) {
                /* Secondary process will attempt to request regions. */
-               rte_eth_link_get(mq->in_port, &link);
+               ret = rte_eth_link_get(mq->in_port, &link);
+               if (ret < 0)
+                       MIF_LOG(ERR, "Failed to get port %u link info: %s",
+                               mq->in_port, rte_strerror(-ret));
                return 0;
        }
 
@@ -392,8 +395,13 @@ eth_memif_tx(void *queue, struct rte_mbuf **bufs, uint16_t nb_pkts)
        if (unlikely((pmd->flags & ETH_MEMIF_FLAG_CONNECTED) == 0))
                return 0;
        if (unlikely(ring == NULL)) {
+               int ret;
+
                /* Secondary process will attempt to request regions. */
-               rte_eth_link_get(mq->in_port, &link);
+               ret = rte_eth_link_get(mq->in_port, &link);
+               if (ret < 0)
+                       MIF_LOG(ERR, "Failed to get port %u link info: %s",
+                               mq->in_port, rte_strerror(-ret));
                return 0;
        }