Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add pending state back to jl_thread_suspend_and_get_state-machine #55622

Merged
merged 1 commit into from
Aug 29, 2024

Conversation

vtjnash
Copy link
Member

@vtjnash vtjnash commented Aug 29, 2024

Fixes an issue with #55500, where signals may abruptly abort the process as they observe it is still processing the resume SIGUSR2 message and are not able to wait for that processing to end before setting the new message to exit.

@vtjnash vtjnash added the regression 1.12 Regression in the 1.12 release label Aug 29, 2024
@vtjnash vtjnash force-pushed the jn/sigusr2-processing branch 2 times, most recently from ae8043a to 5120d05 Compare August 29, 2024 03:04
Fixes an issue with #55500, where signals may abruptly abort the process
as they observe it is still processing the resume SIGUSR2 message and
are not able to wait for that processing to end before setting the new
message to exit.
@vtjnash vtjnash force-pushed the jn/sigusr2-processing branch from 5120d05 to c13053d Compare August 29, 2024 16:47
@vtjnash vtjnash merged commit da3468c into master Aug 29, 2024
4 of 7 checks passed
@vtjnash vtjnash deleted the jn/sigusr2-processing branch August 29, 2024 18:45
KristofferC pushed a commit that referenced this pull request Sep 12, 2024
…5622)

Fixes an issue with #55500, where signals may abruptly abort the process
as they observe it is still processing the resume SIGUSR2 message and
are not able to wait for that processing to end before setting the new
message to exit.
IanButterworth pushed a commit to IanButterworth/julia that referenced this pull request Oct 27, 2024
…liaLang#55622)

Fixes an issue with JuliaLang#55500, where signals may abruptly abort the process
as they observe it is still processing the resume SIGUSR2 message and
are not able to wait for that processing to end before setting the new
message to exit.

(cherry picked from commit da3468c)
@IanButterworth IanButterworth added backport 1.11 Change should be backported to release-1.11 and removed backport 1.11 Change should be backported to release-1.11 labels Oct 28, 2024
@IanButterworth IanButterworth mentioned this pull request Oct 28, 2024
43 tasks
KristofferC added a commit that referenced this pull request Nov 21, 2024
Backported PRs:
- [x] #55886 <!-- irrationals: restrict assume effects annotations to
known types -->
- [x] #55867 <!-- update `hash` doc string: `widen` not required any
more -->
- [x] #56084 <!-- slightly improve inference in precompilation code -->
- [x] #56088 <!-- make `Base.ANSIIterator` have a concrete field -->
- [x] #54093 <!-- Fix `JULIA_CPU_TARGET` being propagated to workers
precompiling stdlib pkgimages -->
- [x] #56165 <!-- Fix markdown list in installation.md -->
- [x] #56148 <!-- Make loading work when stdlib deps are missing in the
manifest -->
- [x] #56174 <!-- Fix implicit `convert(String, ...)` in several places
-->
- [x] #56159 <!-- Add invalidation barriers for `displaysize` and
`implicit_typeinfo` -->
- [x] #56089 <!-- Call `MulAddMul` instead of multiplication in
_generic_matmatmul! -->
- [x] #56195 <!-- Include default user depot when JULIA_DEPOT_PATH has
leading empty entry -->
- [x] #56215 <!-- [REPL] fix lock ordering mistake in load_pkg -->
- [x] #56251 <!-- REPL: run repl hint generation for modeswitch chars
when not switching -->
- [x] #56092 <!-- stream: fix reading LibuvStream into array -->
- [x] #55870 <!-- fix infinite recursion in `promote_type` for
`Irrational` -->
- [x] #56227 <!-- Do not call `rand` during sysimage precompilation -->
- [x] #55741 <!-- Change annotations to use a NamedTuple -->
- [x] #56149 <!-- Specialize adding/subtracting mixed
Upper/LowerTriangular -->
- [x] #56214 <!-- fix precompile process flags -->
- [x] #54471
- [x] #55622
- [x] #55704
- [x] #55764
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
regression 1.12 Regression in the 1.12 release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants