[jdk17u-dev] Integrated: 8277946: NMT: Remove VM.native_memory shutdown jcmd command option

[Thomas Stuefe]

On Wed, 25 Jan 2023 13:50:32 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

> Hi all,
> 
> This is *not* a clean downport. The upstream variant in JDK 18 outright removes the NMT shutdown dcmd option. For JDK 17, I leave it in but deprecate it and stub it out. See associated CSR backport: https://bugs.openjdk.org/browse/JDK-8299848 
> 
> Original issue: https://bugs.openjdk.org/browse/JDK-8277946
> Original CSR: https://bugs.openjdk.org/browse/JDK-8277947
> Backport CSR: https://bugs.openjdk.org/browse/JDK-8299848
> Backport Issue: https://bugs.openjdk.org/browse/JDK-8299846
> 
> 
> Differences to original CSR:
> 
> The original CSR was about removing the sub-option. That was fine for a head release.
> 
> This CSR, aimed at a maintenance release, proposes to deprecate it instead:
> 
>     jcmd would still accept the "shutdown" sub option and return 0 as exit code.
>     But jcmd should report "This option has been deprecated" instead of "NMT is shutting down"
> 
> 
>     Thanks!

This pull request has now been integrated.

Changeset: 229288bf
Author:    Thomas Stuefe <stuefe at openjdk.org>
URL:       https://git.openjdk.org/jdk17u-dev/commit/229288bf09ba6c37e92a678b250fbf8db9a5c87b
Stats:     124 lines in 4 files changed: 0 ins; 121 del; 3 mod

8277946: NMT: Remove VM.native_memory shutdown jcmd command option

Reviewed-by: mbaesken
Backport-of: abaa073bcbdb202658c8a97401ffb098d71e0f16

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

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