Skip to content

Commit

Permalink
8326127: JFR: Add SafepointCleanupTask to hardToTestEvents of TestLoo…
Browse files Browse the repository at this point in the history
…kForUntestedEvents

Reviewed-by: mgronlun
  • Loading branch information
D-D-H committed Feb 20, 2024
1 parent 7d32a1a commit 9fbcc00
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2018, 2023, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2018, 2024, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
Expand Down Expand Up @@ -56,7 +56,8 @@ public class TestLookForUntestedEvents {
Arrays.asList(
"DataLoss", "IntFlag", "ReservedStackActivation", "NativeLibraryUnload",
"DoubleFlag", "UnsignedLongFlagChanged", "IntFlagChanged",
"UnsignedIntFlag", "UnsignedIntFlagChanged", "DoubleFlagChanged")
"UnsignedIntFlag", "UnsignedIntFlagChanged", "DoubleFlagChanged",
"SafepointCleanupTask")
);

// GC uses specific framework to test the events, instead of using event names literally.
Expand Down
3 changes: 1 addition & 2 deletions test/jdk/jdk/jfr/event/runtime/TestSafepointEvents.java
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2013, 2023, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2013, 2024, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
Expand Down Expand Up @@ -52,7 +52,6 @@ public class TestSafepointEvents {
static final String[] EVENT_NAMES = new String[] {
EventNames.SafepointBegin,
EventNames.SafepointStateSynchronization,
// EventNames.SafepointCleanupTask,
EventNames.SafepointCleanup,
EventNames.SafepointEnd
};
Expand Down

1 comment on commit 9fbcc00

@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.