Request for review (s) - 8015851 :UseAdaptiveGCBoundary is broken

[Bengt Rutisson]

Hi Jon,

On 6/6/13 2:25 AM, Jon Masamitsu wrote:
> Fixed the CR # in the subject.
>
> On 6/5/2013 1:27 PM, Jon Masamitsu wrote:
>> A recent changeset effectively removed the initialization of
>> performance counters used  by the option UseAdaptiveGCBoundary

Which change broke the initialization of the performance counters?

One comment:

The code you add in ASPSOldGen::initialize_work() is very similar to the 
code in PSOldGen::initialize(). The PSOldGen::initialize() method is 
called from one (why not both?!?!) of the constructors for PSOldGen, 
which means that it is called from one of the ASPSOldGen constructors. 
But these constructors seem to be unused. Would it be possible to move 
your code from ASPSOldGen::initialize_work() in to the constructor of 
PSOldGen that is actually used?

Thanks,
Bengt

>>
>> This changeset is built on top of the one for 8014546 and the
>> webrev comes in two parts.
>>
>> The fix plus the test
>>
>> http://cr.openjdk.java.net/~jmasa/8015851/webrev.00/
>>
>> Some cleanups of the code
>>
>> http://cr.openjdk.java.net/~jmasa/8015851/webrev_refactor.00/
>>
>> Thanks.
>>
>> Jon
>