Skip to content

Commit d5631d6

Browse files
jbachorikclaude
andcommitted
Remove process liveness check after perf stat
The benchmark runs in a loop and may restart between measurements. Perf data is still valid even if the specific PID changes. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
1 parent 42ab635 commit d5631d6

1 file changed

Lines changed: 3 additions & 5 deletions

File tree

ddprof-lib/benchmarks/branch-prediction/test_branch_prediction_perf.sh

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -163,11 +163,9 @@ run_perf_stat() {
163163

164164
log_info "perf stat results saved to ${output_file}"
165165

166-
# Verify process is still alive
167-
if ! kill -0 ${pid} 2>/dev/null; then
168-
log_error "Process ${pid} died during perf stat. Check /tmp/renaissance_*.log"
169-
return 1
170-
fi
166+
# Note: We don't verify the specific PID is still alive because the benchmark
167+
# may complete and restart within the measurement window. The wrapper loop
168+
# ensures continuous execution. We collected valid perf data during the window.
171169
}
172170

173171
# Run perf record for detailed analysis

0 commit comments

Comments
 (0)