[11u] Backport 8229352: Use of an uninitialized register in 32-bit ARM template interpreter

[christoph.goettschkes at microdoc.com]

Hi Boris,

I looked into this today and it seems to be some issue on my side. I can 
only reproduce it with my own builds on 32-bit ARM. If I download the 
11.0.4+11 (from adoptopenjdk), everything is fine. Building 11.0.4+11 on 
my own, CDS is not working. I have to look into this later on.

Yes, fix request looks fine to me, thanks for that.

-- Christoph

Boris Ulasevich <boris.ulasevich at bell-sw.com> wrote on 2019-08-21 
17:49:27:

> From: Boris Ulasevich <boris.ulasevich at bell-sw.com>
> To: christoph.goettschkes at microdoc.com
> Cc: jdk-updates-dev at openjdk.java.net
> Date: 2019-08-21 17:49
> Subject: Re: [11u] Backport 8229352: Use of an uninitialized register in 
32-
> bit ARM template interpreter
> 
> Hi Christoph,
> 
> I do not reproduce the CDS issue with ARM hard float. I think what you 
> see is some kind of user error or a different problem.
> 
> Since tier1 is Ok we can ask jdk11u-fix-request for 8229352. I propose 
> this message:
> 
>  > Fix Request
>  > Patch fixes uninitialized register value error which makes 
> interpreter to go though wrong path.
>  > The fix is critical for performance and stability reasons.
>  > Patch applies cleanly to 11u and passes tier1 on ARM sflt/hflt.
> 
> regards,
> Boris
> 
> On 21.08.2019 14:26, christoph.goettschkes at microdoc.com wrote:
> > Hi Boris,
> > 
> > I have the following issue:
> > 
> > The hotspot tier1 JTreg tests do not show any regresseion (as with the
> > jdk/jdk repository), but the CDS archive tests are failing (with and
> > without the patch applied). I guess this would block the backporting.
> > Should I already send you the "Fix Request" comment and file a new bug
> > report for that?
> > 
> > I did not see this issue when I tested the jdk/jdk repository (not 
sure if
> > the issue is there too). I guess this is due to the fact that I 
changed
> > the test setup (I have to use a different test device right now).
> > 
> > Not even the following test works:
> > 
> > $ ./jdk/bin/java -cp $(pwd) -XX:+UnlockDiagnosticVMOptions
> > -XX:SharedArchiveFile=$(pwd)/cds.jsa -Xshare:dump -version
> > 
> > I am not able to reproduce this issue on amd64, but it is present on
> > 32-bit ARM hard and soft float.
> > 
> > Thanks, Christoph
>