Re: [dpdk-dev] [PATCH v2 09/33] crypto/octeontx: adds symmetric capabilities

2018-10-23 Thread Thomas Monjalon
23/10/2018 10:48, Joseph, Anoob: > From: Thomas Monjalon > > 22/10/2018 05:49, Joseph, Anoob: > > > Hi Fiona, > > > > > > I do agree that your solution seems to be a neat way for organizing > > capabilities. But Akhil & Thomas were against that idea and we had to come > > up > > with one array wi

Re: [dpdk-dev] [PATCH v2 09/33] crypto/octeontx: adds symmetric capabilities

2018-10-23 Thread Joseph, Anoob
gt; ; Dwivedi, Ankur > ; Dabilpuram, Nithin > ; Jayaraman, Ragothaman > ; Srinivasan, Srisivasubramanian > ; Tejasree, Kondoj > > Subject: Re: [dpdk-dev] [PATCH v2 09/33] crypto/octeontx: adds symmetric > capabilities > > External Email > > 22/10/2018 05:49, Joseph, An

Re: [dpdk-dev] [PATCH v2 09/33] crypto/octeontx: adds symmetric capabilities

2018-10-21 Thread Thomas Monjalon
22/10/2018 05:49, Joseph, Anoob: > Hi Fiona, > > I do agree that your solution seems to be a neat way for organizing > capabilities. But Akhil & Thomas were against that idea and we had to come up > with one array with all capabilities. This would not scale well when we start > supporting devic

Re: [dpdk-dev] [PATCH v2 09/33] crypto/octeontx: adds symmetric capabilities

2018-10-21 Thread Joseph, Anoob
lo ; Murthy, Nidadavolu > > ; Jacob, Jerin > > ; Athreya, Narayana Prasad > > ; Dwivedi, Ankur > > ; Dabilpuram, Nithin > > ; Jayaraman, Ragothaman > > ; Srinivasan, Srisivasubramanian > > ; Tejasree, Kondoj > > > > Subject: RE: [dpdk-dev] [PAT

Re: [dpdk-dev] [PATCH v2 09/33] crypto/octeontx: adds symmetric capabilities

2018-10-19 Thread Trahe, Fiona
; > To: Thomas Monjalon ; Trahe, Fiona > > > > Cc: dev@dpdk.org; Akhil Goyal ; Joseph, Anoob > > ; De Lara Guarch, Pablo > > ; Murthy, Nidadavolu > > ; Jacob, Jerin > > ; Athreya, Narayana Prasad > > ; Dwivedi, Ankur > > ; Dabilpuram, Nith

Re: [dpdk-dev] [PATCH v2 09/33] crypto/octeontx: adds symmetric capabilities

2018-10-16 Thread Joseph, Anoob
Jerin > ; Athreya, Narayana Prasad > ; Dwivedi, Ankur > ; Dabilpuram, Nithin > ; Jayaraman, Ragothaman > ; Srinivasan, Srisivasubramanian > ; Tejasree, Kondoj > > Subject: Re: [dpdk-dev] [PATCH v2 09/33] crypto/octeontx: adds symmetric > capabilities > > Hi Fio

Re: [dpdk-dev] [PATCH v2 09/33] crypto/octeontx: adds symmetric capabilities

2018-10-09 Thread Joseph, Anoob
Hi Fiona, We were following the QAT approach for defining the capabilities. OCTEON TX crypto PMD has similar number of capabilities and QAT was the close model that we could follow. I can see the advantages of the macro approach, but that would give a checkpatch warning. Also, Thomas didn't r

Re: [dpdk-dev] [PATCH v2 09/33] crypto/octeontx: adds symmetric capabilities

2018-10-08 Thread Thomas Monjalon
08/10/2018 17:59, Trahe, Fiona: > Hi Akhil, Joseph, Thomas, > Just spotted this now. > See below. > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > 24/09/2018 13:36, Joseph, Anoob: > > > Hi Fiona, > > > > > > Can you please comment on this? > > > > > > We are adding all capabilities of oc

Re: [dpdk-dev] [PATCH v2 09/33] crypto/octeontx: adds symmetric capabilities

2018-10-08 Thread Trahe, Fiona
De Lara Guarch, Pablo > ; > Murthy NSSR ; Jerin Jacob > ; Narayana Prasad > ; Ankur Dwivedi > ; Nithin Dabilpuram > ; Ragothaman Jayaraman > ; Srisivasubramanian S > ; > Tejasree Kondoj > Subject: Re: [dpdk-dev] [PATCH v2 09/33] crypto/octeontx: adds symmetric > cap

Re: [dpdk-dev] [PATCH v2 09/33] crypto/octeontx: adds symmetric capabilities

2018-10-01 Thread Thomas Monjalon
24/09/2018 13:36, Joseph, Anoob: > Hi Fiona, > > Can you please comment on this? > > We are adding all capabilities of octeontx-crypto PMD as a macro in > otx_cryptodev_capabilites.h file and then we are using it from > otx_cryptodev_ops.c. This is the approach followed by QAT crypto PMD. As >

Re: [dpdk-dev] [PATCH v2 09/33] crypto/octeontx: adds symmetric capabilities

2018-09-28 Thread Joseph, Anoob
Hi Fiona, Did you get a chance to look at this? Thanks, Anoob On 24-09-2018 17:06, Joseph, Anoob wrote: Hi Fiona, Can you please comment on this? We are adding all capabilities of octeontx-crypto PMD as a macro in otx_cryptodev_capabilites.h file and then we are using it from otx_cryptodev_

Re: [dpdk-dev] [PATCH v2 09/33] crypto/octeontx: adds symmetric capabilities

2018-09-24 Thread Joseph, Anoob
Hi Fiona, Can you please comment on this? We are adding all capabilities of octeontx-crypto PMD as a macro in otx_cryptodev_capabilites.h file and then we are using it from otx_cryptodev_ops.c. This is the approach followed by QAT crypto PMD. As per my understanding, this is to ensure that cr

Re: [dpdk-dev] [PATCH v2 09/33] crypto/octeontx: adds symmetric capabilities

2018-09-17 Thread Joseph, Anoob
Hi Akhil, On 17-09-2018 17:31, Akhil Goyal wrote: External Email diff --git a/drivers/crypto/octeontx/otx_cryptodev_ops.c b/drivers/crypto/octeontx/otx_cryptodev_ops.c index d25f9c1..cc0030e 100644 --- a/drivers/crypto/octeontx/otx_cryptodev_ops.c +++ b/drivers/crypto/octeontx/otx_cryptodev_o

Re: [dpdk-dev] [PATCH v2 09/33] crypto/octeontx: adds symmetric capabilities

2018-09-17 Thread Akhil Goyal
diff --git a/drivers/crypto/octeontx/otx_cryptodev_ops.c b/drivers/crypto/octeontx/otx_cryptodev_ops.c index d25f9c1..cc0030e 100644 --- a/drivers/crypto/octeontx/otx_cryptodev_ops.c +++ b/drivers/crypto/octeontx/otx_cryptodev_ops.c @@ -10,9 +10,15 @@ #include "cpt_pmd_logs.h" #include "ot

[dpdk-dev] [PATCH v2 09/33] crypto/octeontx: adds symmetric capabilities

2018-09-03 Thread Anoob Joseph
From: Murthy NSSR This patch adds the symmetric algorithms capabilities supported by octeontx crypto hardware. Signed-off-by: Ankur Dwivedi Signed-off-by: Anoob Joseph Signed-off-by: Murthy NSSR Signed-off-by: Nithin Dabilpuram Signed-off-by: Ragothaman Jayaraman Signed-off-by: Srisivasubra