mbox series

[v4,0/2] crypto/scheduler: add support for DOCSIS security protocol

Message ID 20230919141440.39305-1-david.coyle@intel.com (mailing list archive)
Headers
Series crypto/scheduler: add support for DOCSIS security protocol |

Message

Coyle, David Sept. 19, 2023, 2:14 p.m. UTC 
  This patchset adds support to the cryptodev scheduler PMD and unit
tests for the DOCSIS security protocol. Support for other stateless
security protocols can be added in the future as required with relative
ease now that the framework is in place.

v4:
* Address review comments from Anoob
  * Remove unnecessary pointer type casting
  * Free capability and worker name memory if memory allocation fails
    during scheduler PMD creation
  * Use RTE_MAX when getting max worker session size

v3:
* Limit support to the DOCSIS security protocol only

v2:
* Improve inclusion of rte_security header files
* Fix typo in commit message

Series-acked-by: Ciara Power <ciara.power@intel.com>

David Coyle (2):
  crypto/scheduler: support DOCSIS security protocol
  test/crypto: add DOCSIS security tests for cryptodev scheduler

 app/test/test_cryptodev.c                     |  12 +-
 doc/guides/rel_notes/release_23_11.rst        |   4 +
 drivers/crypto/scheduler/meson.build          |   2 +-
 .../scheduler/rte_cryptodev_scheduler.c       | 218 ++++++++++-
 drivers/crypto/scheduler/scheduler_failover.c |  12 +-
 .../crypto/scheduler/scheduler_multicore.c    |  10 +-
 .../scheduler/scheduler_pkt_size_distr.c      |  54 +--
 drivers/crypto/scheduler/scheduler_pmd.c      |  59 ++-
 drivers/crypto/scheduler/scheduler_pmd_ops.c  | 370 +++++++++++++-----
 .../crypto/scheduler/scheduler_pmd_private.h  | 155 +++++---
 .../crypto/scheduler/scheduler_roundrobin.c   |   6 +-
 11 files changed, 668 insertions(+), 234 deletions(-)

Comments

Anoob Joseph Sept. 20, 2023, 5:45 a.m. UTC | #1 
> 
> ----------------------------------------------------------------------
> This patchset adds support to the cryptodev scheduler PMD and unit tests
> for the DOCSIS security protocol. Support for other stateless security
> protocols can be added in the future as required with relative ease now that
> the framework is in place.
> 
> v4:
> * Address review comments from Anoob
>   * Remove unnecessary pointer type casting
>   * Free capability and worker name memory if memory allocation fails
>     during scheduler PMD creation
>   * Use RTE_MAX when getting max worker session size
> 
> v3:
> * Limit support to the DOCSIS security protocol only
> 
> v2:
> * Improve inclusion of rte_security header files
> * Fix typo in commit message
> 
> Series-acked-by: Ciara Power <ciara.power@intel.com>
> 
> David Coyle (2):
>   crypto/scheduler: support DOCSIS security protocol
>   test/crypto: add DOCSIS security tests for cryptodev scheduler
> 
>  app/test/test_cryptodev.c                     |  12 +-
>  doc/guides/rel_notes/release_23_11.rst        |   4 +
>  drivers/crypto/scheduler/meson.build          |   2 +-
>  .../scheduler/rte_cryptodev_scheduler.c       | 218 ++++++++++-
>  drivers/crypto/scheduler/scheduler_failover.c |  12 +-
>  .../crypto/scheduler/scheduler_multicore.c    |  10 +-
>  .../scheduler/scheduler_pkt_size_distr.c      |  54 +--
>  drivers/crypto/scheduler/scheduler_pmd.c      |  59 ++-
>  drivers/crypto/scheduler/scheduler_pmd_ops.c  | 370 +++++++++++++-----
> .../crypto/scheduler/scheduler_pmd_private.h  | 155 +++++---
>  .../crypto/scheduler/scheduler_roundrobin.c   |   6 +-
>  11 files changed, 668 insertions(+), 234 deletions(-)
> 
> --

Series Acked-by: Anoob Joseph <anoobj@marvell.com>
Akhil Goyal Sept. 21, 2023, 7:18 a.m. UTC | #2 
> > This patchset adds support to the cryptodev scheduler PMD and unit tests
> > for the DOCSIS security protocol. Support for other stateless security
> > protocols can be added in the future as required with relative ease now that
> > the framework is in place.
> >
> > v4:
> > * Address review comments from Anoob
> >   * Remove unnecessary pointer type casting
> >   * Free capability and worker name memory if memory allocation fails
> >     during scheduler PMD creation
> >   * Use RTE_MAX when getting max worker session size
> >
> > v3:
> > * Limit support to the DOCSIS security protocol only
> >
> > v2:
> > * Improve inclusion of rte_security header files
> > * Fix typo in commit message
> >
> > Series-acked-by: Ciara Power <ciara.power@intel.com>
> >
> > David Coyle (2):
> >   crypto/scheduler: support DOCSIS security protocol
> >   test/crypto: add DOCSIS security tests for cryptodev scheduler
> >
> >  app/test/test_cryptodev.c                     |  12 +-
> >  doc/guides/rel_notes/release_23_11.rst        |   4 +
> >  drivers/crypto/scheduler/meson.build          |   2 +-
> >  .../scheduler/rte_cryptodev_scheduler.c       | 218 ++++++++++-
> >  drivers/crypto/scheduler/scheduler_failover.c |  12 +-
> >  .../crypto/scheduler/scheduler_multicore.c    |  10 +-
> >  .../scheduler/scheduler_pkt_size_distr.c      |  54 +--
> >  drivers/crypto/scheduler/scheduler_pmd.c      |  59 ++-
> >  drivers/crypto/scheduler/scheduler_pmd_ops.c  | 370 +++++++++++++-----
> > .../crypto/scheduler/scheduler_pmd_private.h  | 155 +++++---
> >  .../crypto/scheduler/scheduler_roundrobin.c   |   6 +-
> >  11 files changed, 668 insertions(+), 234 deletions(-)
> >
> > --
> 
> Series Acked-by: Anoob Joseph <anoobj@marvell.com>

Series Applied to dpdk-next-crypto

Thanks.