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