mbox series

[v2,0/4] Add module parameters to control CCP activation

Message ID 156268468861.18577.13211913750250195885.stgit@sosrh3.amd.com (mailing list archive)
Headers show
Series Add module parameters to control CCP activation | expand

Message

Gary R Hook July 9, 2019, 3:07 p.m. UTC
Firstly, add a switch to allow/disallow debugfs code to be built into
the CCP driver.

This rest of the patch series implements a set of module parameters
that allows control over which CCPs on a system are enabled by the
driver, and how many queues on each device are activated.

A switch to enable/disable DMA engine registration is implemented.

Details:
nqueues - configure N queues per CCP (default: 0 - all queues enabled)
max_devs - maximum number of devices to enable (default: 0 - all
           devices activated)
dmaengine - Register services with the DMA subsystem (default: true)

Only activated devices will have their DMA services registered,
comprehensively controlled by the dmaengine parameter.

Changes since v1:
 - Remove debugfs patches that duplicates sysfs function
 - Remove patches for filtering by pcibus and pci device ID
 - Utilize underscores for consistency in variable names
 - Correct commit message for nqueues regarding default value
 - Alter verbage of parameter description (dmaengine)
 - Help text in Kconfig: remove reference to parameters in debugfs

---

Gary R Hook (4):
      crypto: ccp - Make CCP debugfs support optional
      crypto: ccp - Add a module parameter to specify a queue count
      crypto: ccp - module parameter to limit the number of enabled CCPs
      crypto: ccp - Add a module parameter to control registration for DMA


 drivers/crypto/ccp/Kconfig         |    8 ++++++++
 drivers/crypto/ccp/Makefile        |    4 ++--
 drivers/crypto/ccp/ccp-dev-v3.c    |    2 +-
 drivers/crypto/ccp/ccp-dev-v5.c    |   11 ++++++-----
 drivers/crypto/ccp/ccp-dev.c       |   29 ++++++++++++++++++++++++++++-
 drivers/crypto/ccp/ccp-dev.h       |    1 +
 drivers/crypto/ccp/ccp-dmaengine.c |   12 +++++++++++-
 7 files changed, 57 insertions(+), 10 deletions(-)

--
Signature

Comments

Herbert Xu July 26, 2019, 12:33 p.m. UTC | #1
Hook, Gary <Gary.Hook@amd.com> wrote:
> Firstly, add a switch to allow/disallow debugfs code to be built into
> the CCP driver.
> 
> This rest of the patch series implements a set of module parameters
> that allows control over which CCPs on a system are enabled by the
> driver, and how many queues on each device are activated.^M
> 
> A switch to enable/disable DMA engine registration is implemented.
> 
> Details:
> nqueues - configure N queues per CCP (default: 0 - all queues enabled)
> max_devs - maximum number of devices to enable (default: 0 - all
>           devices activated)
> dmaengine - Register services with the DMA subsystem (default: true)
> 
> Only activated devices will have their DMA services registered,
> comprehensively controlled by the dmaengine parameter.
> 
> Changes since v1:
> - Remove debugfs patches that duplicates sysfs function
> - Remove patches for filtering by pcibus and pci device ID
> - Utilize underscores for consistency in variable names
> - Correct commit message for nqueues regarding default value
> - Alter verbage of parameter description (dmaengine)
> - Help text in Kconfig: remove reference to parameters in debugfs
> 
> ---
> 
> Gary R Hook (4):^M
>      crypto: ccp - Make CCP debugfs support optional
>      crypto: ccp - Add a module parameter to specify a queue count
>      crypto: ccp - module parameter to limit the number of enabled CCPs
>      crypto: ccp - Add a module parameter to control registration for DMA
> 
> 
> drivers/crypto/ccp/Kconfig         |    8 ++++++++
> drivers/crypto/ccp/Makefile        |    4 ++--
> drivers/crypto/ccp/ccp-dev-v3.c    |    2 +-
> drivers/crypto/ccp/ccp-dev-v5.c    |   11 ++++++-----
> drivers/crypto/ccp/ccp-dev.c       |   29 ++++++++++++++++++++++++++++-
> drivers/crypto/ccp/ccp-dev.h       |    1 +
> drivers/crypto/ccp/ccp-dmaengine.c |   12 +++++++++++-
> 7 files changed, 57 insertions(+), 10 deletions(-)

All applied.  Thanks.