On 16 January 2018 at 08:41, Steve Capper <steve.capper@xxxxxxx> wrote: > On Fri, Jan 12, 2018 at 03:13:56PM +0000, Ard Biesheuvel wrote: >> On 12 January 2018 at 13:15, Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx> wrote: >> > Add an implementation of SHA3 to arm64 using the new special instructions (#4) >> > >> > In preparation of that, fix a bug in the SHA3 and refactor it a bit so it >> > can serve as a fallback for the other code. Also, add some new test vectors >> > to get better test coverage. >> > >> > Ard Biesheuvel (5): >> > crypto/generic: sha3 - fixes for alignment and big endian operation >> > crypto/generic: sha3 - simplify code >> > crypto/generic: sha3 - export init/update/final routines >> > crypto/arm64: sha3 - new implementation based on special instructions >> >> Forgot to mention: this is an RFT for patch #4, as it has not been >> validated against a real implementation, only against my own QEMU >> code. > > Hi Ard, > I have tested this patch set applied to 4.15-rc7 running in a model. > > I used the following tcrypt modes: > 48, 49, 50, 51, 111, 112, 113, 114, 187, 188, 322, 323, 324, 325, 418, > 419, 420 and 421. > > Also, I added some logic to double check that sha3_ce_transform(.) > was being called rather than sha3_scalar_transform(.). > (Because both the scalar and ce code paths are contained in the > sha3-x-arm64 drivers). > > So, please feel free to add for the series: > Tested-by: Steve Capper <steve.capper@xxxxxxx> > Thanks Steve!