Skip to content

Fix race condition in ThreadedLoop#3215

Merged
jdtournier merged 1 commit intomasterfrom
threaded_loop_fix_race_condition
Oct 15, 2025
Merged

Fix race condition in ThreadedLoop#3215
jdtournier merged 1 commit intomasterfrom
threaded_loop_fix_race_condition

Conversation

@jdtournier
Copy link
Member

@jdtournier jdtournier commented Oct 9, 2025

Issue identified by thread sanitizer while working on #3172.

@jdtournier jdtournier requested a review from a team October 9, 2025 18:09
@jdtournier jdtournier added the bug label Oct 9, 2025
@jdtournier jdtournier added this pull request to the merge queue Oct 15, 2025
Merged via the queue into master with commit 4de85a4 Oct 15, 2025
5 checks passed
@jdtournier jdtournier deleted the threaded_loop_fix_race_condition branch October 15, 2025 08:48
@jdtournier jdtournier mentioned this pull request Oct 20, 2025
Merged
@Lestropie
Copy link
Member

Lestropie commented Nov 18, 2025

@jdtournier How may commands beyond tckglobal may have been deleteriously affected by this one?

Lestropie added a commit that referenced this pull request Jan 12, 2026
@Lestropie
Manual propagation of master fixes to dev
34dfece 
Pull Requests that required manual code porting due to incompatibility of code bases between master and dev:
- #3148
- #3107
- #3181
- #3207
- #3195
- #3197
- #3205
- #3187
- #3050
- #3204
- #3215
- #3172
- #3233
Lestropie added a commit that referenced this pull request Jan 12, 2026
@Lestropie
Restore compilation following merging master to dev ##3255
c3e6507 
In particular this includes resolving #3215 and #2755.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@daljit46 daljit46 daljit46 approved these changes

Assignees

No one assigned

Labels

bug

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jdtournier @Lestropie @daljit46