| Message ID | 20220302085304.10146-1-ike.pan@canonical.com |
|---|---|
| Headers | show |
| Series | Update Hisilicon crypto driver to use one wq | expand |
Acked-by: Tim Gardner <tim.gardner@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(-) >
On 02.03.22 09: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(-) > Like Tim I would like to note that it is not a good idea to have two bug reports mingled into a single submission. If those are independent, then better submit as two individual sets. If those are rather the same, then duplicate one against the other. Chances are just high that something goes wrong with this where not all of the patches have all the bugs linked. And if it is only an odd layout of the changelog... Acked-by: Stefan Bader <stefan.bader@canonical.com>
On 02.03.22 09: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(-) > Applied to focal:linux/master-next. Thanks. -Stefan