RFR(XXS) JDK-8191042 Deprecate VM option CheckEndorsedAndExtDirs

[Ioi Lam]

Hi David,

Thanks for the review. The CSR has been approved, so I will push this.

Thanks

- Ioi


On 11/15/17 6:19 PM, David Holmes wrote:
> Hi Ioi,
>
> Looks good.
>
> Just need to wait for the CSR to be approved before pushing.
>
> Thanks,
> David
>
> On 16/11/2017 8:04 AM, Ioi Lam wrote:
>> Hi, please review this very small change:
>>
>>      https://bugs.openjdk.java.net/browse/JDK-8191042
>>
>> The endorsed standards override mechanism and the extensions mechanism
>> were removed in JDK 9 via JEP 220.
>>
>> The CheckEndorsedAndExtDirs option is disabled by default. However, if
>> it's enabled in the command-line, the VM would check the endorsed and 
>> ext
>> directories so that the VM terminates if they exist. These checks are no
>> longer needed in JDK 10 and beyond and can be removed.
>>
>> This task is the first step in the removal -- we need to mark this 
>> option
>> as deprecated and print a warning message when it's used. The option
>> itself, and the code that uses it, will be removed in a subsequent JDK
>> release.
>>
>> Tested with hotspot tier1/2
>>
>> Thanks
>> - Ioi
>>
>> ===================================
>>
>> diff -r d76a6042f5d7 src/hotspot/share/runtime/arguments.cpp
>> --- a/src/hotspot/share/runtime/arguments.cpp    Wed Nov 08 09:03:24 
>> 2017 -0800
>> +++ b/src/hotspot/share/runtime/arguments.cpp    Wed Nov 15 14:00:58 
>> 2017 -0800
>> @@ -383,6 +383,7 @@
>>     { "MinRAMFraction",               JDK_Version::jdk(10), 
>> JDK_Version::undefined(), JDK_Version::undefined() },
>>     { "InitialRAMFraction",           JDK_Version::jdk(10), 
>> JDK_Version::undefined(), JDK_Version::undefined() },
>>     { "IgnoreUnverifiableClassesDuringDump", JDK_Version::jdk(10), 
>> JDK_Version::undefined(), JDK_Version::undefined() },
>> +  { "CheckEndorsedAndExtDirs",      JDK_Version::jdk(10), 
>> JDK_Version::undefined(), JDK_Version::undefined() },
>>
>>     // --- 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() },
>>