> -----Original Message-----
> From: Doherty, Declan
> Sent: Tuesday, June 20, 2017 3:34 PM
> To: De Lara Guarch, Pablo <pablo.de.lara.gua...@intel.com>;
> tho...@monjalon.net
> Cc: dev@dpdk.org
> Subject: Re: [PATCH 01/12] cryptodev: store device pointer in virtual devices
> 
> On 24/05/2017 4:27 PM, Pablo de Lara wrote:
> > Only non virtual devices were storing the pointer to rte_device
> > structure in rte_cryptodev, which will be needed to retrieve the
> > driver name for any device.
> >
> > Signed-off-by: Pablo de Lara <pablo.de.lara.gua...@intel.com>
> > ---
> ...
> 
> > diff --git a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c
> > b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c
> > index 45b25c9..49d3f32 100644
> > --- a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c
> > +++ b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c
> > @@ -699,12 +699,14 @@ cryptodev_aesni_mb_create(const char *name,
> >     }
> >
> >     dev = rte_cryptodev_pmd_virtual_dev_init(init_params->name,
> > -                   sizeof(struct aesni_mb_private), init_params-
> >socket_id);
> > +                   sizeof(struct aesni_mb_private), init_params-
> >socket_id,
> > +                   vdev);
> >     if (dev == NULL) {
> >             MB_LOG_ERR("failed to create cryptodev vdev");
> >             goto init_error;
> >     }
> >
> > +   dev->device = &vdev->device;
> 
> This is set in the rte_cryptodev_pmd_virtual_dev_init now so it isn't needed
> here.

Will fix in v2.

Thanks,
Pablo

Reply via email to