Merge pull request #283 from singh264/openj9_issues_18399_and_18514

Add NotCheckpointSafe annotations to PhantomCleanable
ibmruntimes · Nov 27, 2023 · 7b49cd4 · 7b49cd4
2 parents 67b9bcb + 09c3a13
commit 7b49cd4
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 0 deletions.
diff --git a/closed/GensrcJ9JCL.gmk b/closed/GensrcJ9JCL.gmk
@@ -47,6 +47,7 @@ $(eval $(call SetupCopyFiles,COPY_OVERLAY_FILES, \
  src/java.base/share/classes/java/util/TimerTask.java \
  src/java.base/share/classes/java/util/concurrent/ConcurrentHashMap.java \
  src/java.base/share/classes/jdk/internal/access/JavaNetInetAddressAccess.java \
+ src/java.base/share/classes/jdk/internal/ref/PhantomCleanable.java \
  src/java.base/share/classes/sun/security/jca/ProviderConfig.java \
  src/java.base/share/classes/sun/security/jca/ProviderList.java \
  src/java.base/unix/classes/java/lang/ProcessEnvironment.java \

diff --git a/src/java.base/share/classes/jdk/internal/ref/PhantomCleanable.java b/src/java.base/share/classes/jdk/internal/ref/PhantomCleanable.java
@@ -22,6 +22,11 @@
  * or visit www.oracle.com if you need additional information or have any
  * questions.
  */
+/*
+ * ===========================================================================
+ * (c) Copyright IBM Corp. 2023, 2023 All Rights Reserved
+ * ===========================================================================
+ */
 
 package jdk.internal.ref;
 
@@ -30,6 +35,10 @@
 import java.lang.ref.PhantomReference;
 import java.util.Objects;
 
+/*[IF CRIU_SUPPORT]*/
+import openj9.internal.criu.NotCheckpointSafe;
+/*[ENDIF] CRIU_SUPPORT */
+
 /**
  * PhantomCleanable subclasses efficiently encapsulate cleanup state and
  * the cleaning action.
@@ -83,6 +92,9 @@ public PhantomCleanable(T referent, Cleaner cleaner) {
  /**
  * Insert this PhantomCleanable after the list head.
  */
+ /*[IF CRIU_SUPPORT]*/
+ @NotCheckpointSafe
+ /*[ENDIF] CRIU_SUPPORT */
  private void insert() {
  synchronized (list) {
  prev = list;
@@ -98,6 +110,9 @@ private void insert() {
  * @return true if Cleanable was removed or false if not because
  * it had already been removed before
  */
+ /*[IF CRIU_SUPPORT]*/
+ @NotCheckpointSafe
+ /*[ENDIF] CRIU_SUPPORT */
  private boolean remove() {
  synchronized (list) {
  if (next != this) {
@@ -116,6 +131,9 @@ private boolean remove() {
  *
  * @return true if the list is empty
  */
+ /*[IF CRIU_SUPPORT]*/
+ @NotCheckpointSafe
+ /*[ENDIF] CRIU_SUPPORT */
  boolean isListEmpty() {
  synchronized (list) {
  return list == list.next;