RFR: 8149650: Create a trace event for G1 heap region type transitions

David Lindholm david.lindholm at oracle.com
Thu Feb 11 16:16:05 UTC 2016


Hi Jesper,

Thanks for looking at this!

On 2016-02-11 14:35, Jesper Wilhelmsson wrote:
> Hi,
>
> heapRegion.hpp, set_*() - I don't really like that we have code in 
> .hpp files. Since these functions now do more than just set stuff I 
> would prefer if they moved into the .cpp file.

Absolutely. New webrev:

http://cr.openjdk.java.net/~david/JDK-8149650/webrev.01/


Thanks,
David

> /Jesper
>
> Den 11/2/16 kl. 13:14, skrev David Lindholm:
>> Hi,
>>
>> Please review this patch that adds a trace event that the vm sends 
>> each time a
>> G1 Heap Region changes type.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8149650
>> Webrev: http://cr.openjdk.java.net/~david/JDK-8149650/webrev.00/
>>
>>
>> Thanks,
>> David



More information about the hotspot-gc-dev mailing list