Skip to content

Commit

Permalink
8315051: jdk/jfr/jvm/TestGetEventWriter.java fails with non-JVMCI GCs
Browse files Browse the repository at this point in the history
Reviewed-by: phh, egahlin
  • Loading branch information
shipilev committed Aug 31, 2023
1 parent 486fa08 commit 145d8bc
Showing 1 changed file with 25 additions and 4 deletions.
29 changes: 25 additions & 4 deletions test/jdk/jdk/jfr/jvm/TestGetEventWriter.java
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@
import jdk.vm.ci.runtime.JVMCI;

/**
* @test TestGetEventWriter
* @test id=default
* @key jfr
* @requires vm.hasJFR
* @library /test/lib
Expand All @@ -57,9 +57,6 @@
*
* @run main/othervm jdk.jfr.jvm.TestGetEventWriter
*
* @run main/othervm -XX:+UnlockExperimentalVMOptions -XX:+EnableJVMCI -Dtest.jvmci=true --add-exports=jdk.jfr/jdk.jfr.internal.event=ALL-UNNAMED
* jdk.jfr.jvm.TestGetEventWriter
*
* @run main/othervm/timeout=300 -Xint -XX:+UseInterpreter -Dinterpreted=true
* jdk.jfr.jvm.TestGetEventWriter
*
Expand All @@ -72,6 +69,30 @@
* @run main/othervm/timeout=300 -Xcomp -XX:TieredStopAtLevel=4 -XX:-TieredCompilation -XX:-UseInterpreter -Dinterpreted=false
* jdk.jfr.jvm.TestGetEventWriter
*/

/**
* @test id=jvmci
* @key jfr
* @requires vm.hasJFR
* @requires vm.jvmci
* @library /test/lib
* @modules jdk.internal.vm.ci/jdk.vm.ci.meta
* jdk.internal.vm.ci/jdk.vm.ci.runtime
*
* @compile PlaceholderEventWriter.java
* @compile PlaceholderEventWriterFactory.java
* @compile E.java
* @compile NonEvent.java
* @compile RegisteredTrueEvent.java
* @compile RegisteredFalseEvent.java
* @compile MyCommitRegisteredTrueEvent.java
* @compile MyCommitRegisteredFalseEvent.java
* @compile StaticCommitEvent.java
*
* @run main/othervm -XX:+UnlockExperimentalVMOptions -XX:+EnableJVMCI -Dtest.jvmci=true --add-exports=jdk.jfr/jdk.jfr.internal.event=ALL-UNNAMED
* jdk.jfr.jvm.TestGetEventWriter
*/

public class TestGetEventWriter {

static class InitializationEvent extends Event {
Expand Down

1 comment on commit 145d8bc

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.