RFR: 8029329: tmtools tests fail with NPE (in the tool) when run with G1 and FlightRecorder

[Bengt Rutisson]

Hi Stefan,

Looks good.

Bengt


On 2013-12-02 15:54, Stefan Johansson wrote:
> Hi again,
>
> After discussions with Bengt I realized that the limit I actually want 
> to use is the currently committed number of regions. Here is an 
> updated webrev:
> http://cr.openjdk.java.net/~sjohanss/8029329/webrev.01/
>
> Stefan
>
> On 2013-11-29 14:26, Stefan Johansson wrote:
>> Hi,
>>
>> Please review this fix for:
>> https://bugs.openjdk.java.net/browse/JDK-8029329
>>
>> Webrev:
>> http://cr.openjdk.java.net/~sjohanss/8029329/webrev.00/
>>
>> Summary:
>> The iterator used by the SA for walking through G1s heap regions used 
>> the total number of regions as the limit for hasNext. This caused a 
>> NullPointerException when reaching the first unallocated region. 
>> Updated the SA to initialize the iterator with the currently 
>> allocated number of regions as limit instead.
>>
>> Testing:
>> * Tested builds through JPRT
>> * Ran vm.tmtools.testlist throught Aurora and observed no failures on 
>> the test I expected to pass.
>>
>> Thanks,
>> Stefan
>