Avoid getting the GraalVM version at deployment time for AWT

Reduces the times we start a container when using a builder-image
zakkak · Oct 11, 2024 · ab9bb13 · ab9bb13
1 parent aea9c27
commit ab9bb13
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 18 deletions.
diff --git a/core/runtime/src/main/java/io/quarkus/runtime/util/JavaVersionUtil.java b/core/runtime/src/main/java/io/quarkus/runtime/util/JavaVersionUtil.java
@@ -12,6 +12,7 @@ public class JavaVersionUtil {
     private static boolean IS_GRAALVM_JDK;
     private static boolean IS_JAVA_16_OR_OLDER;
     private static boolean IS_JAVA_17_OR_NEWER;
+    private static boolean IS_JAVA_19_OR_NEWER;
 
     static {
         performChecks();
@@ -26,11 +27,13 @@ static void performChecks() {
             IS_JAVA_13_OR_NEWER = (first >= 13);
             IS_JAVA_16_OR_OLDER = (first <= 16);
             IS_JAVA_17_OR_NEWER = (first >= 17);
+            IS_JAVA_19_OR_NEWER = (first >= 19);
         } else {
             IS_JAVA_11_OR_NEWER = false;
             IS_JAVA_13_OR_NEWER = false;
             IS_JAVA_16_OR_OLDER = false;
             IS_JAVA_17_OR_NEWER = false;
+            IS_JAVA_19_OR_NEWER = false;
         }
 
         String vmVendor = System.getProperty("java.vm.vendor");
@@ -53,6 +56,10 @@ public static boolean isJava17OrHigher() {
         return IS_JAVA_17_OR_NEWER;
     }
 
+    public static boolean isJava19OrHigher() {
+        return IS_JAVA_19_OR_NEWER;
+    }
+
     public static boolean isGraalvmJdk() {
         return IS_GRAALVM_JDK;
     }

diff --git a/extensions/awt/deployment/src/main/java/io/quarkus/awt/deployment/AwtProcessor.java b/extensions/awt/deployment/src/main/java/io/quarkus/awt/deployment/AwtProcessor.java
@@ -128,12 +128,6 @@ JniRuntimeAccessBuildItem setupJava2DClasses(NativeImageRunnerBuildItem nativeIm
             Optional<ProcessInheritIODisabledBuildItem> processInheritIODisabledBuildItem) {
         nativeImageRunnerBuildItem.getBuildRunner()
                 .setup(processInheritIODisabled.isPresent() || processInheritIODisabledBuildItem.isPresent());
-        final GraalVM.Version v;
-        if (nativeImageRunnerBuildItem.getBuildRunner() instanceof NoopNativeImageBuildRunner) {
-            v = CURRENT;
-        } else {
-            v = nativeImageRunnerBuildItem.getBuildRunner().getGraalVMVersion();
-        }
         final List<String> classes = new ArrayList<>();
         classes.add("com.sun.imageio.plugins.jpeg.JPEGImageReader");
         classes.add("com.sun.imageio.plugins.jpeg.JPEGImageWriter");
@@ -258,18 +252,11 @@ JniRuntimeAccessBuildItem setupJava2DClasses(NativeImageRunnerBuildItem nativeIm
         // A new way of dynamically loading shared objects instead
         // of baking in static libs: https://github.com/oracle/graal/issues/4921
         classes.add("sun.awt.X11FontManager");
-        if (v.javaVersion.feature() != 19) {
-            classes.add("java.awt.GraphicsEnvironment");
-            classes.add("sun.awt.X11GraphicsConfig");
-            classes.add("sun.awt.X11GraphicsDevice");
-            classes.add("sun.java2d.SunGraphicsEnvironment");
-            classes.add("sun.java2d.xr.XRSurfaceData");
-        }
-
-        // Added for JDK 19+ due to: https://github.com/openjdk/jdk20/commit/9bc023220 calling FontUtilities
-        if (v.jdkVersionGreaterOrEqualTo("19")) {
-            classes.add("sun.font.FontUtilities");
-        }
+        classes.add("java.awt.GraphicsEnvironment");
+        classes.add("sun.awt.X11GraphicsConfig");
+        classes.add("sun.awt.X11GraphicsDevice");
+        classes.add("sun.java2d.SunGraphicsEnvironment");
+        classes.add("sun.java2d.xr.XRSurfaceData");
 
         return new JniRuntimeAccessBuildItem(true, true, true, classes.toArray(new String[0]));
     }

diff --git a/extensions/awt/runtime/src/main/java/io/quarkus/awt/runtime/graal/DarwinAwtFeature.java b/extensions/awt/runtime/src/main/java/io/quarkus/awt/runtime/graal/DarwinAwtFeature.java
@@ -6,6 +6,7 @@
 import org.graalvm.nativeimage.hosted.RuntimeClassInitialization;
 
 import io.quarkus.runtime.util.JavaVersionUtil;
+import org.graalvm.nativeimage.hosted.RuntimeJNIAccess;
 
 @Platforms({ Platform.DARWIN_AMD64.class, Platform.DARWIN_AARCH64.class })
 public class DarwinAwtFeature implements Feature {
@@ -15,5 +16,14 @@ public void afterRegistration(AfterRegistrationAccess access) {
             // Quarkus run time init for AWT in Darwin
             RuntimeClassInitialization.initializeAtRunTime("sun.lwawt.macosx");
         }
+
+        // Added for JDK 19+ due to: https://github.com/openjdk/jdk20/commit/9bc023220 calling FontUtilities
+        if (JavaVersionUtil.isJava19OrHigher()) {
+            try {
+                RuntimeJNIAccess.register(Class.forName("sun.font.FontUtilities"));
+            } catch (ClassNotFoundException e) {
+                throw new RuntimeException("Failed to register FontUtilities for JNI Access", e);
+            }
+        }
     }
 }