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