Review request: 8009174: add class load event tracing

[Staffan Larsen]

On 1 mar 2013, at 07:57, David Holmes <David.Holmes at oracle.com> wrote:

> Hi Calvin,
> 
> So this adds the code that would allow for the event tracing but it doesn't actually add the code that will do the event tracing ???

It does add the code that creates the event and pushes it into the event tracing framework. This is all that is required to add a new event to the event tracing.

> Second problem: you can't store raw oops into the event. Even if classloader oops were somehow non-moving, the loader may not exist at the time the event is examined.

You can store raw oops into the event. We take care to make sure they are never referenced after the commit() is called. 

/Staffan

> 
> David
> 
> On 1/03/2013 9:24 AM, Calvin Cheung wrote:
>> This change is for adding the class load event tracing.
>> The event will include the following fields:
>> loaded class
>> defining class loader
>> initiating class loader
>> 
>> webrev: http://cr.openjdk.java.net/~ccheung/8009174/webrev/
>> 
>> thanks,
>> Calvin