Hello,

The fix for this bug had to be backed out with '8227178: Backout of 8215523' because it had caused timeout failures for some of the CMS tests. 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 updated webrev:
http://cr.openjdk.java.net/~poonam/8215523/webrev.02/

Thanks,
Poonam


On 7/2/19 6:42 AM, Poonam Parhar wrote:
Hi Aleksey, Thomas,

It wasn't meant to be non-public. I have opened it.

Thanks,
Poonam

On 7/2/19 3:36 AM, Thomas Schatzl wrote:
Hi,

On Tue, 2019-07-02 at 10:10 +0200, Aleksey Shipilev wrote:
Hi,

On 6/21/19 10:30 PM, Poonam Parhar wrote:
On 6/21/19 12:21 PM, Poonam Parhar wrote:
Bug 8215523 <https://bugs.openjdk.java.net/browse/JDK-8215523>:
jstat reports incorrect values for
OU for CMS GC
This bug is non-public, was it really meant to be?

   there does not seem to be anything confidential in the public areas
of the bug. Maybe Poonam can open it after looking at it again, and
eventually open it (and add a token "Description" ;) ).

Thanks,
   Thomas



Reply via email to