RFR: 7160570: Intrinsification support for tracing framework

Tom Rodriguez tom.rodriguez at oracle.com
Fri Apr 13 10:21:52 PDT 2012


It's unfortunate that we can't use sizeof(_thread_id) in a static context.

In osThread_solaris.hpp there's a type name mismatch.

+   static size_t thread_id_size()   { return sizeof(pid_t); }
    thread_t thread_id() const       { return _thread_id; }

Otherwise it looks good.

tom

On Apr 13, 2012, at 6:49 AM, Rickard Bäckman wrote:

> Thank you Staffan,
> 
> still need one Reviewer, any takers?
> 
> Thanks
> /R
> 
> On 04/13/2012 11:21 AM, Staffan Larsen wrote:
>> Looks good.
>> 
>> /Staffan
>> 
>> On 11 apr 2012, at 14:55, Rickard Bäckman wrote:
>> 
>>> Hi all,
>>> 
>>> can I get reviews for the following change where we add support for intrinsics for a couple of methods specified by the tracing framework?
>>> 
>>> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7160570
>>> 
>>> http://cr.openjdk.java.net/~rbackman/7160570/webrev
>>> 
>>> Thank you.
>>> /R
>> 
> 



More information about the serviceability-dev mailing list