RFR(XS) 8249096: Clean up code for DumpLoadedClassList
Yumin Qi
yumin.qi at oracle.com
Fri Aug 21 19:05:03 UTC 2020
Hi, Dan
Thanks
Yumin
On 8/21/20 12:02 PM, Daniel D. Daugherty wrote:
> On 8/21/20 2:52 PM, Yumin Qi wrote:
>> Hi, Ioi
>>
>> Thanks! Do I need second review? I think I could push since it is XS.
>
> It is Trivial reviews that only require a single (R)eviewer and don't
> have to wait for 24 hours. And your single reviewer has to agree that
> the review is Trivial.
>
> Hold on and I'll be your second reviewer.
>
> Dan
>
>>
>>
>> Thanks
>>
>> Yumin
>>
>> On 8/21/20 9:48 AM, Ioi Lam wrote:
>>> Hi Yumin,
>>>
>>> This new version looks good to me. Thanks!
>>>
>>> - Ioi
>>>
>>> On 8/20/20 4:28 PM, Yumin Qi wrote:
>>>> Hi, Ioi
>>>>
>>>> updated at new link: http://cr.openjdk.java.net/~minqi/2020/8249096/webrev-02/
>>>>
>>>> Moved the call to 'log_to_classlist' to beginning of 'print_class_load_logging'. Removed the check for if class+load logging is enabled, since it will by pass the output for DumpLoadedClassList if not enabled.
>>>>
>>>> In 'print_class_load_logging', there exists a check for if the log is enabled.
>>>>
>>>> Thanks
>>>>
>>>> Yumin
>>>>
>>>>
>>>> On 8/20/20 1:57 PM, Yumin Qi wrote:
>>>>> Ioi,
>>>>>
>>>>> Thanks for the review, I will update with new webrev --- yes, print_class_load_logging will work the same way.
>>>>>
>>>>>
>>>>> Thanks
>>>>>
>>>>> YUmin
>>>>>
>>>>> On 8/20/20 11:37 AM, Ioi Lam wrote:
>>>>>> Hi Yumin,
>>>>>>
>>>>>> This looks like a good clean up. I think it can be further simplified by moving the call to InstanceKlass::log_to_classlist to beginning of InstanceKlass::print_class_load_logging().
>>>>>>
>>>>>> Thanks
>>>>>> - Ioi
>>>>>>
>>>>>> On 8/20/20 10:46 AM, Yumin Qi wrote:
>>>>>>> Hi, Please review the very small change for cleaning up DumpLoadedClassList code.
>>>>>>>
>>>>>>>
>>>>>>> bug: https://bugs.openjdk.java.net/browse/JDK-8249096
>>>>>>>
>>>>>>> Webrev: http://cr.openjdk.java.net/~minqi/2020/8249096/webrev-01/
>>>>>>>
>>>>>>>
>>>>>>> There are two places where when DumpLoadedClassList is set we log loaded class to log file, one is in ClassFileParser::parse_stream, and the other is in SystemDictionaryShared::load_shared_class_misc which is after shared class loaded from CDS. The former is positioned after parsing constants, and before parsing interface, fields and methods etc. There is possibility an exception or error happens before the InstanceKlass is created, so the fix move the logging to after the InstanceKlass is successfully created, also move the logging code to InstanceKlass which seems more reasonable.
>>>>>>>
>>>>>>>
>>>>>>> Test: local jtreg on cds.
>>>>>>>
>>>>>>> Mach5 tier1-4
>>>>>>>
>>>>>>>
>>>>>>> Thanks
>>>>>>>
>>>>>>> Yumin
>>>>>>>
>>>>>>
>>>
>
More information about the hotspot-runtime-dev
mailing list