RFR(T): 8230938: Deprecate MonitorBound

Daniel D. Daugherty daniel.daugherty at oracle.com
Fri Sep 13 22:52:37 UTC 2019


Thanks David!

Dan

On 9/13/19 6:43 PM, David Holmes wrote:
> 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
>>



More information about the hotspot-runtime-dev mailing list