[11u] RFR: 8248845: AArch64: stack corruption after spilling vector register

[Yangfei (Felix)]

Hi,

> -----Original Message-----
> From: Severin Gehwolf [mailto:sgehwolf at redhat.com]
> Sent: Wednesday, August 26, 2020 2:39 PM
> To: Yangfei (Felix) <felix.yang at huawei.com>; Nick Gasson
> <nick.gasson at arm.com>; jdk-updates-dev at openjdk.java.net
> Subject: Re: [11u] RFR: 8248845: AArch64: stack corruption after spilling
> vector register
> 
> Hi,
> 
> On Wed, 2020-08-26 at 02:03 +0000, Yangfei (Felix) wrote:
> > Hi Nick,
> >
> > I see it was approved for jdk11u.  I will help push it for you.
> 
> While it has jdk11u-fix-yes, there seem to be conflicting statements being
> made. This thread says it doesn't apply cleanly hence this review thread. The
> bug comment from 2020-07-08 says "Patch applies cleanly".
> Which one is it? Assuming the former, we should get the review done first
> before pushing.

True.  I also noticed this when I tried to apply the original patch. 
I will hold on pushing this before the backport webrev is reviewed.

Thanks,
Felix

> Thanks,
> Severin
> 
> > Felix
> >
> > > -----Original Message-----
> > > From: jdk-updates-dev [mailto:jdk-updates-dev-retn at openjdk.java.net]
> > > On Behalf Of Nick Gasson
> > > Sent: Friday, August 21, 2020 6:02 PM
> > > To: jdk-updates-dev at openjdk.java.net
> > > Subject: Re: [11u] RFR: 8248845: AArch64: stack corruption after
> > > spilling vector register
> > >
> > > Hi,
> > >
> > > Is anyone available to help with this backport?
> > >
> > > --
> > > Thanks,
> > > Nick
> > >
> > > On 07/24/20 13:57 pm, Nick Gasson wrote:
> > > > Hi,
> > > >
> > > > Is anyone able to sponsor this backport for me?
> > > >
> > > > Bug: https://bugs.openjdk.java.net/browse/JDK-8248845
> > > > Original change:
> > > > https://hg.openjdk.java.net/jdk/jdk15/rev/d5be95758352
> > > > Review thread:
> > > > https://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2020-
> > > > July
> > > > /038886.html
> > > >
> > > > The patch does not apply cleanly on 11u as the ScheduleAndBundle
> > > > function has been moved and renamed. I've prepared another webrev
> > > that
> > > > applies on 11u:
> > > >
> > > > http://cr.openjdk.java.net/~ngasson/8248845/webrev.11u.0/
> > > >
> > > > Tested tier1 on AArch64.