RFR (XS): 8021879: G1: G1HeapRegionSize flag value not updated correctly

[Bengt Rutisson]

Hi all,

I made two updates to the test. First I added a filter to skip the test 
if it is not run with -XX:+UseG1GC. This way the test will not cause 
failures when our testing harness passes other GC flags to the test. 
Second I added a test case for a too large region size to test that it 
is limited to 32m.

If there are no objections to this test I will go ahead an push it.

Thanks,
Bengt


On 7/31/13 2:55 PM, Bengt Rutisson wrote:
>
> Hi Thomas,
>
> On 7/31/13 2:47 PM, Thomas Schatzl wrote:
>> Hi,
>>
>> On Wed, 2013-07-31 at 10:12 +0200, Bengt Rutisson wrote:
>>> On 7/30/13 7:06 PM, Jon Masamitsu wrote:
>>>> Change looks good.
>>> Thanks for the review Jon!
>>>
>>> I added a regression test and provided a new webrev. The code change is
>>> still the same. Let me know if you have any issues with the test
>>> otherwise I'll push if Thomas is happy with the test.
>>>
>>> http://cr.openjdk.java.net/~brutisso/8021879/webrev.01/
>> I'm happy now :)
>
> Great! Thanks!
>
> Bengt
>
>>
>> Thanks,
>> Thomas
>>
>