Skip to content

Disable event reordering by default#464

Merged
nsavoire merged 1 commit into
mainfrom
nsavoire/disable_event_reordering
Oct 31, 2025
Merged

Disable event reordering by default#464
nsavoire merged 1 commit into
mainfrom
nsavoire/disable_event_reordering

Conversation

@nsavoire

Copy link
Copy Markdown
Collaborator

What does this PR do?

Additional latency introduced by event reordering causes dealloc events to be lost when the free rate is high.

Event reordering causes deallo events to be lost when the free rate is high.
@nsavoire
nsavoire requested a review from r1viollet as a code owner October 31, 2025 08:27
@pr-commenter

pr-commenter Bot commented Oct 31, 2025

Copy link
Copy Markdown

Benchmark results for collatz

Parameters

Baseline Candidate
config baseline candidate
profiler-version ddprof 0.22.0+6a42cd02.80781584 ddprof 0.22.0+648984ba.80846247

Summary

Found 0 performance improvements and 0 performance regressions! Performance is the same for 1 metrics, 0 unstable metrics.

See unchanged results
scenario Δ mean execution_time
scenario:ddprof -S bench-collatz --preset cpu_only collatz_runner.sh same

@pr-commenter

pr-commenter Bot commented Oct 31, 2025

Copy link
Copy Markdown

Benchmark results for BadBoggleSolver_run

Parameters

Baseline Candidate
config baseline candidate
profiler-version ddprof 0.22.0+6a42cd02.80781584 ddprof 0.22.0+648984ba.80846247

Summary

Found 0 performance improvements and 0 performance regressions! Performance is the same for 1 metrics, 0 unstable metrics.

See unchanged results
scenario Δ mean execution_time
scenario:ddprof -S bench-bad-boggle-solver BadBoggleSolver_run work 1000 same

@nsavoire
nsavoire merged commit 09e7f16 into main Oct 31, 2025
2 checks passed
@nsavoire
nsavoire deleted the nsavoire/disable_event_reordering branch October 31, 2025 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant