RFR: 8000973: SA on windows thread inspection is broken

serguei.spitsyn at oracle.com serguei.spitsyn at oracle.com
Thu Jan 31 11:10:17 PST 2013

Hi Yumin,

Looks good.

A couple of minor comments.

1. All the copyright comments are outdated.


   37   //The address argument must be the address of the OSThread::_thread_id

  Space is missed at the beginning of comment.

   39     this.debugger = debugger;
*   40     this.sysId   =  (long)addr.getCIntegerAt(0, 4, true);*

  The '=' is not aligned properly and extra space after '='.


On 1/30/13 11:16 PM, Yumin Qi wrote:
> Please have your comments on:
> http://cr.openjdk.java.net/~minqi/8000973/
> This only affected Windows platform.
> Summary: After bug 7161732, On Windows SA could not find correct 
> address of thread_id of OSThread since _thread_id moved to end of the 
> class . The presupposition of the address is following thread handle 
> no longer stands. Fix by adding thread_id field to OSThread and 
> getting the  address directly from OSThread.
> Reviewed-by:
> Contributed-by: yumin.qi at oracle.com
> Thanks
> Yumin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/attachments/20130131/abe743b6/attachment-0001.html 

More information about the hotspot-runtime-dev mailing list