mbox series

[v2,0/5] Add session-less, RSA, RSA-CRT to QAT

Message ID 20191011100339.12912-1-arkadiuszx.kusztal@intel.com (mailing list archive)
Headers
Series Add session-less, RSA, RSA-CRT to QAT |

Message

Arkadiusz Kusztal Oct. 11, 2019, 10:03 a.m. UTC
  This patchset adds session-less option, RSA algorithm, RSA-CRT algorithm
to Intel QuickAssist Technology PMD.

This patchset depends on the
[v5] cryptodev: extend api of asymmetric crypto by session-less [1]

[1] http://patchwork.dpdk.org/patch/60882/

v2:
- fixed alg size problem in crt
- fixed rsa capabilities
- added rsa test cases

Arek Kusztal (5):
  crypto/qat: add sessionless implementation to asym pmd
  crypto/qat: add rsa implementation to asym pmd
  crypto/qat: add rsa crt implementation to asym pmd
  test/crypto: add sessionless to asymmetric mod exp
  test/crypto: add rsa tests to qat

 app/test/test_cryptodev_asym.c                     | 182 ++++--
 app/test/test_cryptodev_rsa_test_vectors.h         | 213 +++++++
 doc/guides/cryptodevs/features/qat.ini             |   4 +
 doc/guides/rel_notes/release_19_11.rst             |   8 +
 .../qat/qat_adf/qat_pke_functionality_arrays.h     |  27 +
 drivers/crypto/qat/qat_asym.c                      | 616 ++++++++++++++++-----
 drivers/crypto/qat/qat_asym.h                      |  29 +-
 drivers/crypto/qat/qat_asym_capabilities.h         |  21 +
 drivers/crypto/qat/qat_asym_pmd.c                  |   5 +-
 9 files changed, 925 insertions(+), 180 deletions(-)
  

Comments

Akhil Goyal Oct. 15, 2019, 10:22 a.m. UTC | #1
Hi Arek,
> 
> This patchset adds session-less option, RSA algorithm, RSA-CRT algorithm
> to Intel QuickAssist Technology PMD.
> 
> This patchset depends on the
> [v5] cryptodev: extend api of asymmetric crypto by session-less [1]
> 
> [1]
> http://patchwork.dpdk.org/patch/60882/
> 
Could you please rebase this patch so that compilation can be tested by CI.
The dependent patch is merged.
Also please update the patch titles as below.

@fiona.trahe@intel.com: could you please review this series and provide Ack

> v2:
> - fixed alg size problem in crt
> - fixed rsa capabilities
> - added rsa test cases
> 
> Arek Kusztal (5):
>   crypto/qat: add sessionless implementation to asym pmd
crypto/qat: support asymmetric sessionless ops

>   crypto/qat: add rsa implementation to asym pmd
crypto/qat: support RSA
>   crypto/qat: add rsa crt implementation to asym pmd
crypto/qat: support RSA CRT
>   test/crypto: add sessionless to asymmetric mod exp
test/crypto: support sessionless in asymmetric cases
>   test/crypto: add rsa tests to qat
test/crypto: add RSA cases in QAT

Also please add more meaningful patch description.

> 
>  app/test/test_cryptodev_asym.c                     | 182 ++++--
>  app/test/test_cryptodev_rsa_test_vectors.h         | 213 +++++++
>  doc/guides/cryptodevs/features/qat.ini             |   4 +
>  doc/guides/rel_notes/release_19_11.rst             |   8 +
>  .../qat/qat_adf/qat_pke_functionality_arrays.h     |  27 +
>  drivers/crypto/qat/qat_asym.c                      | 616 ++++++++++++++++-----
>  drivers/crypto/qat/qat_asym.h                      |  29 +-
>  drivers/crypto/qat/qat_asym_capabilities.h         |  21 +
>  drivers/crypto/qat/qat_asym_pmd.c                  |   5 +-
>  9 files changed, 925 insertions(+), 180 deletions(-)
> 
> --
> 2.1.0
  
Fiona Trahe Oct. 17, 2019, 9:27 a.m. UTC | #2
Hi Akhil,
Arek will add RSA signature tests with padding-none option.
I will review and ack after this
Fiona

> -----Original Message-----
> From: Akhil Goyal [mailto:akhil.goyal@nxp.com]
> Sent: Tuesday, October 15, 2019 11:23 AM
> To: Kusztal, ArkadiuszX <arkadiuszx.kusztal@intel.com>; dev@dpdk.org; Trahe, Fiona
> <fiona.trahe@intel.com>
> Subject: RE: [PATCH v2 0/5] Add session-less, RSA, RSA-CRT to QAT
> 
> Hi Arek,
> >
> > This patchset adds session-less option, RSA algorithm, RSA-CRT algorithm
> > to Intel QuickAssist Technology PMD.
> >
> > This patchset depends on the
> > [v5] cryptodev: extend api of asymmetric crypto by session-less [1]
> >
> > [1]
> > http://patchwork.dpdk.org/patch/60882/
> >
> Could you please rebase this patch so that compilation can be tested by CI.
> The dependent patch is merged.
> Also please update the patch titles as below.
> 
> @fiona.trahe@intel.com: could you please review this series and provide Ack
> 
> > v2:
> > - fixed alg size problem in crt
> > - fixed rsa capabilities
> > - added rsa test cases
> >
> > Arek Kusztal (5):
> >   crypto/qat: add sessionless implementation to asym pmd
> crypto/qat: support asymmetric sessionless ops
> 
> >   crypto/qat: add rsa implementation to asym pmd
> crypto/qat: support RSA
> >   crypto/qat: add rsa crt implementation to asym pmd
> crypto/qat: support RSA CRT
> >   test/crypto: add sessionless to asymmetric mod exp
> test/crypto: support sessionless in asymmetric cases
> >   test/crypto: add rsa tests to qat
> test/crypto: add RSA cases in QAT
> 
> Also please add more meaningful patch description.
> 
> >
> >  app/test/test_cryptodev_asym.c                     | 182 ++++--
> >  app/test/test_cryptodev_rsa_test_vectors.h         | 213 +++++++
> >  doc/guides/cryptodevs/features/qat.ini             |   4 +
> >  doc/guides/rel_notes/release_19_11.rst             |   8 +
> >  .../qat/qat_adf/qat_pke_functionality_arrays.h     |  27 +
> >  drivers/crypto/qat/qat_asym.c                      | 616 ++++++++++++++++-----
> >  drivers/crypto/qat/qat_asym.h                      |  29 +-
> >  drivers/crypto/qat/qat_asym_capabilities.h         |  21 +
> >  drivers/crypto/qat/qat_asym_pmd.c                  |   5 +-
> >  9 files changed, 925 insertions(+), 180 deletions(-)
> >
> > --
> > 2.1.0
  
Fiona Trahe Oct. 21, 2019, 10:47 a.m. UTC | #3
Hi Akhil,
There are also some minor changes needed in QAT asymmetric PMD patches - Arek plans to send 
a v3 later today with the RSA signature tests and those QAT changes.
Fiona

> -----Original Message-----
> From: Trahe, Fiona
> Sent: Thursday, October 17, 2019 10:27 AM
> To: Akhil Goyal <akhil.goyal@nxp.com>; Kusztal, ArkadiuszX <arkadiuszx.kusztal@intel.com>;
> dev@dpdk.org
> Cc: Trahe, Fiona <fiona.trahe@intel.com>
> Subject: RE: [PATCH v2 0/5] Add session-less, RSA, RSA-CRT to QAT
> 
> Hi Akhil,
> Arek will add RSA signature tests with padding-none option.
> I will review and ack after this
> Fiona
> 
> > -----Original Message-----
> > From: Akhil Goyal [mailto:akhil.goyal@nxp.com]
> > Sent: Tuesday, October 15, 2019 11:23 AM
> > To: Kusztal, ArkadiuszX <arkadiuszx.kusztal@intel.com>; dev@dpdk.org; Trahe, Fiona
> > <fiona.trahe@intel.com>
> > Subject: RE: [PATCH v2 0/5] Add session-less, RSA, RSA-CRT to QAT
> >
> > Hi Arek,
> > >
> > > This patchset adds session-less option, RSA algorithm, RSA-CRT algorithm
> > > to Intel QuickAssist Technology PMD.
> > >
> > > This patchset depends on the
> > > [v5] cryptodev: extend api of asymmetric crypto by session-less [1]
> > >
> > > [1]
> > > http://patchwork.dpdk.org/patch/60882/
> > >
> > Could you please rebase this patch so that compilation can be tested by CI.
> > The dependent patch is merged.
> > Also please update the patch titles as below.
> >
> > @fiona.trahe@intel.com: could you please review this series and provide Ack
> >
> > > v2:
> > > - fixed alg size problem in crt
> > > - fixed rsa capabilities
> > > - added rsa test cases
> > >
> > > Arek Kusztal (5):
> > >   crypto/qat: add sessionless implementation to asym pmd
> > crypto/qat: support asymmetric sessionless ops
> >
> > >   crypto/qat: add rsa implementation to asym pmd
> > crypto/qat: support RSA
> > >   crypto/qat: add rsa crt implementation to asym pmd
> > crypto/qat: support RSA CRT
> > >   test/crypto: add sessionless to asymmetric mod exp
> > test/crypto: support sessionless in asymmetric cases
> > >   test/crypto: add rsa tests to qat
> > test/crypto: add RSA cases in QAT
> >
> > Also please add more meaningful patch description.
> >
> > >
> > >  app/test/test_cryptodev_asym.c                     | 182 ++++--
> > >  app/test/test_cryptodev_rsa_test_vectors.h         | 213 +++++++
> > >  doc/guides/cryptodevs/features/qat.ini             |   4 +
> > >  doc/guides/rel_notes/release_19_11.rst             |   8 +
> > >  .../qat/qat_adf/qat_pke_functionality_arrays.h     |  27 +
> > >  drivers/crypto/qat/qat_asym.c                      | 616 ++++++++++++++++-----
> > >  drivers/crypto/qat/qat_asym.h                      |  29 +-
> > >  drivers/crypto/qat/qat_asym_capabilities.h         |  21 +
> > >  drivers/crypto/qat/qat_asym_pmd.c                  |   5 +-
> > >  9 files changed, 925 insertions(+), 180 deletions(-)
> > >
> > > --
> > > 2.1.0