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