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

[Severin Gehwolf]

On Mon, 2018-11-19 at 17:23 +0100, Severin Gehwolf wrote:
> 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?

Anyone willing to review this?

Thanks,
Severin

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