RFR: 8248266: ZGC: TestUncommit.java fails due to "Exception: Uncommitted too fast" again

Per Liden per.liden at oracle.com
Tue Aug 18 11:34:56 UTC 2020


Thanks Erik!

/Per

On 8/18/20 1:33 PM, Erik Österlund wrote:
> Hi Per,
> 
> Looks good.
> 
> Thanks,
> /Erik
> 
> On 2020-08-18 11:39, Per Liden wrote:
>> Hi!
>>
>> Most people are back from vacations now (including myself), so I'm 
>> giving this thread a bump as I'm still looking for reviews on this 
>> change.
>>
>> cheers,
>> Per
>>
>> On 6/26/20 3:28 PM, Per Liden wrote:
>>> Hi,
>>>
>>> The test gc/z/TestUncommit.java sometimes fail because of bad timing, 
>>> caused by what appears to be an heavily loaded machine so the test 
>>> thread doesn't get to execute in a timely manner.
>>>
>>> I've restructured the test a bit, to be less sensitive to this. 
>>> Instead of sleeping and then checking if uncommit has happened, the 
>>> test will now wait until uncommit happens and record the time. When 
>>> checking the time it now uses TIMEOUT_FACTOR to control how strict 
>>> the check should be.
>>>
>>> To keep things simple, I've also broken out the part that tests with 
>>> uncommit disabled into a separate test.
>>>
>>> The change in ZPage/ZPageCache rounds up the last_used/last_committed 
>>> timestamps to the nearest second. Without this we will always be 
>>> rounding the time down, which means we can sometimes uncommit one 
>>> second too early, which is wrong but not a big problem, but this can 
>>> also cause TestUncommit.java to fail with "Uncommitted too fast".
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8248266
>>> Webrev: http://cr.openjdk.java.net/~pliden/8248266/webrev.0
>>>
>>> Testing: Ran TestUncommit 100+ times on all Oracle-platforms.
>>>
>>> /Per
> 



More information about the hotspot-gc-dev mailing list