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

Thomas Schatzl thomas.schatzl at oracle.com
Thu Jan 23 10:56:27 UTC 2014


Hi Erik,

On Wed, 2014-01-22 at 18:19 +0100, 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

  looks good :) Thanks for cleaning up the naming in the test too.

Thomas





More information about the hotspot-gc-dev mailing list