Must use anchors for platform threads also when skipping virtual thread frames.

Author: peter-hofer

substratevm/src/com.oracle.svm.core/src/com/oracle/svm/core/jdk/StackTraceUtils.java

@@ -94,7 +94,7 @@ public static StackTraceElement[] getStackTraceAtSafepoint(Thread thread) {
     public static StackTraceElement[] getThreadStackTraceAtSafepoint(IsolateThread isolateThread, Pointer endSP) {
         assert VMOperation.isInProgressAtSafepoint();
         BuildStackTraceVisitor visitor = new BuildStackTraceVisitor(false, SubstrateOptions.MaxJavaStackTraceDepth.getValue());
-        JavaStackWalker.walkThread(isolateThread, endSP, visitor);
+        JavaStackWalker.walkThread(isolateThread, endSP, visitor, null);
         return visitor.trace.toArray(NO_ELEMENTS);
     }
 

substratevm/src/com.oracle.svm.core/src/com/oracle/svm/core/stack/JavaStackWalker.java

@@ -259,7 +259,7 @@ public static <T> boolean walkCurrentThread(Pointer startSP, ParameterizedStackF
         return walkCurrentThread(startSP, WordFactory.nullPointer(), FrameAccess.singleton().readReturnAddress(startSP), visitor, data);
     }
 
-    @Uninterruptible(reason = "Prevent deoptimization of stack frames while in this method.")
+    @Uninterruptible(reason = "Called from uninterruptible code.", mayBeInlined = true)
     public static <T> boolean walkCurrentThread(Pointer startSP, CodePointer startIP, ParameterizedStackFrameVisitor<T> visitor) {
         return walkCurrentThread(startSP, WordFactory.nullPointer(), startIP, visitor, null);
     }
@@ -275,10 +275,6 @@ public static boolean walkThread(IsolateThread thread, StackFrameVisitor visitor
         return walkThread(thread, visitor, null);
     }
 
-    public static boolean walkThread(IsolateThread thread, Pointer endSP, StackFrameVisitor visitor) {
-        return walkThread(thread, endSP, visitor, null);
-    }
-
     public static <T> boolean walkThread(IsolateThread thread, ParameterizedStackFrameVisitor<T> visitor, T data) {
         return walkThread(thread, WordFactory.nullPointer(), visitor, data);
     }
@@ -299,8 +295,6 @@ public static void walkThreadAtSafepoint(Pointer startSP, Pointer endSP, CodePoi
         assert VMOperation.isInProgressAtSafepoint();
         JavaStackWalk walk = StackValue.get(JavaStackWalk.class);
         initWalk(walk, startSP, endSP, startIP);
-        walk.setAnchor(WordFactory.nullPointer());
-        walk.setEndSP(endSP);
         doWalk(walk, visitor, null);
     }