[XXS] RFR 8137230: TEST_BUG: java/nio/channels/FileChannel/LoopingTruncate.java timed out
ivan.gerasimov at oracle.com
Mon Sep 28 20:01:53 UTC 2015
On 28.09.2015 20:59, Roger Riggs wrote:
> Hi Ivan,
> Looks ok.
> One comment about timeouts, the test library has a function to extend
> the timeout based
> on the -timeoutOption; Using jdk.testlibrary.Utils.adjustTime(nn) is
> useful if the extension
> of the timeout will be useful depending on the execution machine or vm
Yes, it'll be even safer with adjustTime(), thanks for suggesting.
Here's the updated webrev:
> On 9/28/2015 1:19 PM, Ivan Gerasimov wrote:
>> The just added test started to intermittently fail on some machines.
>> It appears that creating a temporary file and doing some simple IO
>> with it can sometimes take more than 10 seconds even without an
>> infinite loop.
>> The proposal is:
>> - increase the timeout;
>> - limit the amount of work to only truncation of the file;
>> - if a case of failure, print a stacktrace of the frozen thread for a
>> better diagnostic.
>> With this change, the test passed on all the platforms.
>> Would you please help review the fix?
>> BUGURL: https://bugs.openjdk.java.net/browse/JDK-8137230
>> WEBREV: http://cr.openjdk.java.net/~igerasim/8137230/00/webrev/
>> Sincerely yours,
More information about the nio-dev