RFR: 8154110: Update class* and safepoint* logging subsystems

[Coleen Phillimore]

Hi,

Yes, his looks good, but I agree that class, loaderconstraints => class, 
loader, constraints
and class, loaderdata => class, loader, data

But I think constraints should be plural, we're logging the constraint 
checking (more than one constraint) so plural makes more sense to me.

Okay, I think I now prefer the combination of short words to do logging 
after seeing this change.

Thanks,
Coleen


On 4/26/16 3:44 PM, Robbin Ehn wrote:
> Hi Max,
>
> I was just looking at these tags and thought maybe they should be 
> split, nice !
>
> I have request for the ' constraints' tag, can make it singular 
> (without ending 's') ?
>
> (there are a few other ones in plural which we also should fix)
>
> Otherwise I think this looks good!
>
> Thanks!
>
> /Robbin
>
> On 04/26/2016 07:42 AM, Max Ockner wrote:
>> Hello,
>>
>> This change factors the tags from the class and safepoint logging 
>> subsystems into smaller tags, including "class" and "safepoint" tags 
>> which are included in tag combination in their respective subsystems.
>>
>> classresolve -> class+resolve
>> classload -> class+load
>> classunload -> class+unload
>> classpath -> class+path
>> classloaderdata -> class+loaderdata
>> classload+constraints -> class+loaderconstraints
>> classinit -> class+init
>> classload+preorder -> class+preorder
>>
>> safepointcleanup -> safepoint+cleanup
>>
>> class+loaderdata can be further factored into class+load+data. Same 
>> with class+loaderconstraints.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8154110
>> webrev(hotspot): http://cr.openjdk.java.net/~mockner/8154110.hotspot/
>> webrev(jdk): http://cr.openjdk.java.net/~mockner/8154110.jdk/
>>
>> Tested with jtreg hotspot tests. jdk tests currently running.
>>
>> Thanks,
>> Max
>