On Sun, 5 Oct 2025 13:00:44 GMT, Jonas Norlinder <[email protected]> wrote:
>> Hi all,
>>
>> This PR augments the CPU time sampling measurement capabilities that a user
>> can perform from Java code with the addition of
>> `MemoryMXBean.getGcCpuTime()`. With this patch it will be possible for a
>> user to measure process and GC CPU time during critical section or
>> iterations in benchmarks to name a few. This new method complements the
>> existing `OperatingSystemMXBean.getProcessCpuTime()` for a refined
>> understanding.
>>
>> `CollectedHeap::gc_threads_do` may operate on terminated GC threads during
>> shutdown, but thanks to JDK-8366865 by @walulyai we can piggyback on the new
>> `Universe::is_shutting_down`. I have implemented a stress-test
>> `test/jdk/java/lang/management/MemoryMXBean/GetGcCpuTime.java` that may
>> identify reading CPU time of terminated threads. Synchronizing on
>> `Universe::is_shutting_down` and `Heap_lock` resolves this problem.
>>
>> FWIW; To my understanding we don't want to add a
>> `Universe::is_shutting_down` check in gc_threads_do as this may introduce a
>> performance penalty that is unacceptable, therefore we must be careful about
>> the few places where external users call upon gc_threads_do and may race
>> with a terminating VM.
>>
>> Tested: test/jdk/java/lang/management/MemoryMXBean/GetGcCpuTime.java,
>> jdk/javax/management/mxbean hotspot/jtreg/vmTestbase/nsk/monitoring on Linux
>> x64, Linux aarch64, Windows x64, macOS x64 and macOS aarch64 with release
>> and fastdebug.
>
> Jonas Norlinder has updated the pull request incrementally with one
> additional commit since the last revision:
>
> Move from j.l.management to com.sun.management etc.
Or:
* @implNote Time calculation is implementation-specific, and may include
details
* such as CPU time for driver threads, workers, VM Operations and
string deduplication.
* The return value can be -1 if called when measurement is not
possible, such as during shutdown.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/27537#issuecomment-3405648131
