On 2018-02-14 16:26, Mikhailo Seledtsov wrote:
Hi Igor,
Thank you for review. This is what I see from the test execution
logs on Windows:
// At archive creation time:
> -XX:SharedArchiveFile=T:/testoutput/jtreg/cds_archive.jsa -Xshare:dump
// At archive use time:
> -vmoption:-XX:+UnlockDiagnosticVMOptions
-vmoption:-XX:SharedArchiveFile=/cygdrive/t/testoutput/jtreg/cds_archive.jsa
-javaoptions:-Xshare:auto -javaoptions:-Xlog:class+path=trace
-javaoptions:-showversion
I am not a Windows expert, and especially not a wiz in Cygwin. Do you
know if these 2 paths are equivalent?
That's likely incorrect. Java will not be able to correctly interpred
/cygdrive/t. I believe the fix should be to do:
CDS_VM_ARGS := -XX:+UnlockDiagnosticVMOptions
-XX:SharedArchiveFile=$(shell $(GETMIXEDPATH) "$(CDS_ARCHIVE_FILE)")
instead.
Otherwise the patch looks OK to me.
/Magnus
[1] T:/testoutput/jtreg/cds_archive.jsa
[2] /cygdrive/t/testoutput/jtreg/cds_archive.jsa
If these are equivalent, then the code is OK as is. If not, I will
change it and rerun the tests. Please let me know.
Thank you,
Misha
On 2/13/18, 8:41 PM, Igor Ignatyev wrote:
Hi Misha,
have you check that it works on windows w/ GENERATE_CDS_ARCHIVE=true?
it seems that you should called GETMIXEDPATH in CDS_VM_ARGS:
CDS_VM_ARGS := -XX:+UnlockDiagnosticVMOptions
-XX:SharedArchiveFile=$(CDS_ARCHIVE_FILE)
vs
CDS_VM_ARGS := -XX:+UnlockDiagnosticVMOptions
-XX:SharedArchiveFile=$(shell $(GETMIXEDPATH) "$(CDS_ARCHIVE_FILE)")
otherwise, tests won't be able to find cds archive during test
execution.
-- Igor
On Feb 13, 2018, at 8:17 PM, Mikhailo
Seledtsov<mikhailo.seledt...@oracle.com> wrote:
Please review this small change that enables execution of any jtreg
test(s) in CDS mode
via make. Please see bug description for details.
Thanks to Erik and Igor for their help with this change.
JBS: https://bugs.openjdk.java.net/browse/JDK-8197906
WebRev: http://cr.openjdk.java.net/~mseledtsov/8197906.01/
Testing:
- ran hotspot_runtime in CDS mode with this change - Linux-x64
- no new failures
- running several test sets in CDS mode via distributed test
system - pass
- running pre-integration sanity testing (HS tier1, tier2) -
in progress
Thank you,
Misha