On Tue, 28 Oct 2025 12:53:59 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 three
> additional commits since the last revision:
>
> - Also stress-test Serial and Epsilon
> - Add basic test for MemoryMXBean.getTotalGcCpuTime
> - Change how test is invoked and fix review comment
Couple of grammar nits in the getTotalGcCpuTime javadoc.
"in GC is highly implementation" => "in GC are highly implementation"
"The default implementation return {@code -1}" => "The default implementation
returns {@code -1}"
-------------
PR Review: https://git.openjdk.org/jdk/pull/27537#pullrequestreview-3394838128
