RFR: 8350854: Include thread counts in safepoint logging

Coleen Phillimore coleenp at openjdk.org
Tue Mar 4 13:31:03 UTC 2025


On Tue, 4 Mar 2025 13:20:54 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> Hi David, thanks for the review, I referred to the format used in safepoint [stats log](https://github.com/openjdk/jdk/pull/23838/files#diff-d61020d12394708828d066d097d823180c01b74d35d4c3e369aead062abc11efR913) where format `INT32_FORMAT_W(8)` is used for thread counts, but there is no need print thread count in fixed width with safepoint log, so I removed the fixed width from the format.
>
> The question now is, why not just `%d` then?

Maybe we should get rid of INT32_FORMAT too?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23838#discussion_r1979434281


More information about the hotspot-runtime-dev mailing list