Re: [dpdk-dev] [PATCH 1/2] cryptodev: add min headroom and tailroom requirement

2018-06-28 Thread Joseph, Anoob
Hi Pablo, On 28-06-2018 17:11, De Lara Guarch, Pablo wrote: External Email Hi Anoob, -Original Message- From: Joseph, Anoob [mailto:anoob.jos...@caviumnetworks.com] Sent: Thursday, June 28, 2018 3:56 AM To: Doherty, Declan ; De Lara Guarch, Pablo Cc: Akhil Goyal ; Ankur Dwivedi ; Jer

Re: [dpdk-dev] [PATCH 1/2] cryptodev: add min headroom and tailroom requirement

2018-06-28 Thread De Lara Guarch, Pablo
Hi Anoob, > -Original Message- > From: Joseph, Anoob [mailto:anoob.jos...@caviumnetworks.com] > Sent: Thursday, June 28, 2018 3:56 AM > To: Doherty, Declan ; De Lara Guarch, Pablo > > Cc: Akhil Goyal ; Ankur Dwivedi > ; Jerin Jacob > ; Narayana Prasad > ; dev@dpdk.org > Subject: Re: [PATC

Re: [dpdk-dev] [PATCH 1/2] cryptodev: add min headroom and tailroom requirement

2018-06-27 Thread Joseph, Anoob
Hi Declan, Please see inline. Thanks, Anoob On 26-06-2018 15:42, Doherty, Declan wrote: External Email On 19/06/2018 7:26 AM, Anoob Joseph wrote: Enabling crypto devs to specify the minimum headroom and tailroom it expects in the mbuf. For net PMDs, standard headroom has to be honoured by

Re: [dpdk-dev] [PATCH 1/2] cryptodev: add min headroom and tailroom requirement

2018-06-26 Thread Doherty, Declan
On 19/06/2018 7:26 AM, Anoob Joseph wrote: Enabling crypto devs to specify the minimum headroom and tailroom it expects in the mbuf. For net PMDs, standard headroom has to be honoured by applications, which is not strictly followed for crypto devs. This How is this done for NET PMDs, I don't se

Re: [dpdk-dev] [PATCH 1/2] cryptodev: add min headroom and tailroom requirement

2018-06-22 Thread Akhil Goyal
Hi Anoob, On 6/22/2018 12:22 PM, Joseph, Anoob wrote: Hi Akhil, The change adds couple of entries in cryptodev_info structure. I was under the impression this will be an ABI change and would need one release cycle. If we can get this through in 18.08, it would be great. Do you want me to re

Re: [dpdk-dev] [PATCH 1/2] cryptodev: add min headroom and tailroom requirement

2018-06-21 Thread Joseph, Anoob
Hi Akhil, The change adds couple of entries in cryptodev_info structure. I was under the impression this will be an ABI change and would need one release cycle. If we can get this through in 18.08, it would be great. Do you want me to revise the patch stating it that way? Thanks, Anoob On 21

Re: [dpdk-dev] [PATCH 1/2] cryptodev: add min headroom and tailroom requirement

2018-06-21 Thread Akhil Goyal
On 6/19/2018 11:56 AM, Anoob Joseph wrote: Enabling crypto devs to specify the minimum headroom and tailroom it expects in the mbuf. For net PMDs, standard headroom has to be honoured by applications, which is not strictly followed for crypto devs. This prevents crypto devs from using free spa