ACK/Cmnt: [PATCH 0/4][SRU F] Update Hisilicon crypto driver to use one wq

Tim Gardner 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(-)
> 

-- 
-----------
Tim Gardner
Canonical, Inc



More information about the kernel-team mailing list