net/vhost: allocate interface name from heap

This patch allocates iface_name of pmd_internal from heap
in order to be able to refer from secondary processes.

Fixes: 4852aa8f6e21 ("drivers/net: enable hotplug on secondary process")
Cc: stable@dpdk.org

Signed-off-by: Itsuro Oda <oda@valinux.co.jp>
Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>
This commit is contained in:
Itsuro Oda 2020-02-06 10:39:33 +09:00 committed by Ferruh Yigit
parent 966f89d998
commit e045e85844

View File

@ -1009,7 +1009,7 @@ eth_dev_close(struct rte_eth_dev *dev)
rte_free(dev->data->tx_queues[i]);
free(internal->dev_name);
free(internal->iface_name);
rte_free(internal->iface_name);
rte_free(internal);
dev->data->dev_private = NULL;
@ -1256,9 +1256,11 @@ eth_dev_vhost_create(struct rte_vdev_device *dev, char *iface_name,
internal->dev_name = strdup(name);
if (internal->dev_name == NULL)
goto error;
internal->iface_name = strdup(iface_name);
internal->iface_name = rte_malloc_socket(name, strlen(iface_name) + 1,
0, numa_node);
if (internal->iface_name == NULL)
goto error;
strcpy(internal->iface_name, iface_name);
list->eth_dev = eth_dev;
pthread_mutex_lock(&internal_list_lock);
@ -1306,7 +1308,7 @@ eth_dev_vhost_create(struct rte_vdev_device *dev, char *iface_name,
error:
if (internal) {
free(internal->iface_name);
rte_free(internal->iface_name);
free(internal->dev_name);
}
rte_free(vring_state);