Proposal to add Diagnostic option to enhance CDS testing

[Coleen Phillimore]

I support this proposal because it'll make it possible to add Class Data 
Sharing tests to the repository.  This might help find CDS issues in the 
future.   The test should be tolerant of not being able to map the 
archive because on some OS's you can't get the address you ask for.

Thanks,
Coleen


On 05/06/2013 04:06 PM, harold seigel wrote:
> Hi,
>
> Please review this proposal to add a JVM 
> -XX:+SharedArchiveFile=<file-path> diagnostic option.
>
> Currently, the CDS shared archive path is hardwired to be 
> <JVM_PATH>/classes.jsa.  This makes it difficult to write tests for 
> CDS because the tests will fail if the tester does not have write 
> privileges to <JVM_PATH>.  As a result, CDS is not tested as well as 
> it could be.  So, CDS bugs do not always get discovered as early as 
> possible.
>
> In order to make it easier to write CDS tests, I would like to add a 
> diagnostic option, called SharedArchiveFile.  The SharedArchiveFile 
> option would take a string specifying the path of the shared archive 
> file.  For example, this would specify that the shared archive file is 
> /tmp/my_classes.jsa.
>
>     java -XX:+UnlockDiagnosticVMOptions
>     -XX:SharedArchiveFile=/tmp/my_classes.jsa -Xshare:dump
>
> The option would need to be specified when writing the archive 
> (-Xshare:dump) and when reading it (-Xshare:<on|auto>).  The default 
> value would be <JVM_PATH>/classes.jsa.
>
> Click here <http://cr.openjdk.java.net/%7Ehseigel/SharedArchiveFile/> 
> to see a possible implementation and a sample test.
>
> Thanks, Harold

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/attachments/20130506/77488371/attachment-0001.html