RFR: 8316967: Correct the scope of vmtimer in UnregisteredClasses::load_class [v3]

Ioi Lam iklam at openjdk.org
Mon Oct 16 21:13:04 UTC 2023


On Mon, 16 Oct 2023 18:27:15 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:

>> Simple fix for removing the vmtimer from `UnregisteredClasses::load_class` and the corresponding `PerfClassTraceTime` constructor.
>> 
>> Passed tier1 testing.
>> --------------------
>> Update: correct the scope of the vmtimer instead of removing it.
>
> Calvin Cheung has updated the pull request incrementally with one additional commit since the last revision:
> 
>   use perf_app_classload_time

Should these be removed as well?


./share/classfile/classLoader.cpp:PerfCounter*    ClassLoader::_perf_sys_class_lookup_time = nullptr;
./share/classfile/classLoader.cpp:    NEWPERFTICKCOUNTER(_perf_sys_class_lookup_time, SUN_CLS, "lookupSysClassTime");
./share/classfile/classLoader.hpp:  static PerfCounter* _perf_sys_class_lookup_time;

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

PR Comment: https://git.openjdk.org/jdk/pull/16046#issuecomment-1765282328


More information about the hotspot-runtime-dev mailing list