RFR: 8007806: Need a Throwables performance counter

Nils Loodin nils.loodin at oracle.com
Tue Feb 26 10:12:24 UTC 2013


On 02/26/2013 12:18 AM, Mandy Chung wrote:
> On 2/25/2013 8:30 AM, Nils Loodin wrote:
>> I changed it to
>>
>> sun.throwables.numThrowables
>>
>> in http://cr.openjdk.java.net/~nloodin/8007806/webrev.01/
>>
>> Is this better? However, bear in mind that it's not exactly specified
>> where this is going to be incremented from yet. Nothing in this change
>> states that it's going to be incremented from the Throwable constructor.
>
> I'm surprised to see adding a performance counter for an "external"
> instrumentation agent to use.   I understand that we agree that dynamic
> bytecode instrumentation is a more appropriate approach to instrument
> and count the number of thrown throwables.

Just to be clear, what I was trying to say that this review is just to 
add the counter, and the discussion on how and when to access it is 
something that I'd much rather have in context of a review of that code...

But yes, we do agree that dynamic bytecode instrumentation is a more 
appropriate aproach to count the number of throwables. But that code and 
the following review would depend on this one.

Regards,
Nils Loodin




More information about the core-libs-dev mailing list