[11u]: 8255351: Add detection for Graviton 2 CPUs

[Severin Gehwolf]

On Thu, 2020-12-03 at 21:32 +0000, Hohensee, Paul wrote:
> In that case: maintainers, what's your sense of whether 8256488 and
> 8257436 backports would be approved? The two patches apply cleanly
> net of copyright date and context.

Andrew Haley, thoughts? My thinking is that these are performance
improvements, so not necessarily candidates for 11u backport. On the
other hand, if they are low-risk I wouldn't object. They're aarch64-
only. What do you think?

Thanks,
Severin

> -----Original Message-----
> From: Volker Simonis <volker.simonis at gmail.com>
> Date: Thursday, December 3, 2020 at 12:13 PM
> To: "Hohensee, Paul" <hohensee at amazon.com>
> Cc: Vladimir Kozlov <vladimir.kozlov at oracle.com>, "jdk-updates-dev at openjdk.java.net" <jdk-updates-dev at openjdk.java.net>
> Subject: RE: [11u]: 8255351: Add detection for Graviton 2 CPUs
> 
> Hi Paul, Vladimir,
> 
> I think we should only downport this change together with JDK-8257436
> and JDK-8256488 which fix some regressions for the SIMD case.
> 
> I've put a corresponding comment into JDK-8255351 and linked the two
> issues there (sorry, should have done this earlier).
> 
> Best regards,
> Volker
> 
> On Thu, Dec 3, 2020 at 8:54 PM Hohensee, Paul <hohensee at amazon.com> wrote:
> > Thanks for the very quick review, Vladimir. Protocol is to tag issues that need review after review approval. Now that I have yours I've tagged 8255351.
> > 
> > Paul
> > 
> > -----Original Message-----
> > From: jdk-updates-dev <jdk-updates-dev-retn at openjdk.java.net> on behalf of Vladimir Kozlov <vladimir.kozlov at oracle.com>
> > Date: Thursday, December 3, 2020 at 11:35 AM
> > To: "jdk-updates-dev at openjdk.java.net" <jdk-updates-dev at openjdk.java.net>
> > Subject: RE: [11u]: 8255351: Add detection for Graviton 2 CPUs
> > 
> > Changes look good but I don't see backport request in JBS.
> > 
> > Thanks,
> > Vladimir K
> > 
> > On 12/3/20 11:25 AM, Hohensee, Paul wrote:
> > > Please review this 11u backport.
> > > 
> > > JBS: https://bugs.openjdk.java.net/browse/JDK-8255351
> > > Patch: https://git.openjdk.java.net/jdk/commit/2215e5a4
> > > Webrev: http://cr.openjdk.java.net/~phh/8255351/webrev.11u.01/
> > > 
> > > Patch changed by adding copyright date update, and modifying trailing context in hunk #2.
> > > 
> > > Thanks,
> > > Paul
> > >