Re: [dpdk-dev] [PATCH] doc: announce ABI change for cryptodev config

2019-02-28 Thread Anoob Joseph
gt; ; Jerin Jacob Kollanukkaran > ; Narayana Prasad Raju Athreya > ; Shally Verma ; Doherty, > Declan > Subject: Re: [dpdk-dev] [PATCH] doc: announce ABI change for cryptodev > config > > Hi Anoob, > > On 2/1/2019 5:19 PM, Trahe, Fiona wrote: > > Hi Thomas, Akh

Re: [dpdk-dev] [PATCH] doc: announce ABI change for cryptodev config

2019-02-28 Thread Akhil Goyal
b Joseph ; De >> Lara Guarch, Pablo >> ; Trahe, Fiona ; >> Jerin Jacob Kollanukkaran >> ; Narayana Prasad Raju Athreya ; >> Shally Verma >> ; Doherty, Declan >> Subject: Re: [dpdk-dev] [PATCH] doc: announce ABI change for cryptodev config >> >> There

Re: [dpdk-dev] [PATCH] doc: announce ABI change for cryptodev config

2019-02-01 Thread Trahe, Fiona
aran > ; Narayana Prasad Raju Athreya ; > Shally Verma > ; Doherty, Declan > Subject: Re: [dpdk-dev] [PATCH] doc: announce ABI change for cryptodev config > > There is only one ack for this change. > A deprecation requires more agreement (3 valuable acks). > Other opinions?

Re: [dpdk-dev] [PATCH] doc: announce ABI change for cryptodev config

2019-02-01 Thread Thomas Monjalon
There is only one ack for this change. A deprecation requires more agreement (3 valuable acks). Other opinions? 31/01/2019 10:53, Akhil Goyal: > On 1/17/2019 3:09 PM, Anoob Joseph wrote: > > Add new field ff_enable in rte_cryptodev_config. This enables > > applications to control the features ena

Re: [dpdk-dev] [PATCH] doc: announce ABI change for cryptodev config

2019-01-31 Thread Akhil Goyal
On 1/17/2019 3:09 PM, Anoob Joseph wrote: > Add new field ff_enable in rte_cryptodev_config. This enables > applications to control the features enabled on the crypto device. > > Proposed new layout: > > /** Crypto device configuration structure */ > struct rte_cryptodev_config { > int socke

Re: [dpdk-dev] [PATCH] doc: announce ABI change for cryptodev config

2019-01-22 Thread Anoob Joseph
Hi Fiona, Any more comments on this? @ Akhil, Pablo Can you review this change and share your thoughts? Thanks, Anoob > -Original Message- > From: Shally Verma > Sent: 18 January 2019 12:29 > To: Anoob Joseph ; Trahe, Fiona > ; Akhil Goyal ; De Lara Guarch, > Pablo > Cc: Jerin Jacob K

Re: [dpdk-dev] [PATCH] doc: announce ABI change for cryptodev config

2019-01-18 Thread Shally Verma
HI Fiona, Anoob >-Original Message- >From: Anoob Joseph >Sent: 17 January 2019 19:17 >To: Trahe, Fiona ; Akhil Goyal ; >De Lara Guarch, Pablo > >Cc: Jerin Jacob Kollanukkaran ; Narayana Prasad Raju >Athreya ; Shally Verma >; dev@dpdk.org >Subject: RE: [PATCH] doc: announce ABI change for

Re: [dpdk-dev] [PATCH] doc: announce ABI change for cryptodev config

2019-01-17 Thread Anoob Joseph
Hi Fiona, Please see inline. Thanks, Anoob > -Original Message- > From: Trahe, Fiona > Sent: 17 January 2019 17:07 > To: Anoob Joseph ; Akhil Goyal ; > De Lara Guarch, Pablo > Cc: Jerin Jacob Kollanukkaran ; Narayana Prasad Raju > Athreya ; Shally Verma ; > dev@dpdk.org; Trahe, Fiona

Re: [dpdk-dev] [PATCH] doc: announce ABI change for cryptodev config

2019-01-17 Thread Trahe, Fiona
Hi Joseph, > -Original Message- > From: Anoob Joseph [mailto:ano...@marvell.com] > Sent: Thursday, January 17, 2019 9:40 AM > To: Akhil Goyal ; De Lara Guarch, Pablo > ; Trahe, > Fiona > Cc: Anoob Joseph ; Jerin Jacob Kollanukkaran > ; Narayana > Prasad Raju Athreya ; Shally Verma > ;

[dpdk-dev] [PATCH] doc: announce ABI change for cryptodev config

2019-01-17 Thread Anoob Joseph
Add new field ff_enable in rte_cryptodev_config. This enables applications to control the features enabled on the crypto device. Proposed new layout: /** Crypto device configuration structure */ struct rte_cryptodev_config { int socket_id;/**< Socket to allocate resources on */