RFR 8229420: [Redo] jstat reports incorrect values for OU for CMS GC

[Thomas Schatzl]

Hi,

On 14.08.19 15:13, Poonam Parhar wrote:
> Hello,
> 
> The fix for JDK-8215523 
> <https://bugs.openjdk.java.net/browse/JDK-8215523> had to be backed out 
> with '8227178: Backout of 8215523' because it had caused timeout 
> failures for some of the CMS tests.
> 
> Changeset of JDK-8215523: 
> http://hg.openjdk.java.net/jdk/jdk/rev/734e58d8477b
> 
> Those failures get resolved by adding the following check before calling 
> recalculate_used_stable() in CompactibleFreeListSpace::allocate():
> 
> 1387   // During GC we do not need to recalculate the stable used value for
> 1388   // every allocation in old gen. It is done once at the end of GC 
> instead
> 1389   // for performance reasons.
> 1390   if (!CMSHeap::heap()->is_gc_active()) {
> 1391     recalculate_used_stable();
> 1392   }
> 1393
> 
> 
> Please review the webrev with the updated fix:
> http://cr.openjdk.java.net/~poonam/8229420/webrev.00/

   still good.

Thanks,
   Thomas