Hi Igor,
On 08/20/2020 09:23, Igor Ignatyev wrote:
HI Alex,
one minor nit: according to usual java coding conventions, isJVMTIIncluded
should be spelled as isJvmtiIncluded. otherwise the fix looks good to me.
I tried to be consistent with other methods like
isCDSIncludedInVmBuild, isJFRIncludedInVmBuild, isGCSupported,
isGCSelected, etc.
Maybe this should be isJVMTIIncludedInVmBuild..
Other tests will be updated in the follow-ups.
have you already identified all the tests which need this @requires? filed
bugs/RFEs for them?
Not yet.
I had problem with running all hotspot tests with minimal build (for
some reason jtreg was not able to complete it), so I decided start from
the tests mentioned in the jira issue and then test area-by-area, file
and fix the tests in batches.
--alex
Cheers,
-- Igor
On Aug 19, 2020, at 6:02 PM, Alex Menkov <[email protected]> wrote:
Hi all,
please review the fix for
https://bugs.openjdk.java.net/browse/JDK-8251384
webrev:
http://cr.openjdk.java.net/~amenkov/jdk16/minimal_jvmti/webrev/
The fix introduces new @requires option "vm.jvmti":
test/lib/sun/hotspot/WhiteBox.java
test/jtreg-ext/requires/VMProps.java
src/hotspot/share/prims/whitebox.cpp
test/hotspot/jtreg/TEST.ROOT
and updates tests in test/hotspot/jtreg/serviceability/jvmti (the only change in all
tests is added "@requires vm.jvmti")
Other tests will be updated in the follow-ups.
The
