app/testpmd: fix log after detach
authorZhiyong Yang <zhiyong.yang@intel.com>
Mon, 14 May 2018 03:20:48 +0000 (11:20 +0800)
committerThomas Monjalon <thomas@monjalon.net>
Mon, 14 May 2018 10:51:37 +0000 (12:51 +0200)
The second parameter "name" in the function rte_eth_dev_detach
has been already redefined as "char *name __rte_unused",
"port_id" is printed instead of "name" in testpmd.

Fixes: b65ecf199324 ("devargs: rename legacy API")
Cc: stable@dpdk.org
Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>
Acked-by: Matan Azrad <matan@mellanox.com>
app/test-pmd/testpmd.c

index 664c435..02a2819 100644 (file)
@@ -1985,7 +1985,7 @@ detach_port(portid_t port_id)
                port_flow_flush(port_id);
 
        if (rte_eth_dev_detach(port_id, name)) {
-               TESTPMD_LOG(ERR, "Failed to detach port '%s'\n", name);
+               TESTPMD_LOG(ERR, "Failed to detach port %u\n", port_id);
                return;
        }
 
@@ -1993,8 +1993,8 @@ detach_port(portid_t port_id)
 
        update_fwd_ports(RTE_MAX_ETHPORTS);
 
-       printf("Port '%s' is detached. Now total ports is %d\n",
-                       name, nb_ports);
+       printf("Port %u is detached. Now total ports is %d\n",
+                       port_id, nb_ports);
        printf("Done\n");
        return;
 }