Testing disabling frame tracing for performance reasons

This commit is contained in:
Tim Wojtulewicz 2025-09-30 16:11:02 -07:00
parent 54c26f768f
commit 8ea7e0041b
2 changed files with 10 additions and 5 deletions

View file

@ -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;
}

View file

@ -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();