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