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