RE: [PATCH V2 2/4] firmware: xilinx: Add ZynqMP SHA API for SHA3 functionality

-----Original Message-----
From: Harsha Harsha
Sent: Monday, February 21, 2022 5:47 PM
To: Corentin Labbe <clabbe.montjoie@gmail.com>
Cc: herbert@gondor.apana.org.au; davem@davemloft.net; linux-crypto@vger.kernel.org; linux-kernel@vger.kernel.org; linux-arm-
kernel@lists.infradead.org; Michal Simek [off-list ref]; Sarat Chand Savitala [off-list ref]; Harsh Jain
[off-list ref]; git [off-list ref]
Subject: RE: [PATCH V2 2/4] firmware: xilinx: Add ZynqMP SHA API for SHA3 functionality

quoted

-----Original Message-----
From: Corentin Labbe <clabbe.montjoie@gmail.com>
Sent: Friday, February 18, 2022 3:13 PM
To: Harsha Harsha <redacted>
Cc: herbert@gondor.apana.org.au; davem@davemloft.net; linux-crypto@vger.kernel.org; linux-kernel@vger.kernel.org; linux-arm-
kernel@lists.infradead.org; Michal Simek [off-list ref]; Sarat Chand Savitala [off-list ref]; Harsh Jain
[off-list ref]; git [off-list ref]
Subject: Re: [PATCH V2 2/4] firmware: xilinx: Add ZynqMP SHA API for SHA3 functionality

Le Fri, Feb 18, 2022 at 12:44:22AM +0530, Harsha a écrit :

quoted

This patch adds zynqmp_pm_sha_hash API in the ZynqMP firmware to compute
SHA3 hash of given data.

Signed-off-by: Harsha <redacted>
Acked-by: Michal Simek <redacted>
---

Hello

Your signed-off should contain your real name.

My complete name is Harsha which I have mentioned in the signed-off section.

I shall update SOB as Signed-off-by: Harsha Harsha [off-list ref] in the next version of patch.

quoted

Furthermore why did you drop copyright from previous poster ?
See https://patchwork.kernel.org/project/linux-crypto/cover/1556793282-17346-1-git-send-email-kalyani.akula@xilinx.com/ for
reference.

I did not understand the comment. Do you want me to add Kalyani's name also in the Signed of section?

I will retain Kalyani as the author in next version of patch.

quoted

Furthermore, the previous poster didnt answered my questions about parallel processing and tests.

Since SHA3 HW engine in ZynqMPSoC does not support parallel processing of 2 hash requests, so we have changed our approach.
Now  to support parallel processing of 2 hash requests, software fallback is being used for init, update, final, export and import in the
ZynqMP SHA driver.
For digest, the calculation of SHA3 hash is done by the hardened SHA3 accelerator in Xilinx ZynqMP SoC.

Following tests have been done for the driver:
- Enabled kernel self tests and extra run-time crypto self tests
- Tested SHA hash computation for different sizes of data using userspace application
- Tested SHA hash computation using multiple updates of data using userspace application
- Tested parallel hash computation
- Tested using the tcrypt module

quoted

Regards

Regards,
Harsha

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel