[jdk17u-dev] RFR: 8277990: NMT: Remove NMT shutdown capability [v2]

Thomas Stuefe stuefe at openjdk.org
Tue Jan 31 11:01:04 UTC 2023


On Fri, 27 Jan 2023 09:40:40 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

>> This removes the NMT shutdown machinery from hotspot. For details, please see the associated JBS issue, as well as the companion change https://bugs.openjdk.org/browse/JDK-8277946 and CSR https://bugs.openjdk.org/browse/JDK-8277947.
>> 
>> This is not a clean backport, but the fixes are very minor:
>> 
>> - the original patch changes perfmemory_posix.cpp, but this hunk is not needed in JDK17
>> - A small fix was needed since the "shutdown" dcmd option in JDK17 had been deprecated out of carefulness, not outright removed: https://github.com/openjdk/jdk17u-dev/commit/5ce36d6c210d265a59548bde2f9e53b73bfa5c6f . 
>> 
>> Testing:
>> I manually ran runtime/NMT and all gtest variants for NMT on linux x64 and x86.
>> SAP nightlies are scheduled.
>
> Thomas Stuefe has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
> 
>  - Merge branch 'openjdk:master' into tstuefe-backport-5a036ace
>  - Fix remaining mention of shutdown in dcmd
>  - 8277990: NMT: Remove NMT shutdown capability
>    
>    Reviewed-by: stuefe, shade

GHA errors on Windows unrelated. Nightlies at SAP are clean.

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

PR: https://git.openjdk.org/jdk17u-dev/pull/1116


More information about the jdk-updates-dev mailing list