RFR(M): 8030812 : Change the solaris DTrace implementation to use USDT2 instead of USDT1

[Staffan Larsen]

On 20 dec 2013, at 22:07, serguei.spitsyn at oracle.com wrote:

> Staffan,
> 
> This is nice change and cleanup.
> I do not see any issues.

Thanks.

> The make/bsd/makefiles/dtrace.make shows that the jhelper.d is disabled for bsd
> which means that the jstack action is not supported there.
> I'm not very familiar with the BSD DTrace implementation and wonder if you know any details.
> Does it mean there is no jstack action there, or just more work is needed to make the jhelper.d functional?
> Do we have a bug filed on this?

Short answer: I don’t know anything about jstack or jhelper.d.

/Staffan

> 
> 
> Thanks,
> Serguei
> 
> 
> On 12/20/13 3:58 AM, Staffan Larsen wrote:
>> The DTrace static probe implementation in Hotspot was written with an earlier version of DTrace. With newer versions, DTrace can create a header file from the contents of the .d file that describes the probes. This newer version (called USDT2) has been used on OS X. Because we have had both versions active, the code has ended up looking ugly because of all the extra macros.
>> 
>> This is a first step in cleaning that up by moving the solaris implementation to use USDT2. The remaining step before USDT1 can be removed is to change the Linux system tap implementation to also use USDT2.
>> 
>> What I have changed is:
>> - Update the solaris dtrace.make to generate the header files. I have used the same code as on bsd.
>> - While I was there, I removed a lot of commented out code from the bsd dtrace.make file.
>> - Updated the hotspot.d files on bsd and solaris so that they have the same contents.
>> - Fixed some compilation errors in compileBroker.cpp with const char*.
>> - May of the USDT2 macro invocations had an extra line break in them. This both looked ugly and confused the solaris compiler, so I removed them. This lead to a _lot_ of changes in jni.cpp - enough changes so that webrev couldn’t handle it, which is why some of the webrev views are broken for this file.
>> -
>> 
>> Testing: I have run the vm.dtrace.testlist on both Solaris and OS X.
>> 
>> webrev: http://cr.openjdk.java.net/~sla/8030812/webrev.00/
>> bug: https://bugs.openjdk.java.net/browse/JDK-8030812
>> 
>> Thanks,
>> /Staffan
>