Backport: 8216549: Mismatched unsafe access to non escaping object fails

Roman Kennke rkennke at redhat.com
Wed Oct 9 10:06:43 UTC 2019


Hi Paul,

Thanks for checking it!

> The code in library_call.cpp comes seems to come from two patches
> 
> https://bugs.openjdk.java.net/browse/JDK-8160360
> http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/4be0cada20ad

I will look into backporting this change.

> https://bugs.openjdk.java.net/browse/JDK-8162101
> http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/10dad1d40843

This is already in jdk8u:
http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/7ca49bca3c2a

Roman



> I'd backport those first.
> 
> Paul
> 
> On 10/7/19, 11:31 AM, "jdk8u-dev on behalf of Roman Kennke" <jdk8u-dev-bounces at openjdk.java.net on behalf of rkennke at redhat.com> wrote:
> 
>     Ping! This is still awaiting review.
>     
>     Thanks,
>     Roman
>     
>     
>     > This backports:
>     > https://bugs.openjdk.java.net/browse/JDK-8216549
>     > 
>     > Original change:
>     > hg.openjdk.java.net/jdk/jdk12/rev/f0490430ef7a
>     > 
>     > JDK11u backport:
>     > http://hg.openjdk.java.net/jdk-updates/jdk11u-dev/rev/d5fb6ad4203b
>     > 
>     > In order to make it work on 8u, I needed the extra change in
>     > library_call.cpp. Thanks Roland who helped with this.
>     > 
>     > Webrev:
>     > http://cr.openjdk.java.net/~rkennke/JDK-8216549-8u/webrev.00/
>     > 
>     > Testing:
>     > The new testcase fails without the fix, and passes with the fix. tier1 &
>     > tier2 show no regressions.
>     > 
>     > Can I please get a review?
>     > 
>     > Thanks,
>     > Roman
>     > 
>     
>     
> 



More information about the jdk8u-dev mailing list