RFR: 8357396: Refactor nmethod::make_not_entrant to use Enum instead of "const char*" [v4]

Aleksey Shipilev shade at openjdk.org
Tue Jun 3 18:55:19 UTC 2025


On Tue, 3 Jun 2025 18:52:34 GMT, Cesar Soares Lucas <cslucas at openjdk.org> wrote:

>> Please review this refactor to transform the reasons for making an nmethod not entrant from `const char*` into enum values.
>> 
>> Tested on Linux x64 with JTREG tier1-3 in fastdebug and release mode.
>
> Cesar Soares Lucas has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Fix spacing, fix build.

src/hotspot/share/code/nmethod.cpp line 1971:

> 1969:     if (xtty != nullptr) {
> 1970:       ttyLocker ttyl;  // keep the following output all in one block
> 1971:       xtty->begin_elem("make_not_entrant thread='%zu' change_reason='%s'",

Wait, let's not change the actual key here. This is part of XML logging, AFAICS, so this might break some tools.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25338#discussion_r2124657484


More information about the hotspot-dev mailing list