celerity / celerity-runtime

95%

master: 95%

LAST BUILD ON BRANCH fix-sched-exec-race

branch: fix-sched-exec-race

CHANGE BRANCH

x

Reset

• fix-sched-exec-race
• accp_flush_fix
• acpp-sscp
• active-transfer-cache
• add-matchbox-dependency
• async_kernel_launch
• benchmark-improvements
• bump-dpcpp
• bump-fmt-spdlog
• bump-tracy-0.11.1
• ci-badge
• ci-benchmark-ghcr
• ci-detect-missing-clang-format
• ci-stability
• clang-format-18
• clang-tidy-auto-resolve
• compression-demo
• config-header
• coverage
• cpp20
• disable-logging-in-benchmarks
• dpcpp-reenable-mimalloc
• drop-user-benchmarker
• editorconfig
• epoch-promise
• explicit-lifetime-mgmt
• fat-pushes
• find-executor-exception
• fix-accidental-lifetime-extension
• fix-acpp-cmake-dependency
• fix-bench-buffer-capture
• fix-cdag-test-print-graphs
• fix-dbq-tests
• fix-gdb-pretty-region-map-0d
• fix-rt-horizons-test
• fix-testspy-coverage
• generate-single-await-push
• hipsycl-disable-reductions
• idag-bump-sycl
• idag-comm
• idag-device-selection
• idag-execution
• idag-generate
• idag-nd-memory
• idag-ooo-engine
• idag-ooo-fix-reorder
• idag-refactor
• idag-runtime
• idag-threading
• iggen-buffer-opt
• improve-uninit-read-warning
• improve_executor_starvation_test
• instruction-graph-wip
• libenvpp-update
• macos_test_fix
• master
• multi-queue
• opt-polling
• optional-mpi
• parallel-for-integral
• pre-release-fixup
• prepare-0.6.0
• prepare-0.7.0
• refactor-accessor-tests
• refactor-bam
• refactor-cdag
• refactor-cggen
• region-mappers
• region-maps-with-offset
• remove-task-lookup
• remove-website
• rename-dggen
• report-executor-starvation
• revisit-dag-benchmarks
• simplify-feature-macros
• split-boxes
• staging
• struct-transfer-id
• sycl-cleanup
• tdag-testing-infrastructure
• test-dry-run-ho-fence
• thread-safe-simsycl
• tracy
• tracy-colors
• unreachable-coverage
• update-box
• 🥬

Build # 10301061556

Build Type

Pull #273

github

Committed by fknorr

Commit Message

Fix executor-scheduler race between instruction retirement and pruning

live_executor previously dereferenced instruction pointers on retirement,
which raced with their deletion on IDAG pruning when the executing
instruction was an horizon or epoch. To avoid future similar issues,
we now strictly regard the pointer to any instruction that has been issued
as dangling and work with instruction ids instead in those cases.

out_of_order_engine also had potentially hazardous uses of instruction
pointers that might - under the right circumstances - incorrectly regard
two instruction as identical when their pointers happen to alias as the
old one being freed before the new one was allocated in the same memory
location.

Pull Request Pull Request #273: Fix executor-scheduler race between instruction retirement and pruning

Run Details

2980 of 3378 branches covered (88.22%)

Branch coverage included in aggregate %.

22 of 23 new or added lines in 2 files covered. (95.65%)

6547 of 6660 relevant lines covered (98.3%)

1527404.49 hits per line