RFR: JDK-8132148 : G1 hs_err region dump legend out of sync with region values
eric.caspole at oracle.com
Thu Jul 23 17:26:51 UTC 2015
Since the Tag enum is private and not used anywhere else in the code,
I'll leave it like this. Otherwise I'd call something in HeapRegionType
for the legend but it seems like overkill.
On 07/23/2015 11:18 AM, Thomas Schatzl wrote:
> Hi Eric,
> On Thu, 2015-07-23 at 10:46 -0400, Eric Caspole wrote:
>> Hi everybody,
>> The other day we noticed the legend for the G1 regions in the hs_err
>> file was out of sync with the actual values, here is a patch:
>> For bug:
>> Let me know what you think.
> looks good. If we want this to be more future-proof, we might want to
> poke the results of HeapRegion::get_short_str() into the output string
> instead of hardcoding them.
> ("Heap Regions: (%s=young(eden), %s=young(survivor), %s=old, "
> "%s=humongous(starts), %s=humongous(continues), "
> , HeapRegion::get_short_str(HeapRegionType::EdenTag), HeapRegion::get_short_str(HeapRegionType::SurvTage), ...
> It's your call, as the others are fine with just hardcoding the values.
More information about the hotspot-gc-dev