rmid on Unix fails with Exception - maybe aftermath of JDK-8173607 ??
Claes Redestad
claes.redestad at oracle.com
Thu Feb 16 16:32:24 UTC 2017
Done, filed https://bugs.openjdk.java.net/browse/JDK-8175079 and pushed
a fix which also cleans up the hack in System.initPhase3.
/Claes
On 02/16/2017 05:17 PM, Alan Bateman wrote:
> Make sense for now, you can also change System.initPhase3 is not
> trigger eager initialization. I guess the rmid tests must be disabled
> as otherwise this would have been noticed, it's the odd man because it
> sets the security manager in its main method rather than on the
> command line.
>
> -Alan
>
> On 16/02/2017 16:11, Claes Redestad wrote:
>> Yes, verified that reverting to eager initialization of the system
>> image reader avoids this issue. I guess we need to back this
>> optimization out and see if we can improve things in the future:
>>
>> diff -r 32cb9898b630
>> src/java.base/share/classes/jdk/internal/module/ModuleBootstrap.java
>> ---
>> a/src/java.base/share/classes/jdk/internal/module/ModuleBootstrap.java
>> Thu Feb 16 17:05:48 2017 +0100
>> +++
>> b/src/java.base/share/classes/jdk/internal/module/ModuleBootstrap.java
>> Thu Feb 16 17:08:12 2017 +0100
>> @@ -115,12 +115,7 @@
>> long t0 = System.nanoTime();
>>
>> // system modules (may be patched)
>> - ModuleFinder systemModules;
>> - if (SystemModules.MODULE_NAMES.length > 0) {
>> - systemModules = SystemModuleFinder.getInstance();
>> - } else {
>> - systemModules = ModuleFinder.ofSystem();
>> - }
>> + ModuleFinder systemModules = ModuleFinder.ofSystem();
>>
>> PerfCounters.systemModulesTime.addElapsedTimeFrom(t0);
>
More information about the core-libs-dev
mailing list