Re: [Qemu-devel] [PATCH v4 0/4] target-arm: add SHA-3, SM3 and SHA512 instruction support

2018-01-22 Thread Ard Biesheuvel
On 22 January 2018 at 17:11, Peter Maydell  wrote:
> On 19 January 2018 at 18:22, Ard Biesheuvel  wrote:
>> Changes since v3:
>> - don't bother with helpers for the SHA3 instructions: they are simple enough
>>   to be emitted as TCG ops directly
>> - rebase onto Richard's pending SVE work
>
> I've made a few comments about structural nits but otherwise I think
> this looks good. I'm trying to scare up a test setup so I can write
> some risu tests for the instruction patterns...hopefully will be able
> to do that later this week.
>

That would be excellent - thanks.



Re: [Qemu-devel] [PATCH v4 0/4] target-arm: add SHA-3, SM3 and SHA512 instruction support

2018-01-22 Thread Peter Maydell
On 19 January 2018 at 18:22, Ard Biesheuvel  wrote:
> Changes since v3:
> - don't bother with helpers for the SHA3 instructions: they are simple enough
>   to be emitted as TCG ops directly
> - rebase onto Richard's pending SVE work

I've made a few comments about structural nits but otherwise I think
this looks good. I'm trying to scare up a test setup so I can write
some risu tests for the instruction patterns...hopefully will be able
to do that later this week.

thanks
-- PMM



[Qemu-devel] [PATCH v4 0/4] target-arm: add SHA-3, SM3 and SHA512 instruction support

2018-01-19 Thread Ard Biesheuvel
Changes since v3:
- don't bother with helpers for the SHA3 instructions: they are simple enough
  to be emitted as TCG ops directly
- rebase onto Richard's pending SVE work

Changes since v2:
- fix thinko in big-endian aware handling of 64-bit quantities: this is not
  needed given that the NEON registers are represented as arrays of uint64_t
  so they always appear in the correct order.
- add support for SM3 instructions (Chinese SHA derivative)

Changes since v1:
- update SHA512 patch to adhere more closely to the existing style, and to
  the way the instruction encodings are classified in the ARM ARM (#1)
- add patch implementing the new SHA3 instructions EOR3/RAX1/XAR/BCAX (#2)
- enable support for these instructions in user mode emulation (#3)

Ard Biesheuvel (4):
  target/arm: implement SHA-512 instructions
  target/arm: implement SHA-3 instructions
  target/arm: implement SM3 instructions
  target/arm: enable user-mode SHA-3, SM3 and SHA-512 instruction
support

 linux-user/elfload.c   |  18 ++
 target/arm/cpu.h   |   3 +
 target/arm/cpu64.c |   3 +
 target/arm/crypto_helper.c | 192 +++-
 target/arm/helper.h|  10 +
 target/arm/translate-a64.c | 317 
 6 files changed, 542 insertions(+), 1 deletion(-)

-- 
2.11.0