RFR (XS): 8007776: Test6852078.java timeouts

[Vladimir Kozlov]

On 2/26/13 12:10 PM, David Chase wrote:
> Does a change of that sort require a re-JPRT?
> (I can see either answer as equally likely, that's why I am asking.)

No. Just run this test with jtreg on one of lab or your machine.
The same applies to Hotspot sources. When you passed JPRT testing 
already and modified style only (or added parenthesis) after that. Just 
build hotspot on one machine.

Thanks,
Vladimir

>
> David
>
> On 2013-02-26, at 3:07 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
>
>> Fix code style:
>>
>> < if ( i > 100 && System.currentTimeMillis() - start > 100000 )
>> ---
>>> if ((i > 100) && ((System.currentTimeMillis() - start) > 100000))
>>
>> Vladimir
>>
>> On 2/26/13 11:43 AM, David Chase wrote:
>>> http://cr.openjdk.java.net/~drchase/8007776/webrev.01/
>>>
>>> Made changes, reran tests, rechecked early exit with extra iterations.
>>>
>>> On 2013-02-26, at 12:01 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
>>>
>>>> David,
>>>>
>>>> Can you do it as separate check inside loop with 'break;' with a comment? I think it would be easier to read the code such way.
>>>>
>>>> Also you can remove /othervm since no flags are used?
>>>>
>>>> Thanks,
>>>> Vladimir
>>>>
>>>> On 2/26/13 6:24 AM, David Chase wrote:
>>>>>
>>>>> http://cr.openjdk.java.net/~drchase/8007776/webrev.00/
>>>>>
>>>>> Bug: Test6852078.java contained a 2000-test-iteration loop for tickling a sometimes-bug,
>>>>> that also sometimes timed out after as few as 130 iterations on some slower machines.
>>>>>
>>>>> Fix: Inserted an early-out on the test-loop -- if more than 100 seconds and more than
>>>>> 100 iterations have both passed, then exit is allowed.
>>>>> 2000 iterations are still the norm on faster machines (e.g., on a modern Intel laptop it
>>>>> takes 5 seconds to run).
>>>>>
>>>>> Testing:
>>>>> Jtreg, JPRT on that test.
>>>>> Hand-tested the early-out logic by bumping the iteration count to 2 million; it exited after
>>>>> a total test execution time of 100.5 seconds.
>>>>>
>>>
>