review request for 6608862 & 6459085

Xiaobin Lu Xiaobin.Lu at Sun.COM
Tue Aug 12 11:46:51 PDT 2008


Webrev: http://javaweb.sfbay/~xl116366/webrev/6459085/

Details:

This is a trivial fix for 6459085: naked pointer subtractions in class 
data sharing code. We should try to call pointer_delta instead of doing 
naked pointer operation to avoid surprise later. I also throw in a 
trivial fix to our linux make file. It seems like on Redhat 4, chcon 
command does not accept arguments to change the context for shared 
library and the OS in fact allows relocation to shared object file even 
with SELinux enabled. I guess that is because SELinux is in evolution 
and it is not as mature as later version such as RH5 etc. The failure 
execution of chcon command will cause VM build failure and exit. I 
actually removed the "exit 1" to let the build continue. I built and 
tested in dumbbell.east which is a Redhat 4 AS machine.


Webrev: http://javaweb.sfbay/~xl116366/webrev/6608862/

Details:

When some JVMTI operation such as stack walking is executed, 
Threads::threads_do will be called. While in the meantime, the VM is 
exiting as well and as a result, WatcherThread::watcher_thread returns 
NULL and the memory taken by it might be gone. To lock the WatcherThread 
in memory while do_thread is called on it, we grab the Terminator_lock 
which is also needs to be locked when WatcherThread exits and sets 
itself to NULL, this way, we can be certain that the WatcherThread and 
all its data structure is still there for reference. (thanks to David 
Holme's suggestion).

Verified by:
PRT.
Test came with the bug report passed.
NSK vm.full.testlist

Thanks in advance.

-Xiaobin



More information about the hotspot-dev mailing list