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

Stefan Johansson stefan.johansson at oracle.com
Fri Feb 12 08:14:24 UTC 2016



On 2016-02-11 17:16, David Lindholm wrote:
> 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/
Looks ok,
Stefan

>
>
> 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