On Thu, 5 Dec 2024 15:28:12 +0100 Corinna Vinschen <corinna-cyg...@cygwin.com> wrote:
> On Dec 5 21:25, Takashi Yano wrote: > > Takashi Yano (3): > > Cygwin: signal: Remove queue entry from the queue chain when cleared > > Cygwin: signal: Introduce a lock for the signal queue > > Cygwin: Document several fixes for signal handling in release note > > > > winsup/cygwin/exceptions.cc | 12 ++--- > > winsup/cygwin/local_includes/sigproc.h | 5 +- > > winsup/cygwin/release/3.5.5 | 4 ++ > > winsup/cygwin/signal.cc | 4 +- > > winsup/cygwin/sigproc.cc | 72 +++++++++++++++++++------- > > 5 files changed, 68 insertions(+), 29 deletions(-) > > > > -- > > 2.45.1 > > LGTM. Thanks! Pushd. IIRC, the patches that should be cherry-picked for cygwin-3_5-branch are: d243e51ef1d3 Cygwin: signal: Fix deadlock between main thread and sig thread e10f822a2b39 Cygwin: signal: Handle queued signal without explicit __SIGFLUSH 57ce5f1e0bf4 Cygwin: signal: Drop unnecessary queue flush 9ae51bcc51a7 Cygwin: signal: Fix another deadlock between main and sig thread 9b7a84d24aa1 Cygwin: cygtls: Prompt system to switch tasks explicitly in lock() 2544e753963e Cygwin: signal: Fix a short period of deadlock 9a274a967d1f Cygwin: signal: Optimize the priority of the sig thread c48d58d838d9 Cygwin: signal: Increase chance of handling signal in main thread d565aca46f06 Cygwin: signal: Remove queue entry from the queue chain when cleared 496fa7b2ce00 Cygwin: signal: Introduce a lock for the signal queue cfadd852aa57 Cygwin: Document several fixes for signal handling in release note as well as your patches regarding longjmp. Renaming sig to current_sig is not a bug fix, bug without this patch, we should modify patches manually. -- Takashi Yano <takashi.y...@nifty.ne.jp>
