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

Bengt Rutisson bengt.rutisson at oracle.com
Thu Aug 1 06:13:35 UTC 2013

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.


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

More information about the hotspot-gc-dev mailing list