RFR (S) 8202447: Fix unloading_occurred to mean unloading_occurred

serguei.spitsyn at oracle.com serguei.spitsyn at oracle.com
Tue May 1 22:41:36 UTC 2018


I'm glad too. :)
Have not reviewed you fixes for a while.

Thanks,
Serguei

On 5/1/18 15:35, coleen.phillimore at oracle.com wrote:
>
> Thank you, Serguei.  I'm glad you reviewed this!
> Coleen
>
> On 5/1/18 6:10 PM, serguei.spitsyn at oracle.com wrote:
>> Hi Coleen,
>>
>> The fix looks good to me.
>>
>> Thanks,
>> Serguei
>>
>>
>> On 5/1/18 12:24, coleen.phillimore at oracle.com wrote:
>>> Summary: nmethod unloading does not need to test for jvmti to set 
>>> unloading_occurred, nor do we need to clean weak Klasses in metadata 
>>> if unloading does not occur.
>>>
>>> open webrev at http://cr.openjdk.java.net/~coleenp/8202447.01/webrev
>>> bug link https://bugs.openjdk.java.net/browse/JDK-8202447
>>>
>>> Tested with mach5 hs-tier1-4, rerunning hs-tier3.  Kitchensink 24 
>>> hours, and runThese with all 4 GCs.
>>>
>>> Thanks,
>>> Coleen
>>
>



More information about the hotspot-dev mailing list