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

Reply via email to