RFR (XS): 8021301: better event messages

[Albert Noll]

Hi Vladimir,

thanks for looking at the patch. I changed the output format. Do you 
think it is fine now?
Here is the new webrev:
http://cr.openjdk.java.net/~anoll/8021301/webrev.01/ 
<http://cr.openjdk.java.net/%7Eanoll/8021301/webrev.01/>

Best,
Albert


On 30.07.2013 10:24, Vladimir Kozlov wrote:
> Could you use format similar to one in print_cr()?
>
> Vladimir
>
> On 7/30/13 9:28 AM, Albert Noll wrote:
>> Hi all,
>>
>> please review this small patch.
>>
>> jbs: https://jbs.oracle.com/bugs/browse/JDK-8021301
>> webrev: http://cr.openjdk.java.net/~anoll/8021301/webrev.00/ 
>> <http://cr.openjdk.java.net/%7Eanoll/8021301/webrev.00/>
>>
>> Problem:
>> Having event message printing hex-address is not very helpful:
>>
>> Event: 57.947 Thread 0x00007f1f3e955800 Threw 0x00000008060c6e90 at
>> /HUDSON/workspace/2-build-linux-amd64/jdk8/4338/hotspot/src/share/vm/prims/jvm.cpp:1203 
>>
>>
>> Especially when the address is some temporary symbol address you 
>> can't not use in a debugger, like for class loading:
>>
>> EventMark m("loading class " INTPTR_FORMAT, (address)h_name);
>>
>> Solution:
>> Make output better readable.
>>
>> Best,
>> Albert

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20130730/6f05b3a0/attachment.html