RFR (S) 8202447: Fix unloading_occurred to mean unloading_occurred

coleen.phillimore at oracle.com coleen.phillimore at oracle.com
Thu May 3 13:21:22 UTC 2018


Thank you for the review!
Coleen

On 5/3/18 5:55 AM, Robbin Ehn wrote:
> Hi Coleen,
>
> On 05/02/2018 12:10 AM, serguei.spitsyn at oracle.com wrote:
>> Hi Coleen,
>>
>> The fix looks good to me.
>
> +1, thanks for fixing.
>
> /Robbin
>
>>
>> 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