RFR(XS): 8205126: JVMTI spec incorrectly states that PopFrame cannot be called on the current thread

[serguei.spitsyn at oracle.com]

Thanks a lot, Jc!
Serguei


On 6/4/19 6:36 PM, Jean Christophe Beyler wrote:
> Hi Serguei,
>
> Looks good to me too :)
> Jc
>
> On Tue, Jun 4, 2019 at 6:21 PM <serguei.spitsyn at oracle.com 
> <mailto:serguei.spitsyn at oracle.com>> wrote:
>
>     Thanks a lot, Alex!
>     Serguei
>
>     On 6/4/19 6:18 PM, Alex Menkov wrote:
>     > The fix looks good to me.
>     >
>     > --alex
>     >
>     > On 06/04/2019 16:30, serguei.spitsyn at oracle.com
>     <mailto:serguei.spitsyn at oracle.com> wrote:
>     >> Please, review a fix for the JVMTI spec bug:
>     >> https://bugs.openjdk.java.net/browse/JDK-8205126
>     >>
>     >> Webrev:
>     >>
>     http://cr.openjdk.java.net/~sspitsyn/webrevs/2019/8205126-jvmti-spec-popframe.1/
>     <http://cr.openjdk.java.net/%7Esspitsyn/webrevs/2019/8205126-jvmti-spec-popframe.1/>
>
>     >>
>     >>
>     >>
>     >> Related CSR:
>     >> https://bugs.openjdk.java.net/browse/JDK-8225142
>     >>
>     >> Specdiff:
>     >>
>     http://cr.openjdk.java.net/~sspitsyn/webrevs/2019/8205126-jvmti-spec-popframe.1/jvmti-specdiff/
>     <http://cr.openjdk.java.net/%7Esspitsyn/webrevs/2019/8205126-jvmti-spec-popframe.1/jvmti-specdiff/>
>
>     >>
>     >>
>     >>
>     >> Summary:
>     >>    The JVMTI PopFrame() spec does not match the implementation.
>     >>    It says the  specified thread can not be the current thread.
>     >>    The fix aligns:
>     >>     - spec with implementaion
>     >>     - PopFrame spec with ForceEarlyReturn spec
>     >>
>     >> Thanks,
>     >> Serguei
>
>
>
> -- 
>
> Thanks,
> Jc

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20190604/c2c3c878/attachment-0001.html>