RFR(S): 8191229: serviceability/jvmti/GetOwnedMonitorInfo/GetOwnedMonitorInfoTest.java fails with NoClassDefFoundError

[Chris Plummer]

Hi Yasumasa,

It is safe in my version because I move it to a global ref. See the 
NewGlobalRef() call in my webrev.

thanks,

Chris

On 12/7/17 6:14 AM, Yasumasa Suenaga wrote:
> Hi Chris,
>
> I added testcase of GetOwnedMonitorInfo in JDK-8185164.
> I have concern your change to cache the result of FindClass().
>
> According to [1], FindClass() calls find_class_from_class_loader(), it 
> returns JNI local value.
> Is it safe to cache?
>
> So I called FindClass each time in original commit.
>
>
> Thanks,
>
> Yasumasa
>
>
> [1] 
> http://hg.openjdk.java.net/jdk/hs/file/32fd4be602d5/src/hotspot/share/prims/jvm.cpp#l3450
>
>
> On 2017/12/07 16:44, Chris Plummer wrote:
>> New webrev:
>>
>> https://bugs.openjdk.java.net/browse/JDK-8191229
>> http://cr.openjdk.java.net/~cjplummer/8191229/webrev.01/
>>
>> testClass now initialized from JNI_OnLoad(), and use memset to clear
>> callbacks. Also updated to use JVMTI_VERSION_9 when calling GetEnv().
>>
>> thanks,
>>
>> Chris
>>