RFR: 8351661: NMT: VMATree should support separate call-stacks for reserve and commit operations [v29]
Johan Sjölen
jsjolen at openjdk.org
Thu May 22 09:04:00 UTC 2025
On Thu, 22 May 2025 08:55:30 GMT, Afshin Zafari <azafari at openjdk.org> wrote:
>> test/hotspot/gtest/nmt/test_vmatree.cpp line 318:
>>
>>> 316: Tree tree;
>>> 317: VMATree::RegionData rd1(si[0], mtTest);
>>> 318: VMATree::RegionData rd2(si[1], mtNone);
>>
>> We keep using those later in and it is hard to keep track of what `rd1` is. I suggest we use better names for those, such as `rd_Test`, then I know what mem_tag that region uses without having to go back.
>>
>> I renamed other instances, like `call_stack_2` as well.
>>
>> I have changed lots of those and here is a diff file with this change: [gerard.diff.zip](https://github.com/user-attachments/files/20376150/gerard.diff.zip)
>
> Unfortunately, your patch couldn't be applied, since it cannot find some expected text in my local code.
It's a lot easier to apply a patch by merging with git branches. Send the branch that's on your github instead.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24028#discussion_r2102028775
More information about the hotspot-runtime-dev
mailing list