[11u] RFR: 8212937: Parent class loader may not have a referred ClassLoaderData instance when obtained in Klass::class_in_module_of_loader

Severin Gehwolf sgehwolf at redhat.com
Mon Nov 19 16:23:43 UTC 2018


On Fri, 2018-11-16 at 10:11 -0500, Lois Foltan wrote:
> On 11/16/2018 10:09 AM, Severin Gehwolf wrote:
> 
> > On Thu, 2018-11-15 at 20:09 +0100, Severin Gehwolf wrote:
> > > Hi,
> > > 
> > > Please review this 11u backport for JDK-8212937. While the JDK 12 patch
> > > applies cleanly it doesn't quite do the right thing. The JDK 12 test
> > > would still crash the JVM due to the lack of the changes in
> > > src/hotspot/share/classfile/javaClasses.cpp as describe_external is
> > > being used in JDK 11. That function has been removed with JDK-
> > > 8205611[1] in JDK 12.
> > > 
> > > In addition, the exception messages aren't the same between JDK 12 and
> > > JDK 11, so Test.java needed an update to account for that. Also due to
> > > lack of JDK-8205611[1] in JDK 11. Everything else is the same as for
> > > JDK 12.
> > > 
> > > Bug: https://bugs.openjdk.java.net/browse/JDK-8212937
> > > webrev: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8212937/jdk11/webrev.01/
> > > 
> > > Testing: Added regression test, internal test.
> > 
> > Also:
> > 
> > tier1 testing showed no regressions with this on linux x86_64.
> 
> Hi Severin,
> 
> Thank you for doing this testing.  I think your change looks good!

Thanks, Lois. Could I get a second review for this, please?

Thanks,
Severin

> 
> > 
> > Thanks,
> > Severin
> > 
> > > Thoughts?
> > > 
> > > Thanks,
> > > Severin
> > > 
> > > [1] https://bugs.openjdk.java.net/browse/JDK-8205611
> 
> 



More information about the hotspot-dev mailing list