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

[David Buck]

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
>>>
>>