[Optimize] Add debug information for VMs

wqyfavor · jianliang00 · commit 6b5b768f464f · 2026-06-02T01:56:21.000+08:00
To better display VM information in trace, add debug information to MTS and BTS virtual machines.
diff --git a/core/runtime/js/jsi/jsc/jsc_context_group_wrapper.h b/core/runtime/js/jsi/jsc/jsc_context_group_wrapper.h
@@ -6,6 +6,7 @@
 
 #include <JavaScriptCore/JavaScript.h>
 
+#include <string>
 #include <unordered_map>
 
 #include "core/runtime/js/jsi/jsi.h"
@@ -17,7 +18,8 @@ class JSCContextGroupWrapper : public VMInstance {
  public:
   JSCContextGroupWrapper() = default;
   ~JSCContextGroupWrapper() override;
-  JSRuntimeType GetRuntimeType() override { return JSRuntimeType::jsc; }
+  JSRuntimeType GetRuntimeType() const override { return JSRuntimeType::jsc; }
+  std::string GetDebugDescription() const override { return "jsc"; }
 
   void InitContextGroup();
   inline JSContextGroupRef GetContextGroup() { return group_; }
diff --git a/core/runtime/js/jsi/jsi.h b/core/runtime/js/jsi/jsi.h
@@ -1734,7 +1734,8 @@ class VMInstance {
   // static VMInstance createVM();
  public:
   virtual ~VMInstance() = default;
-  virtual JSRuntimeType GetRuntimeType() = 0;
+  virtual JSRuntimeType GetRuntimeType() const = 0;
+  virtual std::string GetDebugDescription() const = 0;
 };
 
 class HostGlobal {
diff --git a/core/runtime/js/jsi/jsvm/jsvm_runtime_wrapper.h b/core/runtime/js/jsi/jsvm/jsvm_runtime_wrapper.h
@@ -7,6 +7,8 @@
 #include <ark_runtime/jsvm.h>
 #include <ark_runtime/jsvm_types.h>
 
+#include <string>
+
 #include "core/runtime/js/jsi/jsi.h"
 
 namespace lynx {
@@ -17,7 +19,8 @@ class JSVMRuntimeInstance : public VMInstance {
   JSVMRuntimeInstance() = default;
   ~JSVMRuntimeInstance() override;
 
-  JSRuntimeType GetRuntimeType() override { return JSRuntimeType::jsvm; };
+  JSRuntimeType GetRuntimeType() const override { return JSRuntimeType::jsvm; };
+  std::string GetDebugDescription() const override { return "jsvm"; }
 
   void InitInstance();
   JSVM_VM GetVM() const { return vm_; }
diff --git a/core/runtime/js/jsi/quickjs/quickjs_runtime_wrapper.cc b/core/runtime/js/jsi/quickjs/quickjs_runtime_wrapper.cc
@@ -138,6 +138,14 @@ void QuickjsRuntimeInstance::RemoveObserver(JSIObserver* obs) {
   obs_set_ptr_.erase(obs);
 }
 
+std::string QuickjsRuntimeInstance::GetDebugDescription() const {
+  if (rt_) {
+    return std::string("quickjs(") + (LEPUS_IsGCModeRT(rt_) ? "gc)" : "rc)");
+  } else {
+    return "";
+  }
+}
+
 void QuickjsRuntimeInstance::AddToIdContainer() {
   GetFunctionIdContainer().insert({rt_, s_function_id_});
   GetObjectIdContainer().insert({rt_, s_object_id_});
diff --git a/core/runtime/js/jsi/quickjs/quickjs_runtime_wrapper.h b/core/runtime/js/jsi/quickjs/quickjs_runtime_wrapper.h
@@ -86,7 +86,10 @@ class QuickjsRuntimeInstance : public VMInstance, public GCObserver {
   void AddObserver(JSIObserver* obs);
   void RemoveObserver(JSIObserver* obs);
 
-  JSRuntimeType GetRuntimeType() override { return JSRuntimeType::quickjs; }
+  JSRuntimeType GetRuntimeType() const override {
+    return JSRuntimeType::quickjs;
+  }
+  std::string GetDebugDescription() const override;
 
   // Must exec in use thread.
   void AddToIdContainer();
diff --git a/core/runtime/js/jsi/v8/v8_isolate_wrapper.h b/core/runtime/js/jsi/v8/v8_isolate_wrapper.h
@@ -4,6 +4,7 @@
 #ifndef CORE_RUNTIME_JS_JSI_V8_V8_ISOLATE_WRAPPER_H_
 #define CORE_RUNTIME_JS_JSI_V8_V8_ISOLATE_WRAPPER_H_
 
+#include <string>
 #include <unordered_map>
 
 #include "core/base/observer/observer_list.h"
@@ -25,7 +26,8 @@ class V8IsolateInstance : public VMInstance {
   // observers_.RemoveObserver(obs);
   // }
   virtual v8::Isolate* Isolate() const = 0;
-  JSRuntimeType GetRuntimeType() { return JSRuntimeType::v8; }
+  JSRuntimeType GetRuntimeType() const override { return JSRuntimeType::v8; }
+  std::string GetDebugDescription() const override { return "v8"; }
 };
 
 }  // namespace js
diff --git a/core/runtime/lepus/vm_context.h b/core/runtime/lepus/vm_context.h
@@ -58,6 +58,7 @@ class VMContext : public runtime::MTSContext {
   virtual runtime::ContextType Type() const override {
     return runtime::ContextType::VMContextType;
   }
+  virtual std::string GetDebugDescription() const override { return "lepus"; }
 
   void RegisterGlobalFunction(const runtime::RenderBindingFunction* funcs,
                               size_t size) override;
diff --git a/core/runtime/lepusng/quick_context.cc b/core/runtime/lepusng/quick_context.cc
@@ -683,6 +683,10 @@ bool QuickContext::ExecuteBinaryInternal(Value* ret_val) {
   return true;
 }
 
+std::string QuickContext::GetDebugDescription() const {
+  return std::string("quickjs(") + (LEPUS_IsGCModeRT(runtime_) ? "gc)" : "rc)");
+}
+
 void QuickContext::TriggerVmGC() {
   if (!gc_flag_) return;
   LEPUS_TrigGC(runtime_);
diff --git a/core/runtime/lepusng/quick_context.h b/core/runtime/lepusng/quick_context.h
@@ -108,6 +108,8 @@ class QuickContext : private LEPUSRuntimeData,
     return runtime::ContextType::LepusNGContextType;
   }
 
+  virtual std::string GetDebugDescription() const override;
+
   virtual void TriggerVmGC() override;
   virtual void UpdateGCTiming(bool is_start) override;
   virtual int64_t GetCurrentHeapSizeBytes() override;
diff --git a/core/runtime/mts_context.h b/core/runtime/mts_context.h
@@ -70,6 +70,7 @@ class MTSContext {
   virtual void Initialize() = 0;
 
   virtual ContextType Type() const = 0;
+  virtual std::string GetDebugDescription() const = 0;
 
   bool IsVMContext() const { return Type() == ContextType::VMContextType; }
   bool IsLepusNGContext() const {
diff --git a/core/shell/lynx_shell.cc b/core/shell/lynx_shell.cc
@@ -158,7 +158,11 @@ LynxShell::LynxShell(base::ThreadStrategyForRendering strategy,
 }
 
 LynxShell::~LynxShell() {
-  TRACE_EVENT(LYNX_TRACE_CATEGORY, LYNX_SHELL_DESTRUCTOR);
+  TRACE_EVENT(LYNX_TRACE_CATEGORY, LYNX_SHELL_DESTRUCTOR,
+              [instance_id = instance_id_](lynx::perfetto::EventContext ctx) {
+                ctx.event()->add_debug_annotations(INSTANCE_ID,
+                                                   std::to_string(instance_id));
+              });
   LOGI("LynxShell release, this:" << this);
   Destroy();
 }
diff --git a/core/shell/lynx_shell_builder.cc b/core/shell/lynx_shell_builder.cc
@@ -171,8 +171,16 @@ LynxShellBuilder& LynxShellBuilder::SetForceLayoutOnBackgroundThread(
 }
 
 LynxShell* LynxShellBuilder::build() {
+  if (this->shell_option_.instance_id_ == kUnknownInstanceId) {
+    this->shell_option_.instance_id_ = LynxShell::NextInstanceId();
+    this->shell_option_.page_options_.SetInstanceID(
+        this->shell_option_.instance_id_);
+  }
   TRACE_EVENT(LYNX_TRACE_CATEGORY, LYNX_SHELL_BUILDER_BUILD,
-              [&](lynx::perfetto::EventContext ctx) {
+              [&, instance_id = this->shell_option_.instance_id_](
+                  lynx::perfetto::EventContext ctx) {
+                ctx.event()->add_debug_annotations(INSTANCE_ID,
+                                                   std::to_string(instance_id));
                 ctx.event()->add_debug_annotations("thread_strategy",
                                                    std::to_string(strategy_));
                 ctx.event()->add_debug_annotations(
@@ -183,11 +191,6 @@ LynxShell* LynxShellBuilder::build() {
                     std::to_string(shell_option_.enable_js_group_thread_));
               });
   LynxShell* shell = new LynxShell(this->strategy_, this->shell_option_);
-  if (this->shell_option_.instance_id_ == kUnknownInstanceId) {
-    this->shell_option_.instance_id_ = shell->instance_id_;
-    this->shell_option_.page_options_.SetInstanceID(shell->instance_id_);
-  }
-
   shell->facade_actor_ = std::make_shared<LynxActor<NativeFacade>>(
       std::move(this->native_facade_), shell->runners_.GetUITaskRunner(),
       shell->instance_id_);
diff --git a/core/shell/runtime/bts/bts_runtime.h b/core/shell/runtime/bts/bts_runtime.h
@@ -194,6 +194,12 @@ class BTSRuntime final {
     }
   }
 
+  const std::string& GetGroupId() const { return group_id_; }
+
+  lynx::runtime::js::JSExecutor* GetJSExecutor() const {
+    return js_executor_.get();
+  }
+
 #if ENABLE_NAPI_BINDING
   void NotifyRuntimeReady(void* env, void* lynx);
 #endif
diff --git a/platform/darwin/ios/lynx_devtool/lynx_recorder/LynxRecorderActionManager.mm b/platform/darwin/ios/lynx_devtool/lynx_recorder/LynxRecorderActionManager.mm
@@ -835,7 +835,10 @@ - (void)avoidLynxViewBeingNull {
 
 - (void)loadTemplate:(NSDictionary*)params {
   [self avoidLynxViewBeingNull];
-  NSString* url = [[LynxRecorderEnv sharedInstance] lynxRecorderUrlPrefix];
+  NSString* url = [params objectForKey:@"url"];
+  if (!url) {
+    url = [[LynxRecorderEnv sharedInstance] lynxRecorderUrlPrefix];
+  }
   if (_globalPropsCache) {
     [_lynxView updateGlobalPropsWithTemplateData:[[LynxTemplateData alloc]
                                                      initWithDictionary:_globalPropsCache]];
@@ -919,7 +922,10 @@ - (void)loadTemplate:(NSDictionary*)params {
 
 - (void)loadTemplateBundle:(NSDictionary*)params {
   [self avoidLynxViewBeingNull];
-  NSString* url = [[LynxRecorderEnv sharedInstance] lynxRecorderUrlPrefix];
+  NSString* url = [params objectForKey:@"url"];
+  if (!url) {
+    url = [[LynxRecorderEnv sharedInstance] lynxRecorderUrlPrefix];
+  }
   if (_globalPropsCache) {
     [_lynxView updateGlobalPropsWithTemplateData:[[LynxTemplateData alloc]
                                                      initWithDictionary:_globalPropsCache]];

Original file line number	Diff line number	Diff line change
`@@ -58,6 +58,7 @@ class VMContext : public runtime::MTSContext {`
`58`	`58`	`virtual runtime::ContextType Type() const override {`
`59`	`59`	`return runtime::ContextType::VMContextType;`
`60`	`60`	`}`
	`61`	`+ virtual std::string GetDebugDescription() const override { return "lepus"; }`
`61`	`62`
`62`	`63`	`void RegisterGlobalFunction(const runtime::RenderBindingFunction* funcs,`
`63`	`64`	`size_t size) override;`
Original file line number	Diff line number	Diff line change
`@@ -108,6 +108,8 @@ class QuickContext : private LEPUSRuntimeData,`
`108`	`108`	`return runtime::ContextType::LepusNGContextType;`
`109`	`109`	`}`
`110`	`110`
	`111`	`+ virtual std::string GetDebugDescription() const override;`
	`112`	`+`
`111`	`113`	`virtual void TriggerVmGC() override;`
`112`	`114`	`virtual void UpdateGCTiming(bool is_start) override;`
`113`	`115`	`virtual int64_t GetCurrentHeapSizeBytes() override;`