From 8ea7e0041b231927abc07f00cffecae85a825ab0 Mon Sep 17 00:00:00 2001 From: Tim Wojtulewicz Date: Tue, 30 Sep 2025 16:11:02 -0700 Subject: [PATCH] Testing disabling frame tracing for performance reasons --- src/Func.cc | 9 ++++++--- src/zeek-setup.cc | 6 ++++-- 2 files changed, 10 insertions(+), 5 deletions(-) diff --git a/src/Func.cc b/src/Func.cc index e6047df71e..2a1247d725 100644 --- a/src/Func.cc +++ b/src/Func.cc @@ -339,7 +339,8 @@ ValPtr ScriptFunc::Invoke(zeek::Args* args, Frame* parent) const { f->SetTriggerAssoc(parent->GetTriggerAssoc()); } - g_frame_stack.push_back(f.get()); // used for backtracing + if ( ! g_policy_debug ) + g_frame_stack.push_back(f.get()); // used for backtracing const CallExpr* call_expr = parent ? parent->GetCall() : nullptr; call_stack.emplace_back(CallInfo{call_expr, this, *args}); @@ -391,7 +392,8 @@ ValPtr ScriptFunc::Invoke(zeek::Args* args, Frame* parent) const { catch ( InterpreterException& e ) { // Already reported, but now determine whether to unwind further. if ( Flavor() == FUNC_FLAVOR_FUNCTION ) { - g_frame_stack.pop_back(); + if ( ! g_policy_debug ) + g_frame_stack.pop_back(); call_stack.pop_back(); // Result not set b/c exception was thrown throw; @@ -448,7 +450,8 @@ ValPtr ScriptFunc::Invoke(zeek::Args* args, Frame* parent) const { g_trace_state.LogTrace("Function return: %s\n", d.Description()); } - g_frame_stack.pop_back(); + if ( ! g_policy_debug ) + g_frame_stack.pop_back(); return result; } diff --git a/src/zeek-setup.cc b/src/zeek-setup.cc index 070aad5308..898ce050a6 100644 --- a/src/zeek-setup.cc +++ b/src/zeek-setup.cc @@ -1105,7 +1105,8 @@ SetupResult setup(int argc, char** argv, Options* zopts) { auto [body, scope] = get_global_stmts(); StmtFlowType flow; Frame f(scope->Length(), nullptr, nullptr); - g_frame_stack.push_back(&f); + if ( ! g_policy_debug ) + g_frame_stack.push_back(&f); try { body->Exec(&f, flow); @@ -1113,7 +1114,8 @@ SetupResult setup(int argc, char** argv, Options* zopts) { reporter->FatalError("failed to execute script statements at top-level scope"); } - g_frame_stack.pop_back(); + if ( ! g_policy_debug ) + g_frame_stack.pop_back(); } clear_script_analysis();