Request for review: 8014138: Add VM option to facilitate the writing of CDS tests

[harold seigel]

Thanks Dan.

I'll fix up the typos.

Harold
On 5/9/2013 6:10 PM, Daniel D. Daugherty wrote:
>
> On 5/9/13 3:12 PM, harold seigel wrote:
>> Hi,
>>
>> Please review this updated webrev that incorporates the previous 
>> comments including fixing up strcpy and strcat.
>>
>> http://cr.openjdk.java.net/~hseigel/bug_8014138_2/ 
>> <http://cr.openjdk.java.net/%7Ehseigel/bug_8014138_2/>
> src/share/vm/runtime/arguments.cpp
> line 3184         file_sep_len + 20, mtInternal);
>     I think the '20' here could be '11' like it below, but 20 is safe.
>
> src/share/vm/runtime/globals.hpp
>     No comments.
>
> test/runtime/SharedArchiveFile/SharedArchiveFile.java
> line 27  * @summary Testing new -XX:ShareArchiveFile=<file-name> option
>     typo: 'ShareArchiveFile' -> 'SharedArchiveFile'
>
> line 36         "-XX:+UnlockDiagnosticVMOptions" , 
> "-XX:SharedArchiveFile=./sample.jsa", "-Xshare:dump");
>     space before ","
>
>
> Dan
>
>
>>
>> Thanks, Harold
>>
>> On 5/9/2013 3:13 PM, Zhengyu Gu wrote:
>>> Thanks.
>>>
>>> set_shared_archive_path more like get_shared_archive_path() ....
>>>
>>> Other than that, look good to me.
>>>
>>> -Zhengyu
>>>
>>> On 5/9/2013 2:56 PM, harold seigel wrote:
>>>> Hi Zhengyu,
>>>>
>>>> Thanks for looking at this.  Comments inline.
>>>>
>>>> Harold
>>>>
>>>> On 5/9/2013 2:35 PM, Zhengyu Gu wrote:
>>>>> Hi Harold,
>>>>>
>>>>> arguments.cpp
>>>>>
>>>>> 3186 - 3187: why not  strcat(shared_archive_path, "classes.jsa")
>>>> I'll clean that up.
>>>>>
>>>>> Also, I don't see where SharedArchiveFile is used.
>>>> In function set_shared_archive_path(), if SharedArchiveFile is null 
>>>> then the original archive file path is used.
>>>> Else the value of SharedArchiveFile is copied to 
>>>> shared_archive_path and returned as the function value.
>>>>>
>>>>> Thanks,
>>>>>
>>>>> -Zhengyu
>>>>>
>>>>>
>>>>>
>>>>> On 5/9/2013 1:44 PM, harold seigel wrote:
>>>>>> Hi,
>>>>>>
>>>>>> Please review this fix for bug 8014138.
>>>>>>
>>>>>> This enhancement adds a new diagnostic option, 
>>>>>> "-XX:SharedArchiveFile=<file-path>" that takes a string 
>>>>>> specifying the path of the shared archive file. Note that if the 
>>>>>> option is specified when the shared archive is written, then it 
>>>>>> must be used when the shared archive is being read (i.e. 
>>>>>> -Xshare:on).
>>>>>>
>>>>>> This change was tested with JCK lang and vm tests, UTE 
>>>>>> vm.quick.testlist and vm.mlvm.testlist tests, and jtreg tests. In 
>>>>>> addition, the test included in this change was run on Linux 
>>>>>> 32/64, Solaris 32/64, Windows 7, and Mac OS.
>>>>>>
>>>>>> Open webrev at http://cr.openjdk.java.net/~hseigel/bug_8014138/ 
>>>>>> <http://cr.openjdk.java.net/%7Ehseigel/bug_8014138/>
>>>>>>
>>>>>> Bug link at http://bugs.sun.com/view_bug.do?bug_id=8014138
>>>>>>
>>>>>> Thanks, Harold
>>>>>
>>>>
>>>
>>
>