RFR (M) 8201556: Disallow reading oops in ClassLoaderData if unloading

[coleen.phillimore at oracle.com]

On 4/18/18 4:42 AM, Andrew Dinn wrote:
> Hi Coleen,
>
> On 17/04/18 17:02, coleen.phillimore at oracle.com wrote:
>> Summary: Move class_loader oop to an OopHandle, and assert that holder
>> is alive when getting class_loader.
>>
>> open webrev at http://cr.openjdk.java.net/~coleenp/8201556.01/webrev
>> bug link https://bugs.openjdk.java.net/browse/JDK-8201556
>>
>> Tested with mach5 tiers 1-7.
> I have just been browsing the code in this area so thought I should
> offer a 2nd review. The changes all look good to me (subtracting the
> typo and legacy closure Kim mentioned).

Thank you Andrew.  I'm glad you're looking at these changes!
Coleen

>
> regards,
>
>
> Andrew Dinn
> -----------
> Senior Principal Software Engineer
> Red Hat UK Ltd
> Registered in England and Wales under Company Registration No. 03798903
> Directors: Michael Cunningham, Michael ("Mike") O'Neill, Eric Shander