RFR: 8264644: Add PrintClassLoaderDataGraphAtExit to print the detailed CLD graph [v4]

Aleksey Shipilev shade at openjdk.java.net
Mon Apr 12 06:23:35 UTC 2021


On Wed, 7 Apr 2021 06:48:52 GMT, Yi Yang <yyang at openjdk.org> wrote:

>> Trivial chanage to make debugging happy, now cld->print() would be:
>> 
>> ClassLoaderData(0x00007ff17432b670)
>>  - name                'platform'
>>  - holder              WeakHandle: 0x00000007fef56678
>>  - class loader        0x7ff17432b828
>>  - metaspace           0x7ff17468a0b0
>>  - unloading           false
>>  - class mirror holder false
>>  - modified oops       true
>>  - keep alive          0
>>  - claim               strong
>>  - handles             43
>>  - dependency count    0
>>  - klasses             {java.sql.SQLFeatureNotSupportedException,java.sql.SQLNonTransientException,java.sql.SQLException,sun.util.resources.provider.NonBaseLocaleDataMetaInfo,org.ietf.jgss.GSSException,javax.sql.DataSource,java.sql.Wrapper,javax.sql.CommonDataSource,java.sql.Time,java.sql.Date,java.sql.Timestamp,sun.util.resources.cldr.provider.CLDRLocaleDataMetaInfo, }
>>  - packages            0x7ff17432bc20
>>  - module              0x7ff17432c520
>>  - unnamed module      0x7ff17432c3d0
>>  - dictionary          0x7ff17432c470
>>  - deallocate list     0x7ff0a4023bd0
>
> Yi Yang has updated the pull request incrementally with one additional commit since the last revision:
> 
>   INTPTR_FORMAT

Looks good with a minor nit.

src/hotspot/share/classfile/classLoaderData.cpp line 943:

> 941:   }
> 942: };
> 943: void ClassLoaderData::print_on(outputStream* out) const {

These declarations should be separated by a newline, I think?

-------------

Marked as reviewed by shade (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/3323


More information about the hotspot-dev mailing list