JDK 9 RFR of JDK-8133588: Place TimeoutLockLoops.java on the problem lis

[Joseph D. Darcy]

Hi David,

On 8/13/2015 2:30 PM, David Holmes wrote:
> Joe,
>
> This is making it hard to clean up everything at once. I'm already 
> delayed from pushing the fix because the "@key intermittent" change is 
> not in hs-rt yet.

If the test needs to be updated as part of the fix for JDK-8029453, 
presumably it should be a trivial merge with the "intermittent" change 
if it doesn't arrive at the same time. If the intermittent-ness give a 
false positive for a while, I think that is acceptable given how changes 
flow around. (The reversal of just the intermittent tag could also be 
done in dev directly after any HotSpot changes for JDK-8029453 get 
integrated into dev.)

> If this change also goes in then I again have to wait for it to arrive 
> before I can reverse it. It will take a month for everything to get 
> back to where it should be. :(

If the HotSpot repos don't pull down this change promptly, I think it is 
fine if the problem list is updated after the HotSpot fix gets into dev.

My main concern here is not have have the test intermittently failing in 
the testing of dev for several more weeks.

Thanks,

-Joe

> This also seems to defeat the purpose of adding the "intermittent" tag.
>
> David
>
> On 14/08/2015 7:18 AM, joe darcy wrote:
>> Hello,
>>
>> Until a fix for JDK-8029453 is in place, the test TimeoutLockLoops.java
>> should be placed on the problem list for Linux since it fails
>> intermittently, but fairly frequently, on that OS:
>>
>> --- a/test/ProblemList.txt    Thu Aug 13 09:36:14 2015 -0700
>> +++ b/test/ProblemList.txt    Thu Aug 13 14:14:39 2015 -0700
>> @@ -353,6 +353,10 @@
>>   # 8062512
>> java/util/spi/ResourceBundleControlProvider/UserDefaultControlTest.java 
>> generic-all
>>
>> +# 8029453
>> +java/util/concurrent/locks/ReentrantLock/TimeoutLockLoops.java 
>> linux-all
>> +
>> +
>> ############################################################################
>>
>>   # jdk_instrument
>>
>> Thanks,
>>
>> -Joe