RFR: 8312132: Add tracking of multiple address spaces in NMT [v49]

Johan Sjölen jsjolen at openjdk.org
Thu May 2 14:21:58 UTC 2024


On Sat, 27 Apr 2024 13:13:53 GMT, Gerard Ziemski <gziemski at openjdk.org> wrote:

>> That's a discussion that should take place in its own PR.
>
> I see 16 instances of the same patter:
> 
> 
>     assert_post_init();
>     if (!enabled()) return
> 
> 
> `in memTracker.hpp` so it's local and isolated to `MemTracker` class. I didn't think this would be controversial/big deal to warrant its own PR?

Maybe not, but this PR is already very large :). Changing this can be done in a separate PR and is not directly related to these changes, so it should be done in its separate PR.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18289#discussion_r1587722672


More information about the hotspot-dev mailing list