RFR: 8356233: NMT: tty->print_cr should not be used in VirtualMemoryTracker::add_reserved_region() [v2]

Gerard Ziemski gziemski at openjdk.org
Tue May 20 16:05:38 UTC 2025


On Tue, 20 May 2025 10:18:18 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> src/hotspot/share/nmt/virtualMemoryTracker.cpp line 411:
>> 
>>> 409:                     "       new region: [" INTPTR_FORMAT "-" INTPTR_FORMAT "), memory tag %s.",
>>> 410:                     p2i(reserved_rgn->base()), p2i(reserved_rgn->end()), NMTUtil::tag_to_name(mem_tag_old),
>>> 411:                     p2i(base_addr), p2i(base_addr + size), NMTUtil::tag_to_name(mem_tag_new));
>> 
>> This change also now prints the name instead of the index. If you do print the name, then you don't need to use `index_to_tag` as they already are `MemTag`s.
>
> +1 on Johan's comment

Good catch.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25308#discussion_r2098339826


More information about the hotspot-runtime-dev mailing list