On Thu, May 19, 2016 at 06:44:44PM +0200, Thomas Monjalon wrote: > 2016-05-19 17:28, Ferruh Yigit: > > On 5/19/2016 9:33 AM, Thomas Monjalon wrote: > > > 2016-05-18 18:10, Ferruh Yigit: > > >> Add rte_eth_from_vhost() API to create vhost PMD dynamically from > > >> applications. > > > > > > How is it different from rte_eth_dev_attach() calling rte_eal_vdev_init()? > > > > > > > When used rte_eth_dev_attach(), application also needs to do:
Sigh.. I was not awared that there is such interface. > > rte_eth_dev_configure() > > rte_eth_rx_queue_setup() > > rte_eth_tx_queue_setup() > > rte_eth_dev_start() > > > > rte_eth_from_vhost() does these internally, easier to use for applications. > > This argument is not sufficient. > We are not going to add new APIs just for wrapping others. +1. --yliu