[8u] RFA for 8185164: GetOwnedMonitorInfo() returns incorrect owned monitor

[David Buck]

Hi Dan!

I that case, it sounds perfectly reasonable to me that the JDK 8 
backport just drop the testcases. Unfortunately, he will still need to 
get a Reviewer to sign off on the backport webrev as it is technically 
no longer a "trivial" backport. Maybe you can do that for him?

Cheers,
-Buck

On 2017/08/10 22:54, Daniel D. Daugherty wrote:
> Hi Buck!
> 
> Serguei and I helped Yasumasa port his reproducer program into a native
> JTREG test which was included in the push to JDK10/hs. I believe that
> infrastructure is only available in JDK9 and JDK10. He does have a
> standalone version of his reproducer.
> 
> Dan
> 
> 
> On 8/10/17 12:40 AM, David Buck wrote:
>> Hi Yasumasa!
>>
>> You will need to provide a new webrev based on the jdk8u-dev forest 
>> and get that reviewed on the relevant public alias (in this case, 
>> serviceability-dev). It at all possible, test cases should be modified 
>> as needed to work on JDK 8. Of course, you should also build and and 
>> test the JDK 8 version of the fix before requesting a review.
>>
>> Once the JDK 8 version of your fix passes code review, please send a 
>> new request for approval [0] this alias, jdk8u-dev. (It would probably 
>> be best to just reply to this thread.)
>>
>> Once both of the above are done correctly, I will be happy to push 
>> this change for you.
>>
>> Cheers,
>> -Buck
>>
>> [0] http://openjdk.java.net/projects/jdk8u/approval-template.html
>>
>> On 2017/08/10 15:29, Yasumasa Suenaga wrote:
>>> Hi David, (CC'ed to original reviewers)
>>>
>>>> This doesn't look like a strictly trivial backport. Have you made 
>>>> and tested
>>>> a diff against the the jdk8u-dev forest [0]?
>>>
>>> I could not import all of the change to jdk8u/hotspot because I could
>>> not apply the changes for testcase.
>>> However, essence of this change is for objectMonitor.cpp. It can apply.
>>>
>>> http://cr.openjdk.java.net/~ysuenaga/JDK-8185164/webrev.05/src/share/vm/runtime/objectMonitor.cpp.udiff.html 
>>>
>>>
>>> Cannot I request backport?
>>> I use this JVMTI API and event callback in my application. So I want
>>> to fix it in current Java releases.
>>>
>>>
>>> Thanks,
>>>
>>> Yasumasa
>>>
>>>
>>> P.S.
>>>    I'm also requesting it to the IcedTea 7. It seems to be accepted.
>>>      http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=3433
>>>
>>>
>>> 2017-08-10 14:51 GMT+09:00 David Buck <david.buck at oracle.com>:
>>>> Hi Yasumasa!
>>>>
>>>> Thank you for your submission.
>>>>
>>>> This doesn't look like a strictly trivial backport. Have you made 
>>>> and tested
>>>> a diff against the the jdk8u-dev forest [0]? Please see our homepage 
>>>> [1] for
>>>> details on how to work on the JDK 8 Updates Project.
>>>>
>>>> Cheers,
>>>> -Buck
>>>>
>>>> [0] http://hg.openjdk.java.net/jdk8u/jdk8u-dev
>>>> [1] http://openjdk.java.net/projects/jdk8u/
>>>>
>>>>
>>>> On 2017/08/10 13:12, Yasumasa Suenaga wrote:
>>>>>
>>>>> Hi all,
>>>>>
>>>>> Please approve the backport of the following fixe to JDK 8u:
>>>>>
>>>>> JBS:
>>>>>     https://bugs.openjdk.java.net/browse/JDK-8185164
>>>>>
>>>>> Review thread:
>>>>>
>>>>> http://mail.openjdk.java.net/pipermail/serviceability-dev/2017-August/021591.html 
>>>>>
>>>>>
>>>>> Webrev:
>>>>> http://cr.openjdk.java.net/~ysuenaga/JDK-8185164/webrev.05/
>>>>>
>>>>>
>>>>> I'm not a jdk8u committer. So I need a sponsor.
>>>>>
>>>>>
>>>>> Thanks,
>>>>>
>>>>> Yasumasa
>>>>>
>>>>
>>
>