DMARC-Filter: OpenDMARC Filter v1.4.2 delorie.com 52CGOfns3875712 Authentication-Results: delorie.com; dmarc=pass (p=none dis=none) header.from=cygwin.com Authentication-Results: delorie.com; spf=pass smtp.mailfrom=cygwin.com DKIM-Filter: OpenDKIM Filter v2.11.0 delorie.com 52CGOfns3875712 Authentication-Results: delorie.com; dkim=pass (1024-bit key, unprotected) header.d=cygwin.com header.i=@cygwin.com header.a=rsa-sha256 header.s=default header.b=dWYvaIht X-Recipient: archive-cygwin AT delorie DOT com DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org E27CC3858C48 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cygwin.com; s=default; t=1741796679; bh=B7cUOeuLn/CFOLiUIxVate4CrzFsv9JeJ/M63GMErxQ=; h=Date:To:Subject:References:In-Reply-To:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc: From; b=dWYvaIhtWCiKLcw/7KD+G6LtHhcxpp3pOXRb+ILdjfQUteZjFV7Lm68guQq2tZwLn FIyjJXbvwDG0Spmh5BlPhcgSdJvuScRLYcluQD/0jMtvRRV9KSKVaVMi1xjYc9wlHD wvaQYA1FbfBkQ27S/BIVtzbXAR8tcodW6k8Uw+/Y= X-Original-To: cygwin AT cygwin DOT com Delivered-To: cygwin AT cygwin DOT com DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org A3E483858D29 Date: Wed, 12 Mar 2025 17:23:32 +0100 To: cygwin AT cygwin DOT com Subject: Re: cygwin 3.6.0: No signals received after swapcontext() is used Message-ID: Mail-Followup-To: cygwin AT cygwin DOT com References: <373993a3-9f0f-9750-60a0-950f83b3b0b5 AT t-online DOT de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-BeenThere: cygwin AT cygwin DOT com X-Mailman-Version: 2.1.30 Precedence: list List-Id: General Cygwin discussions and problem reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Corinna Vinschen via Cygwin Reply-To: cygwin AT cygwin DOT com Cc: Corinna Vinschen Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: cygwin-bounces~archive-cygwin=delorie DOT com AT cygwin DOT com Sender: "Cygwin" On Mar 12 17:06, Corinna Vinschen via Cygwin wrote: > On Mar 12 16:30, Corinna Vinschen via Cygwin wrote: > > On Mar 11 12:32, Christian Franke via Cygwin wrote: > > > The attached testcase should test the following use cases of setcontext: > > > - call from regular user space > > > - call from a signal handler interrupting user space > > > - call from a signal handler interrupting a system call > > > > > > It works as expected ... until the signal count reaches 256. Then signals > > > are again only delivered from inside of a system call. > > > [...] > > > Interesting... Hmm... is there some 8-bit counter which overflows and then > > > stucks at 0xff or 0x00? > > > > It's a kind of stack overflow. Kind of, because it's not the normal > > thread stack, but a special signal stack in the _cygtls area. > > > > When interrupting a running thread to call a signal handler, the context > > of the thread is changed to restart execution in an assembler function > > called sigdelayed(). The original IP of the thread is pushed on the > > aforementioned signal stack. Sigdelayed() calls the signal handler. On > > return it pops the original IP from the signal stack and continues the > > thread. > > > > Now guess what happens if the signal handler bails out with longjmp or > > setcontext/swapcontext. > > > > The signal handler never returns to the sigdelayed() function, the > > original address is never poped from the signal stack, and the signal > > stack has a max. size of 256 address entries... > > > > Theoretically, a small update to sigdelayed() would fix the issue: ather > > then poing the original IP from the signal stack after calling the > > handler, it should pop the IP prior to calling the handler. That would > > avoid filling up the signal stack when long-jumping out of the signal > > handler. It should store the IP in one of the callee-saved registers. > > %r13 is unused in sigdelayed so far. > > > > However, even if we do this, there's still the problem that sigdelayed() > > itself takes space on the stack. If you longjmp/setcontext out of the > > handler, the thread's normal stack will fill up with dead storage of the > > sigdelayed() function, and there's no way out of this trap. We can't > > restore the stack before the handler returns. > > > > So either way, at one point you get a stack overflow one way or the > > other. > > > > The signal stack overflow is actually rather harmless in comparison > > to a real stack overflow. > > > > If you have any idea how to avoid the real stack overflow, I'd be > > all ears. > > Looks like this isn't really a problem with setcontext. It always > corrects the stack pointer as well. Apparently I haven't thought > long enough about this. > > I have a patch for sigdelayed() in the loop, stay tuned. Just pushed. Try cygwin-3.6.0-0.430.ga942476236b5 in a bit. Thanks, Corinna -- Problem reports: https://cygwin.com/problems.html FAQ: https://cygwin.com/faq/ Documentation: https://cygwin.com/docs.html Unsubscribe info: https://cygwin.com/ml/#unsubscribe-simple