RFR (S): 8152724: Sum of Eden, survivor and old capacity may be larger than heap size

[Thomas Schatzl]

Hi Stefan,

On Tue, 2018-12-11 at 11:11 +0100, Stefan Johansson wrote:
> 
> On 2018-11-29 15:26, Thomas Schatzl wrote:
> > Hi all,
> > 
> >    can I have reviews for this small change that fixes some bug in
> > determining the "correct" number of maximum survivor regions for
> > the current GC?
> > 
> > Currently the maximum survivor regions are calculated directly from
> > maximum eden size divided by SurvivorRatio. If the eden is large,
> > and SurvivorRatio small, this could result in strange log output
> > indicating that there is more space in the Java heap available than
> > there actually is space :)
> > 
> > Of course this is not true, and in reality the GC would go into an
> > evacuation failure before allocating that many regions, but
> > nevertheless the output is incorrect.
> > 
> > This change limits the internal max survivor size variable used for
> > printing to the amount of free space.
> > 
> > CR:
> > https://bugs.openjdk.java.net/browse/JDK-8152724
> > Webrev:
> > http://cr.openjdk.java.net/~tschatzl/8152724/webrev/
> 
> Looks good,
> StefanJ

  thanks for your review.

Thomas