RFR: 8312132: Add tracking of multiple address spaces in NMT [v46]
Johan Sjölen
jsjolen at openjdk.org
Mon Apr 29 08:51:14 UTC 2024
On Tue, 23 Apr 2024 12:51:38 GMT, Afshin Zafari <azafari at openjdk.org> wrote:
>> Johan Sjölen has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Remove faulty condition after removing merging
>> - Add failing test case
>
> src/hotspot/share/nmt/nmtMemoryFileTracker.cpp line 48:
>
>> 46: NativeCallStackStorage::StackIndex sidx = _stack_storage.push(stack);
>> 47: DeviceSpace::Metadata metadata(sidx, flag);
>> 48: DeviceSpace::SummaryDiff diff = device->_tree.reserve_mapping(offset, size, metadata);
>
> What if `size == 0`?
Added test, tree should handle it.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18289#discussion_r1582722773
More information about the hotspot-dev
mailing list