RFR: 8028254: gc/arguments/TestMinInitialErgonomics.java failed with unexpected initial heap size

Erik Helin erik.helin at oracle.com
Tue Jan 28 13:45:40 UTC 2014


Bengt, Stefan and Thomas,

thanks for the reviews!

Erik

On 2014-01-28 14:32, Bengt Rutisson wrote:
>
> Hi Erik,
>
> Looks good.
>
> Bengt
>
> On 1/22/14 6:19 PM, Erik Helin wrote:
>> Hi all,
>>
>> this change updates the test hotspot/test/gc/arguments/
>> TestMinInitialErgonomics.java to reflect the changes made to the heap
>> sizing code.
>>
>> The first part is just updating the function WB_PrintHeapSizes in
>> whitebox.cpp to use the same naming as the VM.
>>
>> The second part changes TestMinInitialErgonomics.java to use the same
>> naming as the VM and also align the expectedHeapSize correctly:
>>
>> Webrev:
>> http://cr.openjdk.java.net/~ehelin/8028254/webrev.00/
>>
>> Testing:
>> Reproduced test failure locally and verified the fix
>>
>> Bug:
>> https://bugs.openjdk.java.net/browse/JDK-8028254
>>
>> Thanks,
>> Erik
>



More information about the hotspot-gc-dev mailing list