[aarch64-port-dev ] RFR: 8252204: AArch64: Implement SHA3 accelerator/intrinsic

[Yangfei (Felix)]

Hi all,

> -----Original Message-----
> From: Sean Mullan [mailto:sean.mullan at oracle.com]
> Sent: Wednesday, September 9, 2020 12:16 AM
> To: Andrew Haley <aph at redhat.com>; Yangfei (Felix)
> <felix.yang at huawei.com>; hotspot-compiler-dev at openjdk.java.net; core-
> libs-dev at openjdk.java.net
> Cc: aarch64-port-dev at openjdk.java.net; Stuart Monteith
> <stuart.monteith at linaro.org>
> Subject: Re: [aarch64-port-dev ] RFR: 8252204: AArch64: Implement SHA3
> accelerator/intrinsic
> 
> Since this change affects security code, please make sure you add security-
> dev at openjdk.java.net on any followup code reviews.

Thanks for reminding that.
I have just switched to github and created a PR for this issue: https://github.com/openjdk/jdk/pull/207
Let's switch to this PR for the followup code reviews.
I see security and hotspot labels was automatically added by the bots.  I also added labels for core-libs and hotspot-compiler.

> 
> On 9/1/20 10:44 AM, Andrew Haley wrote:
> > On 01/09/2020 11:53, Yangfei (Felix) wrote:
> >> Sure, I am happy if the original author of the assembly code or someone
> else from Linaro could help here.
> >> I wasn't aware there was such an requirement here given that assembly
> code is licensed under GPL.
> >
> > There sure is. All code must be contributed by its owner and put on
> > the cr.openjdk site. Especially GPL code.

I have added ard.biesheuvel at linaro.org  to Contributed-by: in the git commit msg.
And the newly created PR was Acked-by: Ard Biesheuvel ard.biesheuvel at linaro.org
Hope this addresses Andrew's concern.

Best regards,
Felix