ACK/Cmnt: [PATCH 0/4][SRU F] Update Hisilicon crypto driver to use one wq
tim.gardner at canonical.com
Thu Mar 3 12:32:03 UTC 2022
Acked-by: Tim Gardner <tim.gardner at canonical.com>
Having 2 BugLinks in the same patch series is a little confusing.
On 3/2/22 01:53, Ike Panhc wrote:
> BugLink: https://launchpad.net/bugs/1943301
> BugLink: https://launchpad.net/bugs/1932117
> After checkbox CPU stress test, it takes 2min 30sec to login.
> Checks system resources and find out there are more then 900000
> processes in output of `ps aux` and most of them are [hisi_qm].
> In order to fix this issue, we need to backport patch for adding
> workqueue and using 1 wq per qm. Patch "Unify hardware error
> init/uninit into QM" is pulled in because of patch dependency.
> Another issue is that resetting crypto module can not work. To fix
> this issue we need to backport patch "Update SEC init and reset".
> Longfang Liu (1):
> crypto: hisilicon/sec2 - update SEC initialization and reset
> Shukun Tan (2):
> crypto: hisilicon - Unify hardware error init/uninit into QM
> crypto: hisilicon - Use one workqueue per qm instead of per qp
> Ye Kai (1):
> crypto: hisilicon/sec2 - Add workqueue for SEC driver.
> drivers/crypto/hisilicon/hpre/hpre_main.c | 33 +++---
> drivers/crypto/hisilicon/qm.c | 125 +++++++++++++-------
> drivers/crypto/hisilicon/qm.h | 25 +++-
> drivers/crypto/hisilicon/sec2/sec_main.c | 138 ++++++++++++----------
> drivers/crypto/hisilicon/zip/zip_main.c | 58 ++++-----
> 6 files changed, 226 insertions(+), 155 deletions(-)
More information about the kernel-team