Hi Thomas,
<snip> > What is the root cause of the crash? > I guess it is rte_free(eth_dev->data->mac_addrs) because eth_dev is NULL. Yes, that is the roor cause. > > [...] > > -err: > > - rte_free(pci_dev); > > - rte_free(internals); > > +err4: > > rte_free(eth_dev->data->mac_addrs); > > - > > +err3: > > + rte_free(internals); > > +err2: > > + rte_free(pci_dev); > > +err1: > > return -1; > > } > > rte_free should be harmless with NULL parameter. > It is easier to add "if (eth_dev != NULL)" than maintaining many error labels. I will revise patch to remove multible error labels. > > By the way, why rte_eth_dev_release_port() is not called? I will add a call to rte_eth_dev_release_port(). Regards, Bernard.
