mbox series

[0/2] Update Hisilicon SEC driver parameter

Message ID 20200804081720.11742-1-ike.pan@canonical.com
Headers show
Series Update Hisilicon SEC driver parameter | expand

Message

Ike Panhc Aug. 4, 2020, 8:17 a.m. UTC
BugLink: https://launchpad.net/bugs/1890222

On focal kernel, hisi_sec2 has been disable temporarily so that
cpu stress test will use no accelerator and pass. And now, we
have proper fix to enlarge hisi_sec2 capability and the fix just
hits mainline kernel. Here are the fix and also we need to
re-enable hisi_sec2 driver in focal kernel.

Ike Panhc (1):
  Revert "UBUNTU: [Config] Disable hisi_sec2 temporarily"

Longfang Liu (1):
  crypto: hisilicon - update SEC driver module parameter

 debian.master/abi/5.4.0-41.45/arm64/generic.modules | 1 +
 debian.master/config/annotations                    | 2 +-
 debian.master/config/config.common.ubuntu           | 2 +-
 drivers/crypto/hisilicon/sec2/sec_main.c            | 6 +++---
 4 files changed, 6 insertions(+), 5 deletions(-)

Comments

Colin King Aug. 4, 2020, 8:36 a.m. UTC | #1
On 04/08/2020 09:17, Ike Panhc wrote:
> BugLink: https://launchpad.net/bugs/1890222
> 
> On focal kernel, hisi_sec2 has been disable temporarily so that
> cpu stress test will use no accelerator and pass. And now, we
> have proper fix to enlarge hisi_sec2 capability and the fix just
> hits mainline kernel. Here are the fix and also we need to
> re-enable hisi_sec2 driver in focal kernel.
> 
> Ike Panhc (1):
>   Revert "UBUNTU: [Config] Disable hisi_sec2 temporarily"
> 
> Longfang Liu (1):
>   crypto: hisilicon - update SEC driver module parameter
> 
>  debian.master/abi/5.4.0-41.45/arm64/generic.modules | 1 +
>  debian.master/config/annotations                    | 2 +-
>  debian.master/config/config.common.ubuntu           | 2 +-
>  drivers/crypto/hisilicon/sec2/sec_main.c            | 6 +++---
>  4 files changed, 6 insertions(+), 5 deletions(-)
> 

Clean upstream cherry pick, fix looks good to me. Thanks Ike, good that
it got found by stress-ng, and thanks for following this through with
the fix.

Acked-by: Colin Ian King <colin.king@canonical.com>
Stefan Bader Aug. 4, 2020, 8:58 a.m. UTC | #2
On 04.08.20 10:17, Ike Panhc wrote:
> BugLink: https://launchpad.net/bugs/1890222
> 
> On focal kernel, hisi_sec2 has been disable temporarily so that
> cpu stress test will use no accelerator and pass. And now, we
> have proper fix to enlarge hisi_sec2 capability and the fix just
> hits mainline kernel. Here are the fix and also we need to
> re-enable hisi_sec2 driver in focal kernel.
> 
> Ike Panhc (1):
>   Revert "UBUNTU: [Config] Disable hisi_sec2 temporarily"
> 
> Longfang Liu (1):
>   crypto: hisilicon - update SEC driver module parameter
> 
>  debian.master/abi/5.4.0-41.45/arm64/generic.modules | 1 +
>  debian.master/config/annotations                    | 2 +-
>  debian.master/config/config.common.ubuntu           | 2 +-
>  drivers/crypto/hisilicon/sec2/sec_main.c            | 6 +++---
>  4 files changed, 6 insertions(+), 5 deletions(-)
> 
See https://wiki.ubuntu.com/StableReleaseUpdates/#SRU_Bug_Template about what
the SRU team would expect to see about regression potential.

Otherwise:

Acked-by: Stefan Bader <stefan.bader@canonical.com>
Kelsey Skunberg Aug. 7, 2020, 10:11 p.m. UTC | #3
Applied to Focal/master-next. Thank you!

-Kelsey

On 2020-08-04 16:17:20 , Ike Panhc wrote:
> BugLink: https://launchpad.net/bugs/1890222
> 
> On focal kernel, hisi_sec2 has been disable temporarily so that
> cpu stress test will use no accelerator and pass. And now, we
> have proper fix to enlarge hisi_sec2 capability and the fix just
> hits mainline kernel. Here are the fix and also we need to
> re-enable hisi_sec2 driver in focal kernel.
> 
> Ike Panhc (1):
>   Revert "UBUNTU: [Config] Disable hisi_sec2 temporarily"
> 
> Longfang Liu (1):
>   crypto: hisilicon - update SEC driver module parameter
> 
>  debian.master/abi/5.4.0-41.45/arm64/generic.modules | 1 +
>  debian.master/config/annotations                    | 2 +-
>  debian.master/config/config.common.ubuntu           | 2 +-
>  drivers/crypto/hisilicon/sec2/sec_main.c            | 6 +++---
>  4 files changed, 6 insertions(+), 5 deletions(-)
> 
> -- 
> 2.25.1
> 
> 
> -- 
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
Seth Forshee Aug. 10, 2020, 9:06 p.m. UTC | #4
On Tue, Aug 04, 2020 at 04:17:20PM +0800, Ike Panhc wrote:
> BugLink: https://launchpad.net/bugs/1890222
> 
> On focal kernel, hisi_sec2 has been disable temporarily so that
> cpu stress test will use no accelerator and pass. And now, we
> have proper fix to enlarge hisi_sec2 capability and the fix just
> hits mainline kernel. Here are the fix and also we need to
> re-enable hisi_sec2 driver in focal kernel.

Applied to groovy/master-next, thanks!