RFR: Remove 5 develop options from GC code.
Thomas Schatzl
thomas.schatzl at oracle.com
Thu Oct 15 08:48:22 UTC 2015
Hi David,
On Fri, 2015-10-09 at 15:47 +0200, David Lindholm wrote:
> Hi,
>
> Please review this patch that removes the following 5 develop options
> from the GC code:
>
> ScavengeWithObjectsInToSpace
> ParallelOldGCSplitALot
> ParallelOldGCSplitInterval
> PSAdjustTenuredGenForMinorPause
> PSAdjustYoungGenForMajorPause
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8139277 which is a
> subtask to
> Bug: https://bugs.openjdk.java.net/browse/JDK-8139276
> Webrev: http://cr.openjdk.java.net/~david/JDK-8139277/webrev.00/
>
> Testing: Passed JPRT
>
- psAdaptiveSizePolicy.cpp:532: the change removes the if-clause
completely, potentially changing behavior. I would prefer if that were
not the case, as I cannot immediately see that there is no difference
here.
- not sure how the change in psScavenge.cpp:300, 351, 680, theremoval
of code triggered by ZapUnusedArea is related to this change. Unless I
overlooked something, it seems to be something interesting to keep.
Thanks,
Thomas
More information about the hotspot-gc-dev
mailing list