RFR: JDK-8293997: Unify os::current_thread_cpu_time_info and os::thread_cpu_time_info
Matthias Baesken
mbaesken at openjdk.org
Mon Sep 19 13:17:45 UTC 2022
On Mon, 19 Sep 2022 12:09:55 GMT, David Holmes <dholmes at openjdk.org> wrote:
> @MBaesken this has been proposed before but I objected on the basis that the "info" function should be co-located with the actual "time" function, as the former describes the properties of the latter. As long as the time function is in OS specific code then the info function should be too IMO.
Hi David, I wonder about the following it seems we have just one caller per function , JvmtiEnv::GetCurrentThreadCpuTimerInfo and JvmtiEnv::GetThreadCpuTimerInfo .
Does it make much sense any more to have the functions at all, we could just move the coding to the callers?
-------------
PR: https://git.openjdk.org/jdk/pull/10335
More information about the hotspot-runtime-dev
mailing list