RFR(L) JDK-8188791 Move AppCDS from closed repo to open repo

[mikhailo]

Test changes look good,


Misha


On 10/31/2017 07:43 PM, Ioi Lam wrote:
> Hi,
>
> Here's the new webrev for both the AppCDS implementation and tests. 
> During internal review of the JEP, we have decided to integrate both 
> implementation and tests at the same time.
>
> http://cr.openjdk.java.net/~iklam/jdk10/8188791-open-appcds.v02/
>
> As mentioned before, most of the "diffs" shown in this webrev are the 
> result of copying the closed source files on top of files of the same 
> name in the open repo. So in reviewing, instead of focusing on what's 
> "changed", please focus on the entire content of the new version of 
> each file.
>
> Testing: I did an OpenJDK linux-x64 build (without Oracle closed 
> sources) and all the new appcds tests passed.
>
> Thanks
>
> - Ioi
>
>
> On 10/30/17 8:52 AM, Ioi Lam wrote:
>> Hi Dmitry,
>>
>> In the latest JDK 10 repo, is_jrt has been renamed to 
>> is_modules_image. Please change the code accordingly.
>>
>> I will post my latest diff soon, with some test cases as well.
>>
>> Thanks
>>
>> - Ioi
>>
>>
>> On 10/30/17 4:04 AM, Dmitry Samersoff wrote:
>>> Ioi,
>>>
>>> I'd tried to apply your patch to latest open JDK10 and
>>> the compilation fails with:
>>>
>>> /root/dsamersoff/ESC/appcds/hs/src/hotspot/share/classfile/systemDictionaryShared.cpp:400:16: 
>>>
>>> error: ‘class SharedClassPathEntry’ has no member named ‘is_jrt’
>>>
>>> Did I miss something?
>>>
>>> -Dmitry
>>>
>>> On 13.10.2017 02:48, Ioi Lam wrote:
>>>> Hi,
>>>>
>>>> Please review this change set.
>>>>
>>>> http://cr.openjdk.java.net/~iklam/jdk10/8188791-open-appcds-impl.v01/
>>>>      https://bugs.openjdk.java.net/browse/JDK-8188791
>>>>
>>>> This is the first step of implementing the following JEP, which moves
>>>> AppCDS from
>>>> closed repos into the openjdk repo:
>>>>
>>>>      https://bugs.openjdk.java.net/browse/JDK-8185996
>>>>
>>>> In JDK 9, significant portion of AppCDS code resided in the closed 
>>>> repo.
>>>> As part
>>>> of the open-sourcing effort of JDK 18.3, we will move the source code
>>>> into the
>>>> open repo.
>>>>
>>>> In this changeset, the code is moved verbatim as much as possible. The
>>>> intention is
>>>> only to relocate the sources, not to changing existing behaviors, 
>>>> and not
>>>> to do any sort of refactoring.
>>>>
>>>> Most of the "diffs" shown in this webrev are the result of copying the
>>>> closed source
>>>> files on top of files of the same name in the open repo. So in
>>>> reviewing, instead of
>>>> focusing on what's "changed", it's better to focus on the entire 
>>>> content
>>>> of the new
>>>> version of each file.
>>>>
>>>> The only functional change in this task is that the UseAppCDS flag is
>>>> changed from
>>>> a "commercial" flag to a regular "product" flag. This is because
>>>> "commercial"
>>>> flags are not supported by the OpenJDK build.
>>>>
>>>> Source code refactoring may be desirable, because the old open/closed
>>>> source
>>>> code structure had introduced some intermediary APIs to connect code
>>>> between
>>>> the two repos. Such API should be removed in a separate RFE.
>>>>
>>>> Also, some AppCDS tests are currently in the closed repo. These tests
>>>> will be
>>>> moved in a separate task. See JDK-8188792 for details.
>>>>
>>>> All the AppCDS tests (currently still in closed sources) passed with
>>>> both Oracle JDK
>>>> and OpenJDK.
>>>>
>>>> Thanks
>>>> - Ioi
>>>
>>
>