RFR: 8366658: Add missing locks when accessing the VirtualMemoryTracker instance in tests and MemMapPrinter [v2]

[Johan Sjölen]

On Fri, 12 Sep 2025 08:06:52 GMT, Paul Hübner <phubner at openjdk.org> wrote:

>> Johan Sjölen has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update src/hotspot/share/nmt/virtualMemoryTracker.hpp
>>   
>>   Co-authored-by: David Holmes <62092539+dholmes-ora at users.noreply.github.com>
>
> src/hotspot/share/nmt/memTracker.hpp line 183:
> 
>> 181:   }
>> 182: 
>> 183:   static inline MemoryFileTracker::MemoryFile* register_file(const char* descriptive_name) {
> 
> Same here. I wonder why we have the inline header when it only contains `MemTracker::check_exceeds_limit`...

Hmm, so do I. I've never used the inline header myself. I think moving stuff over to the inline header is something we should think about in a separate PR.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27038#discussion_r2343536623