On Sat, 27 Sep 2025 11:18:58 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. This proposal will probably require discussion as to whether this is a property of a standard MXBean or a JDK-specific MXBean. It might be that GarbageCollectorMXBean is a better place for this. ------------- PR Comment: https://git.openjdk.org/jdk/pull/27537#issuecomment-3341792562
