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

Bengt Rutisson bengt.rutisson at oracle.com
Thu Aug 1 00:13:34 PDT 2013


Forgot the link to the webrev:
http://cr.openjdk.java.net/~brutisso/8021879/webrev.02/

Bengt

On 8/1/13 8:13 AM, Bengt Rutisson wrote:
>
> 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
>>>
>>
>



More information about the hotspot-gc-dev mailing list