Transient IllegalArgumentException: Unknown language or instrument id com_oracle_truffle_api_test_polyglot_polyglotgctest_valuegclanguage.

Author: tzezula

truffle/src/com.oracle.truffle.api/src/com/oracle/truffle/api/TruffleLogger.java

@@ -1046,8 +1046,8 @@ private TruffleLogger getOrCreateLogger(final String loggerName) {
         }
 
         private TruffleLogger getOrCreateLogger(final String id, final String loggerName) {
-            if (!LanguageAccessor.ENGINE.isKnownLoggerId(id)) {
-                throw new IllegalArgumentException("Unknown language or instrument id " + id + ", known ids: " + String.join(", ", LanguageAccessor.ENGINE.getKnownLoggerIds()));
+            if (!LanguageAccessor.ENGINE.isKnownLoggerId(loggerCache, id)) {
+                throw new IllegalArgumentException("Unknown language or instrument id " + id + ", known ids: " + String.join(", ", LanguageAccessor.ENGINE.getKnownLoggerIds(loggerCache)));
             }
             final String globalLoggerId = loggerName == null || loggerName.isEmpty() ? id : id + '.' + loggerName;
             return getOrCreateLogger(globalLoggerId);

truffle/src/com.oracle.truffle.api/src/com/oracle/truffle/api/impl/Accessor.java

@@ -554,9 +554,9 @@ public abstract Thread createThread(Object polyglotLanguageContext, Runnable run
 
         public abstract LogRecord createLogRecord(Object loggerCache, Level level, String loggerName, String message, String className, String methodName, Object[] parameters, Throwable thrown);
 
-        public abstract boolean isKnownLoggerId(String id);
+        public abstract boolean isKnownLoggerId(Object loggerCache, String id);
 
-        public abstract Collection<String> getKnownLoggerIds();
+        public abstract Collection<String> getKnownLoggerIds(Object loggerCache);
 
         public abstract boolean isContextBoundLogger(Object loggerCache);
 

truffle/src/com.oracle.truffle.polyglot/src/com/oracle/truffle/polyglot/EngineAccessor.java

@@ -1435,16 +1435,45 @@ public LogRecord createLogRecord(Object loggerCache, Level level, String loggerN
         }
 
         @Override
-        public boolean isKnownLoggerId(String id) {
-            return PolyglotLoggers.getInternalIds().contains(id) || LanguageCache.languages().containsKey(id) || InstrumentCache.load().containsKey(id);
+        public boolean isKnownLoggerId(Object loggerCache, String id) {
+            if (PolyglotLoggers.getInternalIds().contains(id)) {
+                return true;
+            }
+            PolyglotEngineImpl polyglotEngine = lookupPolyglotEngineForLoggerCache(loggerCache);
+            if (polyglotEngine != null) {
+                return polyglotEngine.idToLanguage.containsKey(id) || polyglotEngine.idToInstrument.containsKey(id);
+            } else {
+                /*
+                 * TruffleLogger#getLogger() is invoked from a static initializer, no vmObject
+                 * anchor is available at this point.
+                 */
+                return LanguageCache.languages().containsKey(id) || InstrumentCache.load().containsKey(id);
+            }
+        }
+
+        private static PolyglotEngineImpl lookupPolyglotEngineForLoggerCache(Object loggerCache) {
+            VMObject vmObject = ((PolyglotLoggers.LoggerCache) loggerCache).getOwner();
+            if (vmObject == null) {
+                vmObject = PolyglotFastThreadLocals.getContext(null);
+            }
+            return vmObject != null ? vmObject.getEngine() : null;
         }
 
         @Override
-        public Collection<String> getKnownLoggerIds() {
-            List<String> ids = new ArrayList<>();
-            ids.addAll(PolyglotLoggers.getInternalIds());
-            ids.addAll(LanguageCache.languages().keySet());
-            ids.addAll(InstrumentCache.load().keySet());
+        public Collection<String> getKnownLoggerIds(Object loggerCache) {
+            List<String> ids = new ArrayList<>(PolyglotLoggers.getInternalIds());
+            PolyglotEngineImpl polyglotEngine = lookupPolyglotEngineForLoggerCache(loggerCache);
+            if (polyglotEngine != null) {
+                ids.addAll(polyglotEngine.idToLanguage.keySet());
+                ids.addAll(polyglotEngine.idToInstrument.keySet());
+            } else {
+                /*
+                 * TruffleLogger#getLogger() is invoked from a static initializer, no vmObject
+                 * anchor is available at this point.
+                 */
+                ids.addAll(LanguageCache.languages().keySet());
+                ids.addAll(InstrumentCache.load().keySet());
+            }
             Collections.sort(ids);
             return ids;
         }