RFR (M) 8140802 - Clean up and refactor of class loading code for CDS

Ioi Lam ioi.lam at oracle.com
Mon Nov 2 18:50:38 UTC 2015


On 10/30/15 1:23 PM, Jiangli Zhou wrote:
> Hi Ioi,
>
> The change looks pretty clean. The new src/share/vm/classfile/systemDictionary_ext.hpp and src/share/vm/classfile/vmSymbols_ext.hpp do not have copyright header. Please add the copyright headers. Please also fix the copyright year for the modified files prior to pushing.
I added the missing heads and fixed the copyright year for all modified 
files.
> In src/share/vm/classfile/sharedPathsMiscInfo.hpp, is the new “#include “classify/classLoader.hpp” necessary?
Yes, it's necessary for this part of the file

   static void trace_class_path(const char* msg, const char* name = NULL) {
     ClassLoader::trace_class_path(msg, name);
   }

I am not sure why it wasn't detected by the C compiler before, but I did 
have a build failure when building without precompiled headers.

Thanks
- Ioi

> I agree with the comments from Colleen and Karen regarding the usage of new_permanent_symbol() and TempNewSymbol in classLoaderExt.cpp. It doesn’t seem to be consistent.
>
> Thanks,
> Jiangli
>
>> On Oct 30, 2015, at 10:00 AM, Ioi Lam <ioi.lam at oracle.com> wrote:
>>
>> Please review the following fix:
>>
>> http://cr.openjdk.java.net/~iklam/8140802-cds-refactoring.v01/
>>
>> Bug: Clean up and refactor of class loading code for CDS
>>
>>     https://bugs.openjdk.java.net/browse/JDK-8140802
>>
>> Summary of fix:
>>
>>     We need to clean up and refactor the class loading code in order
>>     to support CDS in JDK9
>>
>>     [1] Remove code that has been made obsolete by the module changes
>>         (such as supporting code used for meta-index file)
>>     [2] Add new whitebox API to be used by CDS-related tests.
>>     [3] Refactor the parsing of classlist files for future enhancements.
>>     [4] Add new APIs in the class loading code to support Oracle CDS enhancements.
>>
>> Tests:
>>
>>     JPRT
>>     RBT - with same set of tests as hs-rt nightly
>>
>> Thanks
>> - Ioi




More information about the core-libs-dev mailing list