[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Re: RFR (XXXS) 8173693 disable post_class_unload() for non JavaThread initiators
From:       "Daniel D. Daugherty" <daniel.daugherty () oracle ! com>
Date:       2017-01-31 18:37:35
Message-ID: 2084e9dd-3a49-dd6a-e891-3167231ef637 () oracle ! com
[Download RAW message or body]

Jerry,

Thanks for the fast review!

Dan


On 1/31/17 11:36 AM, Gerald Thornbrugh wrote:
> Hi Dan,
>
> Your changes look good.
>
> Thanks,
>
> Jerry
>> On Jan 31, 2017, at 10:18 AM, Daniel D. Daugherty <daniel.daugherty@oracle.com> wrote:
>>
>> Greetings,
>>
>> I have a very small work around for a JVM/TI event posting bug:
>>
>>     JDK-8173693 disable post_class_unload() for non JavaThread initiators
>>     https://bugs.openjdk.java.net/browse/JDK-8173693
>>
>> This work around disables the JVM/TI post_class_unload() function
>> when the thread requesting the event is not a JavaThread. See:
>>
>>     JDK-8173658 JvmtiExport::post_class_unload() is broken for non-JavaThread initiators
>>     https://bugs.openjdk.java.net/browse/JDK-8173658
>>
>> for the gory details of what can (and has) gone wrong.
>>
>> Fixing JDK-8173658 is non-trivial so this work around is proposed
>> to avoid allowing a Class Unload event to crash the VM or silently
>> corrupt the non JavaThread that tried to post the event.
>>
>> Webrev URL: http://cr.openjdk.java.net/~dcubed/8173693-webrev/0-jdk9-hs/
>>
>> Testing: Oracle internal VM/NSK JVM/TI test suite
>>          JPRT testing (in progress)
>>
>>
>> Thanks, in advance, for any comments, questions or suggestions.
>>
>> Dan

[prev in list] [next in list] [prev in thread] [next in thread]