[dpdk-dev] [v2, 3/6] eventdev: add crypto adapter implementation
Gujjar, Abhinandan S
abhinandan.gujjar at intel.com
Thu Apr 26 08:20:11 CEST 2018
Hi Akhil,
> -----Original Message-----
> From: Akhil Goyal [mailto:akhil.goyal at nxp.com]
> Sent: Wednesday, April 25, 2018 7:44 PM
> To: Gujjar, Abhinandan S <abhinandan.gujjar at intel.com>;
> jerin.jacob at caviumnetworks.com; hemant.agrawal at nxp.com;
> akhil.goyal at nxp.com; dev at dpdk.org
> Cc: Vangati, Narender <narender.vangati at intel.com>; Rao, Nikhil
> <nikhil.rao at intel.com>; Eads, Gage <gage.eads at intel.com>
> Subject: Re: [dpdk-dev] [v2,3/6] eventdev: add crypto adapter implementation
>
> On 4/24/2018 6:13 PM, Abhinandan Gujjar wrote:
> > Signed-off-by: Abhinandan Gujjar <abhinandan.gujjar at intel.com>
> > Signed-off-by: Nikhil Rao <nikhil.rao at intel.com>
> > Signed-off-by: Gage Eads <gage.eads at intel.com>
> > ---
>
> [snip..]
> > +int __rte_experimental
> > +rte_event_crypto_adapter_start(uint8_t id) {
> > + struct rte_event_crypto_adapter *adapter;
> > +
> > + RTE_EVENT_CRYPTO_ADAPTER_ID_VALID_OR_ERR_RET(id, -EINVAL);
> > + adapter = eca_id_to_adapter(id);
> > + if (adapter == NULL)
> > + return -EINVAL;
> This check is redundant here. you are null checking it again in eca_adapter_ctrl
Ok.
> > +
> > + return eca_adapter_ctrl(id, 1);
> > +}
> > +
> > +int __rte_experimental
> > +rte_event_crypto_adapter_stop(uint8_t id) {
> > + return eca_adapter_ctrl(id, 0);
> > +}
> > +
>
> [snip..]
> > diff --git a/lib/librte_eventdev/rte_eventdev_version.map
> > b/lib/librte_eventdev/rte_eventdev_version.map
> > index 3ee28f7..774f7c5 100644
> > --- a/lib/librte_eventdev/rte_eventdev_version.map
> > +++ b/lib/librte_eventdev/rte_eventdev_version.map
> > @@ -72,6 +72,18 @@ DPDK_18.02 {
> > global:
> >
> > rte_event_dev_selftest;
> > + rte_event_crypto_adapter_create_ext;
> > + rte_event_crypto_adapter_create;
> > + rte_event_crypto_adapter_free;
> > + rte_event_crypto_adapter_queue_pair_add;
> > + rte_event_crypto_adapter_queue_pair_del;
> > + rte_event_crypto_adapter_start;
> > + rte_event_crypto_adapter_stop;
> > + rte_event_crypto_adapter_stats_get;
> > + rte_event_crypto_adapter_stats_reset;
> > + rte_event_crypto_adapter_service_id_get;
> > + rte_event_crypto_adapter_event_port_get;
> > +
> I believe these shall go in EXPERIMENTAL along with timer.
Ok
> > } DPDK_17.11;
> >
> > DPDK_18.05 {
> >
More information about the dev
mailing list