NACK: [PATCH 0/2] Update SEC driver parameter

[Colin Ian King]

On 30/07/2020 10:25, Stefan Bader wrote:
> On 30.07.20 08:55, Ike Panhc wrote:
>> BugLink: https://launchpad.net/bugs/1867900
>>
>> We have a patch sent to upstream mailing list to enlarge SEC driver
>> capability from 2 to 256 so that cpu stress-ng test passed. This
>> patch has been tested and hit no issue with cpu stress test.
>>
>> This patch has not yet hit mainline kernel but has been ACK from
>> maintainer. Though it does not have stable hash number but this patch
>> is really simple, only 2 define number changed.
>>
>> On focal kernel, hisi_sec2 has been disable temporarily so that
>> cpu stress test will use no accelerator and pass. First patch is
>> to revert that so that on d06 system we will have accelerator and
>> for focal kernel only.
>>
>> Second patch is to enlarge the capability for SEC driver and
>> expect to hit mainline kernel in 5.9 merge window. I am happy to
>> send this patchset again when stable hash available if it makes
>> ubuntu delta much easy to maintain.
>>
>> Ike Panhc (2):
>>   Revert "UBUNTU: [Config] Disable hisi_sec2 temporarily"
>>   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(-)
>>
> The testcase was fixed at least in Focal with 5.4.0-40.44. Now you want to
> re-enable things. This is not the same bug and you cannot just reset the old bug
> report and submit new changes within the other thread.
> Open a new bug report and start a new submission thread on the mailing list.
> 
> -Stefan
> 
> 
Good point Stefan. I was looking at the technicalities and overlooked this.

Colin