Integrated: 8362276: NMT tests should have locks for the entire tests
Gerard Ziemski
gziemski at openjdk.org
Wed Jul 16 15:30:48 UTC 2025
On Tue, 15 Jul 2025 16:14:52 GMT, Gerard Ziemski <gziemski at openjdk.org> wrote:
> We restructure the code to make sure we cover the code that uses the global tree instance (but nothing else that uses the same lock internally) as needed:
>
> - we move MemoryReserver::reserve() to the top to do as the first thing
> - we grab the MemTracker::NmtVirtualMemoryLocker nvml
> - do all the usual test stuff, including checking, which grabs the global instance of the tree
> - we move remove_all() to the very bottom to do as the last thing
>
> Testing: passes Mach5 tier1-4
This pull request has now been integrated.
Changeset: 10ae6029
Author: Gerard Ziemski <gziemski at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/10ae6029444c1381f7b1b3dcb6b6f32a4ae57efa
Stats: 31 lines in 1 file changed: 17 ins; 13 del; 1 mod
8362276: NMT tests should have locks for the entire tests
Reviewed-by: shade, coleenp
-------------
PR: https://git.openjdk.org/jdk/pull/26324
More information about the hotspot-runtime-dev
mailing list