RFR: 8291508: Fix some tests with "requires vm.jvmti & vm.continuations" [v4]
Serguei Spitsyn
sspitsyn at openjdk.org
Tue Aug 2 06:59:24 UTC 2022
On Tue, 2 Aug 2022 06:32:09 GMT, Ao Qi <aoqi at openjdk.org> wrote:
>> `vmTestbase/nsk/jvmti/GetThreadInfo/thrinfo001/TestDescription.java` and `vmTestbase/nsk/jvmti/RedefineClasses/StressRedefineVirtual/StressRedefineVirtual.java` are added or modified by [JDK-8284161](https://bugs.openjdk.org/browse/JDK-8284161), and they are failed if Loom or JVMTI is not supported.
>>
>> The issue could be reproduced by zero.
>
> Ao Qi has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
>
> - Merge branch 'master' into 8291508
> - XMLDecoder tests are filed in JDK-8291640
> - XMLDecoder update
>
> Co-authored-by: Alan Bateman <alanb at openjdk.org>
> - 8291508: Fix some tests with "requires vm.continuations"
I'm surprised these two tests have the `--enable-preview` flag.
@lmesnik Could you, please, take a look at these two tests?
Otherwise, the fix looks good to me.
Thanks,
Serguei
-------------
Marked as reviewed by sspitsyn (Reviewer).
PR: https://git.openjdk.org/jdk/pull/9677
More information about the serviceability-dev
mailing list