Proposal to add Diagnostic option to enhance CDS testing

[David Holmes]

I agree with Coleen, seems a useful and sound proposal.

David

On 7/05/2013 6:06 AM, 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