Only enable recurring callback support on demand.

Author: christianhaeubl

sdk/src/org.graalvm.nativeimage/src/org/graalvm/nativeimage/Threading.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2017, 2019, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2017, 2025, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * The Universal Permissive License (UPL), Version 1.0
@@ -55,13 +55,22 @@ private Threading() {
     }
 
     /**
+     * This method is intended for expert users.
+     * <p>
      * Registers a {@link RecurringCallback callback handler} that is called by the current thread
-     * approximately at the provided interval. Only one callback can be active per thread. Each
-     * thread can have its own callback with a different interval (or none at all). No guarantees
-     * are made about the actual interval. For example, when the thread is waiting for a lock or
-     * executing native code, no callback can be done. Exceptions that are thrown during the
-     * execution of the callback are caught and ignored, unless they are thrown via a call to
-     * {@link RecurringCallbackAccess#throwException(Throwable)}.
+     * approximately at the provided interval. This functionality is only supported if the native
+     * binary is built with {@code -H:+SupportRecurringCallback}. Note that only carefully crafted,
+     * uninterruptible code can be executed safely in a recurring callback. Executing any other code
+     * can result in deadlocks or crashes.
+     * <p>
+     * Only one callback can be active per thread. Each thread can have its own callback with a
+     * different interval (or none at all). No guarantees are made about the actual interval. For
+     * example, when the thread is waiting for a lock or executing native code, no callback can be
+     * done.
+     * <p>
+     * Exceptions that are thrown during the execution of the callback and that are not caught in
+     * the callback are ignored. {@link RecurringCallbackAccess#throwException} can be used to
+     * explicitly throw an exception that is not ignored.
      * <p>
      * Specifying {@code null} for {@code callback} clears the current thread's callback (in which
      * case, the values of {@code interval} and {@code unit} are ignored).

substratevm/CHANGELOG.md

@@ -14,6 +14,7 @@ This changelog summarizes major changes to GraalVM Native Image.
 * (GR-58659) (GR-58660) Support for FFM API ("Panama") has been added for darwin-aarch64 and linux-aarch64.
 * (GR-49525) Introduced `--future-defaults=[all|<options>|none]` that enables options that are planned to become defaults in future releases. The enabled options are:
     1. `run-time-initialized-jdk` shifts away from build-time initialization of the JDK, instead initializing most of it at run time. This transition is gradual, with individual components of the JDK becoming run-time initialized in each release. This process should complete with JDK 29 when this option should not be needed anymore. Unless you store classes from the JDK in the image heap, this option should not affect you. In case this option breaks your build, follow the suggestions in the error messages.
+* (GR-63494) Recurring callback support is no longer enabled by default. If this feature is needed, please specify `-H:+SupportRecurringCallback` at image build-time.
 
 ## GraalVM for JDK 24 (Internal Version 24.2.0)
 * (GR-59717) Added `DuringSetupAccess.registerObjectReachabilityHandler` to allow registering a callback that is executed when an object of a specified type is marked as reachable during heap scanning.

substratevm/src/com.oracle.svm.core/src/com/oracle/svm/core/BuildPhaseProvider.java

@@ -34,6 +34,7 @@
 public final class BuildPhaseProvider {
 
     private boolean featureRegistrationFinished;
+    private boolean setupFinished;
     private boolean analysisFinished;
     private boolean hostedUniverseBuilt;
     private boolean readyForCompilation;
@@ -60,6 +61,14 @@ public static boolean isFeatureRegistrationFinished() {
         return ImageSingletons.contains(BuildPhaseProvider.class) && singleton().featureRegistrationFinished;
     }
 
+    public static void markSetupFinished() {
+        singleton().setupFinished = true;
+    }
+
+    public static boolean isSetupFinished() {
+        return ImageSingletons.contains(BuildPhaseProvider.class) && singleton().setupFinished;
+    }
+
     public static void markAnalysisFinished() {
         singleton().analysisFinished = true;
     }

substratevm/src/com.oracle.svm.core/src/com/oracle/svm/core/jfr/sampler/JfrRecurringCallbackExecutionSampler.java

@@ -54,13 +54,24 @@
 import com.oracle.svm.core.thread.VMThreads;
 import com.oracle.svm.core.util.TimeUtils;
 
+import jdk.graal.compiler.api.replacements.Fold;
+
 public final class JfrRecurringCallbackExecutionSampler extends AbstractJfrExecutionSampler {
     private static final ExecutionSampleCallback CALLBACK = new ExecutionSampleCallback();
 
     @Platforms(Platform.HOSTED_ONLY.class)
     JfrRecurringCallbackExecutionSampler() {
     }
 
+    @Fold
+    public static boolean isPresent() {
+        if (ImageSingletons.contains(JfrExecutionSampler.class)) {
+            JfrExecutionSampler sampler = ImageSingletons.lookup(JfrExecutionSampler.class);
+            return sampler instanceof JfrRecurringCallbackExecutionSampler;
+        }
+        return false;
+    }
+
     @Override
     protected void startSampling() {
         assert VMOperation.isInProgressAtSafepoint();

substratevm/src/com.oracle.svm.core/src/com/oracle/svm/core/thread/RecurringCallbackSupport.java

@@ -34,9 +34,11 @@
 import org.graalvm.nativeimage.Threading.RecurringCallback;
 import org.graalvm.nativeimage.Threading.RecurringCallbackAccess;
 
+import com.oracle.svm.core.BuildPhaseProvider;
 import com.oracle.svm.core.Uninterruptible;
 import com.oracle.svm.core.heap.RestrictHeapAccess;
 import com.oracle.svm.core.jdk.UninterruptibleUtils;
+import com.oracle.svm.core.jfr.sampler.JfrRecurringCallbackExecutionSampler;
 import com.oracle.svm.core.option.HostedOptionKey;
 import com.oracle.svm.core.threadlocal.FastThreadLocalFactory;
 import com.oracle.svm.core.threadlocal.FastThreadLocalInt;
@@ -65,7 +67,7 @@ public static class Options {
 
     public static class ConcealedOptions {
         @Option(help = "Support a per-thread timer that is called at a specific interval.") //
-        public static final HostedOptionKey<Boolean> SupportRecurringCallback = new HostedOptionKey<>(true);
+        public static final HostedOptionKey<Boolean> SupportRecurringCallback = new HostedOptionKey<>(false);
     }
 
     /**
@@ -78,7 +80,8 @@ public static class ConcealedOptions {
 
     @Fold
     public static boolean isEnabled() {
-        return ConcealedOptions.SupportRecurringCallback.getValue();
+        VMError.guarantee(BuildPhaseProvider.isSetupFinished(), "JfrRecurringCallbackExecutionSampler.isPresent() must not be called too early");
+        return ConcealedOptions.SupportRecurringCallback.getValue() || JfrRecurringCallbackExecutionSampler.isPresent();
     }
 
     public static RecurringCallbackTimer createCallbackTimer(long intervalNanos, RecurringCallback callback) {

substratevm/src/com.oracle.svm.hosted/src/com/oracle/svm/hosted/NativeImageGenerator.java

@@ -53,7 +53,6 @@
 import java.util.function.BooleanSupplier;
 import java.util.function.Function;
 
-import com.oracle.svm.core.FutureDefaultsOptions;
 import org.graalvm.collections.EconomicSet;
 import org.graalvm.collections.Pair;
 import org.graalvm.nativeimage.ImageInfo;
@@ -121,6 +120,7 @@
 import com.oracle.svm.core.BuildArtifacts;
 import com.oracle.svm.core.BuildPhaseProvider;
 import com.oracle.svm.core.ClassLoaderSupport;
+import com.oracle.svm.core.FutureDefaultsOptions;
 import com.oracle.svm.core.JavaMainWrapper.JavaMainSupport;
 import com.oracle.svm.core.LinkerInvocation;
 import com.oracle.svm.core.MissingRegistrationSupport;
@@ -1086,6 +1086,7 @@ protected void setupNativeImage(String imageName, OptionValues options, Map<Meth
                     FeatureImpl.DuringSetupAccessImpl config = new FeatureImpl.DuringSetupAccessImpl(featureHandler, loader, bb, debug);
                     featureHandler.forEachFeature(feature -> feature.duringSetup(config));
                 }
+                BuildPhaseProvider.markSetupFinished();
 
                 if (ImageLayerBuildingSupport.buildingExtensionLayer()) {
                     Heap.getHeap().setStartOffset(HostedImageLayerBuildingSupport.singleton().getLoader().getImageHeapSize());