mbox series

[v2,00/11] crypto: qat - add rate limiting feature to qat_4xxx

Message ID 20231020134931.7530-1-damian.muszynski@intel.com
Headers show
Series crypto: qat - add rate limiting feature to qat_4xxx | expand

Message

Damian Muszynski Oct. 20, 2023, 1:49 p.m. UTC
This set enables hardware rate limiting capabilities on QAT 4xxx
accelerators. Rate Limiting allows to control the rate of the requests
that can be submitted to a ring pair (RP). This allows sharing
a QAT device among multiple users while ensuring guaranteed throughput.

The driver provides a mechanism that allows users to set policies, through
a sysfs interface, that are then programmed to the device. The device is
then enforcing the policies.

The first six commits are refactoring and additions in preparation for
  the feature.
Patch #6 introduces a mechanism for retrieving firmware feature
  capabilities.
Patch #8 implements the core of the rate limiting feature by providing
  mechanisms to set rate limiting policies (aka SLAs).
The final three commits add the required sysfs interface that allow
  users to configure SLAs.

Changes since v1:
- Removed unnecessary check
- Simplified a few error paths
- Reduced a few local variables
- Fixed repeated error message
- Moved mutex lock above sla_id existence check.
- Added Reviewed-by tag from Tero Kristo received from an internal review
  of the set.

Ciunas Bennett (3):
  crypto: qat - add rate limiting sysfs interface
  crypto: qat - add rp2svc sysfs attribute
  crypto: qat - add num_rps sysfs attribute

Damian Muszynski (4):
  units: Add BYTES_PER_*BIT
  crypto: qat - add bits.h to icp_qat_hw.h
  crypto: qat - add retrieval of fw capabilities
  crypto: qat - add rate limiting feature to qat_4xxx

Giovanni Cabiddu (4):
  crypto: qat - refactor fw config related functions
  crypto: qat - use masks for AE groups
  crypto: qat - fix ring to service map for QAT GEN4
  crypto: qat - move admin api

 Documentation/ABI/testing/sysfs-driver-qat    |   46 +
 Documentation/ABI/testing/sysfs-driver-qat_rl |  227 ++++
 .../intel/qat/qat_4xxx/adf_4xxx_hw_data.c     |  190 ++-
 .../intel/qat/qat_4xxx/adf_4xxx_hw_data.h     |   13 +-
 .../intel/qat/qat_c3xxx/adf_c3xxx_hw_data.c   |    1 +
 .../intel/qat/qat_c62x/adf_c62x_hw_data.c     |    1 +
 drivers/crypto/intel/qat/qat_common/Makefile  |    3 +
 .../intel/qat/qat_common/adf_accel_devices.h  |   11 +
 .../crypto/intel/qat/qat_common/adf_admin.c   |   71 +
 .../crypto/intel/qat/qat_common/adf_admin.h   |   27 +
 .../crypto/intel/qat/qat_common/adf_clock.c   |    1 +
 .../intel/qat/qat_common/adf_cnv_dbgfs.c      |    1 +
 .../intel/qat/qat_common/adf_common_drv.h     |   10 -
 .../intel/qat/qat_common/adf_fw_counters.c    |    1 +
 .../intel/qat/qat_common/adf_gen4_hw_data.h   |    7 +
 .../crypto/intel/qat/qat_common/adf_gen4_pm.c |    1 +
 .../qat/qat_common/adf_gen4_pm_debugfs.c      |    1 +
 .../intel/qat/qat_common/adf_gen4_timer.c     |    1 +
 .../intel/qat/qat_common/adf_heartbeat.c      |    1 +
 .../qat/qat_common/adf_heartbeat_dbgfs.c      |    1 +
 .../crypto/intel/qat/qat_common/adf_init.c    |   13 +
 drivers/crypto/intel/qat/qat_common/adf_rl.c  | 1169 +++++++++++++++++
 drivers/crypto/intel/qat/qat_common/adf_rl.h  |  176 +++
 .../intel/qat/qat_common/adf_rl_admin.c       |   97 ++
 .../intel/qat/qat_common/adf_rl_admin.h       |   18 +
 .../crypto/intel/qat/qat_common/adf_sysfs.c   |   80 ++
 .../intel/qat/qat_common/adf_sysfs_rl.c       |  451 +++++++
 .../intel/qat/qat_common/adf_sysfs_rl.h       |   11 +
 .../qat/qat_common/icp_qat_fw_init_admin.h    |   41 +
 .../crypto/intel/qat/qat_common/icp_qat_hw.h  |    2 +
 .../qat/qat_dh895xcc/adf_dh895xcc_hw_data.c   |    1 +
 include/linux/units.h                         |    4 +
 32 files changed, 2605 insertions(+), 73 deletions(-)
 create mode 100644 Documentation/ABI/testing/sysfs-driver-qat_rl
 create mode 100644 drivers/crypto/intel/qat/qat_common/adf_admin.h
 create mode 100644 drivers/crypto/intel/qat/qat_common/adf_rl.c
 create mode 100644 drivers/crypto/intel/qat/qat_common/adf_rl.h
 create mode 100644 drivers/crypto/intel/qat/qat_common/adf_rl_admin.c
 create mode 100644 drivers/crypto/intel/qat/qat_common/adf_rl_admin.h
 create mode 100644 drivers/crypto/intel/qat/qat_common/adf_sysfs_rl.c
 create mode 100644 drivers/crypto/intel/qat/qat_common/adf_sysfs_rl.h


base-commit: 1bb03421eab67940b6509fe0869ff43df5fbe3e6

Comments

