[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