RFR: 8226797: serviceability/tmtools/jstat/GcCapacityTest.java fails with Exception: java.lang.RuntimeException: OGCMN > OGCMX (min generation capacity > max generation capacity)

Daniel D. Daugherty daniel.daugherty at oracle.com
Thu Dec 12 16:06:14 UTC 2019


src/hotspot/share/gc/shared/generationSpec.hpp
     No comments.

test/hotspot/jtreg/serviceability/tmtools/jstat/utils/JstatGcCapacityResults.java
     No comments.

Thumbs up.

Dan


On 12/12/19 10:23 AM, Stefan Karlsson wrote:
> In the interest to get this integrated before the RDP cut-off I'm 
> going to push this ASAP. This has gone through tier1-tier3 testing.
>
> StefanK
>
> On 2019-12-12 13:01, Stefan Karlsson wrote:
>> Hi all,
>>
>> Please review this patch to fix a problem with unintialized values in 
>> our generation counters.
>>
>> https://cr.openjdk.java.net/~stefank/8226797/webrev.01/
>> https://bugs.openjdk.java.net/browse/JDK-8226797
>>
>> The jstat values NGCMN and OGCMN both return uninitialized values.
>>
>> I stumbled upon this while creating a patch to remove the 
>> GenerationSpec class.
>>
>> GenerationSpec::_min_size is never initialized, and then used to 
>> create the generations:
>>
>>      case Generation::DefNew:
>>        return new DefNewGeneration(rs, _init_size, _min_size, 
>> _max_size);
>>
>>      case Generation::MarkSweepCompact:
>>        return new TenuredGeneration(rs, _init_size, _min_size, 
>> _max_size, remset);
>>
>> That in turn uses it to initialize the perf counters:
>> DefNewGeneration::DefNewGeneration(ReservedSpace rs,
>>                                     size_t initial_size,
>>                                     size_t min_size,
>>                                     size_t max_size,
>>                                     const char* policy)
>> ...
>>    _gen_counters = new GenerationCounters("new", 0, 3,
>>        min_size, max_size, &_virtual_space);
>>
>> I'm setting the value to _init_size, because it reflects how 
>> MinNewSize and MinOldSize relates to NewSize and OldSize.
>>
>> Thanks,
>> StefanK




More information about the hotspot-gc-dev mailing list