> -----Original Message----- > From: David Marchand [mailto:david.marchand at 6wind.com] > Sent: Thursday, July 28, 2016 5:18 AM > To: De Lara Guarch, Pablo > Cc: dev at dpdk.org; Mcnamara, John > Subject: Re: [dpdk-dev] [PATCH] doc: announce API change for virtual device > initialization > > Hello Pablo, > > On Mon, Jul 4, 2016 at 4:50 PM, Pablo de Lara > <pablo.de.lara.guarch at intel.com> wrote: > > In order to create a virtual device, user needs to call > > rte_eal_vdev_init generally, but this function returns 0 > > on success or negative number if error. Instead, something > > more useful would be to return the port or device id of the > > device created, so the user can call rte_eal_vdev_init > > function and use the device straight away, using the id returned > > by the function. > > > > Signed-off-by: Pablo de Lara <pablo.de.lara.guarch at intel.com> > > --- > > doc/guides/rel_notes/deprecation.rst | 5 +++++ > > 1 file changed, 5 insertions(+) > > > > diff --git a/doc/guides/rel_notes/deprecation.rst > b/doc/guides/rel_notes/deprecation.rst > > index f502f86..60cc7c7 100644 > > --- a/doc/guides/rel_notes/deprecation.rst > > +++ b/doc/guides/rel_notes/deprecation.rst > > @@ -41,3 +41,8 @@ Deprecation Notices > > * The mempool functions for single/multi producer/consumer are > deprecated and > > will be removed in 16.11. > > It is replaced by rte_mempool_generic_get/put functions. > > + > > +* The rte_eal_vdev_init function will be changed in 16.11 to return > > + the port/device id of the device created, instead of 0, when it has been > > + initialized successfully, so user can use the returned value straight > > away > > + to call all the device functions that require that parameter. > > -- > > 2.5.0 > > > > This is a layer violation. > EAL does not know a thing about "ports". > > This information should come from the crypto framework and so an api > in crypto framework is the right place, not EAL. > > This is a NACK for me.
Fair enough. So you mean to use rte_eth_dev_attach in ethdev library and a similar function in cryptodev library? Thanks, Pablo > > > -- > David Marchand