Herbert Xu Oct. 27, 2023, 10:54 a.m. UTC | #1
On Fri, Oct 20, 2023 at 03:49:20PM +0200, Damian Muszynski wrote:
> This set enables hardware rate limiting capabilities on QAT 4xxx
> accelerators. Rate Limiting allows to control the rate of the requests
> that can be submitted to a ring pair (RP). This allows sharing
> a QAT device among multiple users while ensuring guaranteed throughput.
> 
> The driver provides a mechanism that allows users to set policies, through
> a sysfs interface, that are then programmed to the device. The device is
> then enforcing the policies.
> 
> The first six commits are refactoring and additions in preparation for
>   the feature.
> Patch #6 introduces a mechanism for retrieving firmware feature
>   capabilities.
> Patch #8 implements the core of the rate limiting feature by providing
>   mechanisms to set rate limiting policies (aka SLAs).
> The final three commits add the required sysfs interface that allow
>   users to configure SLAs.
> 
> Changes since v1:
> - Removed unnecessary check
> - Simplified a few error paths
> - Reduced a few local variables
> - Fixed repeated error message
> - Moved mutex lock above sla_id existence check.
> - Added Reviewed-by tag from Tero Kristo received from an internal review
>   of the set.
> 
> Ciunas Bennett (3):
>   crypto: qat - add rate limiting sysfs interface
>   crypto: qat - add rp2svc sysfs attribute
>   crypto: qat - add num_rps sysfs attribute
> 
> Damian Muszynski (4):
>   units: Add BYTES_PER_*BIT
>   crypto: qat - add bits.h to icp_qat_hw.h
>   crypto: qat - add retrieval of fw capabilities
>   crypto: qat - add rate limiting feature to qat_4xxx
> 
> Giovanni Cabiddu (4):
>   crypto: qat - refactor fw config related functions
>   crypto: qat - use masks for AE groups
>   crypto: qat - fix ring to service map for QAT GEN4
>   crypto: qat - move admin api
> 
>  Documentation/ABI/testing/sysfs-driver-qat    |   46 +
>  Documentation/ABI/testing/sysfs-driver-qat_rl |  227 ++++
>  .../intel/qat/qat_4xxx/adf_4xxx_hw_data.c     |  190 ++-
>  .../intel/qat/qat_4xxx/adf_4xxx_hw_data.h     |   13 +-
>  .../intel/qat/qat_c3xxx/adf_c3xxx_hw_data.c   |    1 +
>  .../intel/qat/qat_c62x/adf_c62x_hw_data.c     |    1 +
>  drivers/crypto/intel/qat/qat_common/Makefile  |    3 +
>  .../intel/qat/qat_common/adf_accel_devices.h  |   11 +
>  .../crypto/intel/qat/qat_common/adf_admin.c   |   71 +
>  .../crypto/intel/qat/qat_common/adf_admin.h   |   27 +
>  .../crypto/intel/qat/qat_common/adf_clock.c   |    1 +
>  .../intel/qat/qat_common/adf_cnv_dbgfs.c      |    1 +
>  .../intel/qat/qat_common/adf_common_drv.h     |   10 -
>  .../intel/qat/qat_common/adf_fw_counters.c    |    1 +
>  .../intel/qat/qat_common/adf_gen4_hw_data.h   |    7 +
>  .../crypto/intel/qat/qat_common/adf_gen4_pm.c |    1 +
>  .../qat/qat_common/adf_gen4_pm_debugfs.c      |    1 +
>  .../intel/qat/qat_common/adf_gen4_timer.c     |    1 +
>  .../intel/qat/qat_common/adf_heartbeat.c      |    1 +
>  .../qat/qat_common/adf_heartbeat_dbgfs.c      |    1 +
>  .../crypto/intel/qat/qat_common/adf_init.c    |   13 +
>  drivers/crypto/intel/qat/qat_common/adf_rl.c  | 1169 +++++++++++++++++
>  drivers/crypto/intel/qat/qat_common/adf_rl.h  |  176 +++
>  .../intel/qat/qat_common/adf_rl_admin.c       |   97 ++
>  .../intel/qat/qat_common/adf_rl_admin.h       |   18 +
>  .../crypto/intel/qat/qat_common/adf_sysfs.c   |   80 ++
>  .../intel/qat/qat_common/adf_sysfs_rl.c       |  451 +++++++
>  .../intel/qat/qat_common/adf_sysfs_rl.h       |   11 +
>  .../qat/qat_common/icp_qat_fw_init_admin.h    |   41 +
>  .../crypto/intel/qat/qat_common/icp_qat_hw.h  |    2 +
>  .../qat/qat_dh895xcc/adf_dh895xcc_hw_data.c   |    1 +
>  include/linux/units.h                         |    4 +
>  32 files changed, 2605 insertions(+), 73 deletions(-)
>  create mode 100644 Documentation/ABI/testing/sysfs-driver-qat_rl
>  create mode 100644 drivers/crypto/intel/qat/qat_common/adf_admin.h
>  create mode 100644 drivers/crypto/intel/qat/qat_common/adf_rl.c
>  create mode 100644 drivers/crypto/intel/qat/qat_common/adf_rl.h
>  create mode 100644 drivers/crypto/intel/qat/qat_common/adf_rl_admin.c
>  create mode 100644 drivers/crypto/intel/qat/qat_common/adf_rl_admin.h
>  create mode 100644 drivers/crypto/intel/qat/qat_common/adf_sysfs_rl.c
>  create mode 100644 drivers/crypto/intel/qat/qat_common/adf_sysfs_rl.h
> 
> 
> base-commit: 1bb03421eab67940b6509fe0869ff43df5fbe3e6
> -- 
> 2.34.1

All applied.  Thanks.