RFR(T): 8247495: ProblemList vmTestbase/nsk/jvmti/SetFieldAccessWatch/setfldw001/TestDescription.java

[Daniel D. Daugherty]

Yumin,

Thanks for the review!

Dan


On 6/12/20 2:51 PM, Yumin Qi wrote:
> Hi, Dan
>
>   Looks good to me and it is trivial.
>
>
> Thanks
>
> Yumin
>
> On 6/12/20 11:40 AM, Daniel D. Daugherty wrote:
>> Tap, tap, tap... is this thing working?
>>
>> Anyone out there? This is a trivial 1-liner review...
>>
>> Dan
>>
>>
>> On 6/12/20 12:46 PM, Daniel D. Daugherty wrote:
>>> Greetings,
>>>
>>> It's time to reduce the noise in the CI so I'm ProblemListing tests.
>>>
>>> Here's the bug for failure:
>>>
>>>     JDK-8205957 setfldw001/TestDescription.java fails with bad field 
>>> value
>>>     https://bugs.openjdk.java.net/browse/JDK-8205957
>>>
>>> and here's the bug for the ProblemListing:
>>>
>>>     JDK-8247495 ProblemList 
>>> vmTestbase/nsk/jvmti/SetFieldAccessWatch/setfldw001/TestDescription.java 
>>>
>>>     https://bugs.openjdk.java.net/browse/JDK-8247495
>>>
>>> I'm considering this a trivial change so I need a single (R)eviewer.
>>>
>>> Here's the context diff for the change:
>>>
>>> $ hg diff
>>> diff -r 015533451f4c test/hotspot/jtreg/ProblemList.txt
>>> --- a/test/hotspot/jtreg/ProblemList.txt    Fri Jun 12 09:31:08 2020 
>>> -0700
>>> +++ b/test/hotspot/jtreg/ProblemList.txt    Fri Jun 12 12:40:17 2020 
>>> -0400
>>> @@ -141,6 +141,7 @@
>>>  vmTestbase/nsk/jvmti/scenarios/jni_interception/JI05/ji05t001/TestDescription.java 
>>> 8219652 aix-ppc64
>>>  vmTestbase/nsk/jvmti/scenarios/jni_interception/JI06/ji06t001/TestDescription.java 
>>> 8219652 aix-ppc64
>>>  vmTestbase/nsk/jvmti/SetJNIFunctionTable/setjniftab001/TestDescription.java 
>>> 8219652 aix-ppc64
>>> +vmTestbase/nsk/jvmti/SetFieldAccessWatch/setfldw001/TestDescription.java 
>>> 8205957 generic-all
>>>
>>>  vmTestbase/gc/lock/jni/jnilock002/TestDescription.java 
>>> 8208243,8192647 generic-all
>>>
>>>
>>> This issue is actually much older than JDK-8205957 would indicate
>>> (first sighting in JDK11 for that bug ID). The older version of
>>> the test is covered by https://bugs.openjdk.java.net/browse/JDK-6528079
>>> and that failures first sighting is in JDK7.
>>>
>>>
>>> Thanks, in advance, for any comments, questions, or suggestions.
>>>
>>> Dan
>>>
>>