RFR: 8364081: Shenandoah & GenShen logging improvement [v2]

[Rui Li]

On Fri, 25 Jul 2025 21:56:49 GMT, Y. Srinivas Ramakrishna <ysr at openjdk.org> wrote:

> I am wondering if we could just hoist the incrementing of the collections count to run_gc_cycle() which would normally take care of everything at once in one place, with the option true for stw and full cases.

I thought about moving them into `run_gc_cycle()` actually. However, some of the gc runs can be cancelled: 

https://github.com/openjdk/jdk/blob/ed455a803961f412c9e5855ec409e45980d64d00/src/hotspot/share/gc/shenandoah/shenandoahControlThread.cpp#L284-L286

While the total collection means collections that have been "started":

https://github.com/openjdk/jdk/blob/f53d0519b9a3a6eaf64fb938dfedc09795ac31f8/src/hotspot/share/gc/shared/collectedHeap.hpp#L389

So we have to put them into each individual method.


Updated for the other two comments.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/26469#issuecomment-3128171200