RFR(T): 8230938: Deprecate MonitorBound

[David Holmes]

Looks good!

Thanks,
David

On 14/09/2019 6:06 am, Daniel D. Daugherty wrote:
> Greetings,
> 
> I have a trivial change to deprecate the -XX:MonitorBound=N option:
> 
>     JDK-8230938 Deprecate MonitorBound
>     https://bugs.openjdk.java.net/browse/JDK-8230938
> 
>     JDK-8230939 Deprecate MonitorBound (CSR)
>     https://bugs.openjdk.java.net/browse/JDK-8230939
> 
> The CSR has been approved.
> 
> Here's the trivial set of diffs:
> 
> $ hg diff
> diff -r f63f50a4bf43 src/hotspot/share/runtime/arguments.cpp
> --- a/src/hotspot/share/runtime/arguments.cpp    Thu Sep 12 11:07:35 
> 2019 -0700
> +++ b/src/hotspot/share/runtime/arguments.cpp    Thu Sep 12 20:44:45 
> 2019 -0400
> @@ -539,6 +539,7 @@
>     { "FlightRecorder",               JDK_Version::jdk(13), 
> JDK_Version::undefined(), JDK_Version::undefined() },
>     { "FieldsAllocationStyle",        JDK_Version::jdk(14), 
> JDK_Version::jdk(15), JDK_Version::jdk(16) },
>     { "CompactFields",                JDK_Version::jdk(14), 
> JDK_Version::jdk(15), JDK_Version::jdk(16) },
> +  { "MonitorBound",                 JDK_Version::jdk(14), 
> JDK_Version::jdk(15), JDK_Version::jdk(16) },
> 
>     // --- Deprecated alias flags (see also aliased_jvm_flags) - sorted 
> by obsolete_in then expired_in:
>     { "DefaultMaxRAMFraction",        JDK_Version::jdk(8), 
> JDK_Version::undefined(), JDK_Version::undefined() },
> diff -r f63f50a4bf43 src/hotspot/share/runtime/globals.hpp
> --- a/src/hotspot/share/runtime/globals.hpp    Thu Sep 12 11:07:35 2019 
> -0700
> +++ b/src/hotspot/share/runtime/globals.hpp    Thu Sep 12 20:44:45 2019 
> -0400
> @@ -713,7 +713,7 @@
>             "Use LWP-based instead of libthread-based synchronization 
> "       \
>             "(SPARC 
> only)")                                                   \
> \
> -  product(intx, MonitorBound, 0, "Bound Monitor 
> population")                \
> +  product(intx, MonitorBound, 0, "(Deprecated) Bound Monitor 
> population")   \
>             range(0, 
> max_jint)                                                \
> \
>     experimental(intx, MonitorUsedDeflationThreshold, 
> 90,                     \
> 
> Thanks, in advance, for any comments, questions or suggestions...
> 
> Dan
>