RFR(S): 8026396 - Remove information duplication in the collector policy

John Coomes John.Coomes at oracle.com
Fri Jun 6 14:06:11 UTC 2014


Jesper Wilhelmsson (jesper.wilhelmsson at oracle.com) wrote:
> Hi,
> 
> Another step towards cleaner collector policy code.
> 
> This cleanup removes the need to keep the generation sizing flags in sync with 
> the collector policy version of the same variables during setup. The collector 
> policy variables are initialized in the start and then used throughout the setup 
> code. In the end we write the values back to the flags if needed.
> 
> This change builds upon the merged collector policy (8027643) currently in review.
> 
> Webrev: http://cr.openjdk.java.net/~jwilhelm/8026396/webrev/
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8026396

This looks good to me.

There seems to be a chance for underflow here:

534     _min_gen1_size = MIN2(_initial_gen1_size, _min_heap_byte_size - _min_gen0_size);

but your change is ok, since that also existed in the original code.

-John



More information about the hotspot-gc-dev mailing list