RFR: 8347434: Richer VM operations events logging [v2]

Aleksey Shipilev shade at openjdk.org
Fri Jan 17 08:27:16 UTC 2025


On Mon, 13 Jan 2025 06:55:42 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Aleksey Shipilev has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
>> 
>>  - Simplify cause handling
>>  - A bit of white space
>>  - Merge branch 'master' into JDK-8347434-richer-vm-events
>>  - Fix
>
> src/hotspot/share/runtime/vmThread.cpp line 411:
> 
>> 409:            cause != nullptr ? " (" : "",
>> 410:            cause != nullptr ? cause : "",
>> 411:            cause != nullptr ? ")" : "");
> 
> Might be a little cleaner to use a separate print statement only if cause is not null.

Yes! See new commit.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23039#discussion_r1919728796


More information about the hotspot-runtime-dev mailing list