16/04/2018 15:05, Declan Doherty: > --- a/lib/librte_ether/rte_ethdev.c > +++ b/lib/librte_ether/rte_ethdev.c > @@ -348,7 +348,8 @@ rte_eth_dev_release_port(struct rte_eth_dev *eth_dev) > rte_eth_dev_shared_data_prepare(); > > rte_spinlock_lock(&rte_eth_dev_shared_data->ownership_lock); > - > + eth_dev->device = NULL; > + eth_dev->intr_handle = NULL; > eth_dev->state = RTE_ETH_DEV_UNUSED;
Shouldn't it be in a separate patch with a proper explanation? > + * @param device > + * rte_device handle. > + * @param name There is a tab between param and name. > --- a/lib/librte_ether/rte_ethdev_version.map > +++ b/lib/librte_ether/rte_ethdev_version.map > @@ -229,3 +229,11 @@ EXPERIMENTAL { > rte_mtr_stats_update; > > } DPDK_17.11; > + > +EXPERIMENTAL { > + global: > + > + rte_eth_dev_create; > + rte_eth_dev_destroy; > + > +} DPDK_18.05; There is already an EXPERIMENTAL block. Maybe you need to rebase.