Distinguish why AOT is disabled post-restore

Signed-off-by: Irwin D'Souza <[email protected]>
eclipse-openj9 · Sep 26, 2024 · 34aeb1c · 34aeb1c
1 parent 670e183
commit 34aeb1c
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 6 deletions.
diff --git a/runtime/compiler/control/OptionsPostRestore.cpp b/runtime/compiler/control/OptionsPostRestore.cpp
@@ -540,7 +540,7 @@ J9::OptionsPostRestore::invalidateCompiledMethodsIfNeeded(bool invalidateAll)
    }
 
 void
-J9::OptionsPostRestore::disableAOTCompilation()
+J9::OptionsPostRestore::disableAOTCompilation(bool disabledPreCheckpoint)
    {
    static bool aotDisabled = false;
    if (aotDisabled)
@@ -567,7 +567,14 @@ J9::OptionsPostRestore::disableAOTCompilation()
    aotDisabled = true;
    _disableAOTPostRestore = true;
 
-   j9nls_printf(PORTLIB, (UDATA) J9NLS_WARNING, J9NLS_JIT_CHECKPOINT_RESTORE_AOT_DISABLED);
+   if (disabledPreCheckpoint)
+      {
+      j9nls_printf(PORTLIB, (UDATA) J9NLS_WARNING, J9NLS_JIT_CHECKPOINT_RESTORE_AOT_DISABLED_PRE_CHECKPOINT);
+      }
+   else
+      {
+      j9nls_printf(PORTLIB, (UDATA) J9NLS_WARNING, J9NLS_JIT_CHECKPOINT_RESTORE_AOT_DISABLED);
+      }
    }
 
 void
@@ -834,8 +841,10 @@ J9::OptionsPostRestore::processCompilerOptions()
    J9JavaVM *vm = _jitConfig->javaVM;
    PORT_ACCESS_FROM_JAVAVM(vm);
 
+   bool aotEnabledPreCheckpoint = TR_J9SharedCache::aotHeaderValidationDelayed() ? true : TR::Options::sharedClassCache();
+
    bool jitEnabled = TR::Options::canJITCompile();
-   bool aotEnabled = TR_J9SharedCache::aotHeaderValidationDelayed() ? true : TR::Options::sharedClassCache();
+   bool aotEnabled = aotEnabledPreCheckpoint;
 
    _argIndexXjit = FIND_AND_CONSUME_RESTORE_ARG(OPTIONAL_LIST_MATCH, J9::Options::_externalOptionStrings[J9::ExternalOptions::Xjit], 0);
    _argIndexXnojit = FIND_AND_CONSUME_RESTORE_ARG(OPTIONAL_LIST_MATCH, J9::Options::_externalOptionStrings[J9::ExternalOptions::Xnojit], 0);
@@ -848,7 +857,7 @@ J9::OptionsPostRestore::processCompilerOptions()
    // If -Xnoaot was specified pre-checkpoint, there is a lot of infrastructure
    // that needs to be set up. For now, ignore -Xaot post-restore if -Xnoaot
    // was specified pre-checkpoint.
-   if (aotEnabled)
+   if (aotEnabledPreCheckpoint)
       aotEnabled = (_argIndexXaot >= _argIndexXnoaot);
 
    if (!aotEnabled)
@@ -859,7 +868,7 @@ J9::OptionsPostRestore::processCompilerOptions()
    if (_disableAOTPostRestore)
       {
       aotEnabled = false;
-      disableAOTCompilation();
+      disableAOTCompilation(aotEnabledPreCheckpoint);
       }
 
    if (!jitEnabled)

diff --git a/runtime/compiler/control/OptionsPostRestore.hpp b/runtime/compiler/control/OptionsPostRestore.hpp
@@ -119,8 +119,11 @@ class OptionsPostRestore
 
    /**
     * \brief Helper method to disable further AOT compilation.
+    *
+    * \param disabledPreCheckpoint bool to indicate whther AOT was disable post
+    *                              restore or pre checkpoint.
     */
-   void disableAOTCompilation();
+   void disableAOTCompilation(bool disabledPreCheckpoint = false);
 
    /**
     * \brief Helper method to perform tasks prior to processing

diff --git a/runtime/nls/jitm/j9jit.nls b/runtime/nls/jitm/j9jit.nls
@@ -431,3 +431,10 @@ J9NLS_JIT_OPTIONS_PERCENT_OUT_OF_RANGE.sample_input_1=-XX:codecachetotalMaxRAMPe
 J9NLS_JIT_OPTIONS_PERCENT_OUT_OF_RANGE.sample_input_2=0.8
 J9NLS_JIT_OPTIONS_PERCENT_OUT_OF_RANGE.sample_input_3=25
 # END NON-TRANSLATABLE
+
+J9NLS_JIT_CHECKPOINT_RESTORE_AOT_DISABLED_PRE_CHECKPOINT=AOT load and compilation disabled pre-checkpoint and post-restore.
+# START NON-TRANSLATABLE
+J9NLS_JIT_CHECKPOINT_RESTORE_AOT_DISABLED_PRE_CHECKPOINT.explanation=The JVM was running in a mode pre-checkpoint that prevents AOT load and compilation.
+J9NLS_JIT_CHECKPOINT_RESTORE_AOT_DISABLED_PRE_CHECKPOINT.system_action=The compiler will not generate AOT code. AOT code from the shared class cache will not be used.
+J9NLS_JIT_CHECKPOINT_RESTORE_AOT_DISABLED_PRE_CHECKPOINT.user_response=Remove any (pre-checkpoint) option that would have resulted in this behaviour.
+# END NON-TRANSLATABLE