RFR 8215523: jstat reports incorrect values for OU for CMS GC

[Poonam Parhar]

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
>>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20190811/329a9e09/attachment.html>