-----Original Message----- > Date: Thu, 3 May 2018 06:03:01 +0000 > From: "Gujjar, Abhinandan S" <[email protected]> > To: Jerin Jacob <[email protected]> > CC: "[email protected]" <[email protected]>, > "[email protected]" <[email protected]>, "[email protected]" > <[email protected]>, "Vangati, Narender" <[email protected]>, "Rao, > Nikhil" <[email protected]>, "Eads, Gage" <[email protected]> > Subject: RE: [v2,1/6] eventdev: introduce event crypto adapter > > > > > -----Original Message----- > > From: Jerin Jacob <[email protected]> > > Sent: Sunday, April 29, 2018 9:39 PM > > To: Gujjar, Abhinandan S <[email protected]> > > Cc: [email protected]; [email protected]; [email protected]; Vangati, > > Narender <[email protected]>; Rao, Nikhil <[email protected]>; > > Eads, Gage <[email protected]> > > Subject: Re: [v2,1/6] eventdev: introduce event crypto adapter > > > > -----Original Message----- > > > Date: Tue, 24 Apr 2018 18:13:22 +0530 > > > From: Abhinandan Gujjar <[email protected]> > > > To: [email protected], [email protected], > > > [email protected], [email protected] > > > CC: [email protected], [email protected], > > > [email protected], [email protected] > > > Subject: [v2,1/6] eventdev: introduce event crypto adapter > > > X-Mailer: git-send-email 1.9.1 > > > > > > Signed-off-by: Abhinandan Gujjar <[email protected]> > > > Signed-off-by: Nikhil Rao <[email protected]> > > > Signed-off-by: Gage Eads <[email protected]> > > > --- > > > lib/librte_eventdev/rte_event_crypto_adapter.h | 532 > > > +++++++++++++++++++++++++ > > > 1 file changed, 532 insertions(+) > > > create mode 100644 lib/librte_eventdev/rte_event_crypto_adapter.h > > > > > > diff --git a/lib/librte_eventdev/rte_event_crypto_adapter.h > > > b/lib/librte_eventdev/rte_event_crypto_adapter.h > > > new file mode 100644 > > > index 0000000..aa4f32c > > > --- /dev/null > > > +++ b/lib/librte_eventdev/rte_event_crypto_adapter.h > > > @@ -0,0 +1,532 @@ > > > +/* SPDX-License-Identifier: BSD-3-Clause > > > + * Copyright(c) 2017-2018 Intel Corporation */ > > > + > > > +#ifndef _RTE_EVENT_CRYPTO_ADAPTER_ > > > +#define _RTE_EVENT_CRYPTO_ADAPTER_ > > > > Please reword if it makes sense. > It is to the adapter through eventdev. > May be elaborating little more something like, application gets crypto > adapter's > event port by rte_event_crypto_adapter_event_port_get() API. > Application links it's event queue to this event port and starts enqueuing > crypto > operations as events. Adapter dequeue these events and submit the crypto > operations > to the cryptodev. > > Does this make sense?
Yes > > > + RTE_EVENT_CRYPTO_ADAPTER_DEQ_ONLY = 1, > > > > Why to mark it as explicit '1' ? > Nothing specific. Have 0 & 1? Let have 0 then so that enum you don't need to specify '0' explicit as enum starts from 0 Thanks for the comments. Looks like we have sorted out all the issues.
