From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 51617 invoked by alias); 3 Nov 2015 09:00:03 -0000 Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: cygwin-owner@cygwin.com Mail-Followup-To: cygwin@cygwin.com Received: (qmail 51547 invoked by uid 89); 3 Nov 2015 09:00:02 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-5.4 required=5.0 tests=AWL,BAYES_00,KAM_LAZY_DOMAIN_SECURITY autolearn=no version=3.3.2 X-HELO: calimero.vinschen.de Received: from aquarius.hirmke.de (HELO calimero.vinschen.de) (217.91.18.234) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 03 Nov 2015 09:00:01 +0000 Received: by calimero.vinschen.de (Postfix, from userid 500) id CA37FA803EE; Tue, 3 Nov 2015 09:59:58 +0100 (CET) Date: Tue, 03 Nov 2015 09:00:00 -0000 From: Corinna Vinschen To: cygwin@cygwin.com Subject: Re: pthread_kill: signals remain pending after target thread exits Message-ID: <20151103085958.GA18567@calimero.vinschen.de> Reply-To: cygwin@cygwin.com Mail-Followup-To: cygwin@cygwin.com References: <20151021114810.GQ5319@calimero.vinschen.de> <28F5B565B6F6424C87E4AC0DCC84316575D73A86@S1P5DAG5C.EXCHPROD.USA.NET> <20151023125540.GI5319@calimero.vinschen.de> <28F5B565B6F6424C87E4AC0DCC84316575D8071E@S1P5DAG5C.EXCHPROD.USA.NET> <20151028091459.GZ5319@calimero.vinschen.de> <20151029080218.GF5319@calimero.vinschen.de> <28F5B565B6F6424C87E4AC0DCC84316575D81A49@S1P5DAG5C.EXCHPROD.USA.NET> <20151102141838.GB963@calimero.vinschen.de> <20151102175800.GA13010@calimero.vinschen.de> <28F5B565B6F6424C87E4AC0DCC84316575D81F11@S1P5DAG5C.EXCHPROD.USA.NET> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="gKMricLos+KVdGMg" Content-Disposition: inline In-Reply-To: <28F5B565B6F6424C87E4AC0DCC84316575D81F11@S1P5DAG5C.EXCHPROD.USA.NET> User-Agent: Mutt/1.5.23 (2014-03-12) X-SW-Source: 2015-11/txt/msg00073.txt.bz2 --gKMricLos+KVdGMg Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Content-length: 2488 On Nov 2 23:54, John Carey wrote: > > From: Corinna Vinschen [corinna-cygwin@cygwin.com] > > Sent: Monday, November 02, 2015 9:58 AM > > On Nov 2 15:18, Corinna Vinschen wrote: > > > On Oct 29 16:16, John Carey wrote: > > > > > From: Corinna Vinschen [corinna-cygwin@cygwin.com] > > > > > Sent: Thursday, October 29, 2015 1:02 AM > > > > > On Oct 28 10:14, Corinna Vinschen wrote: > > > > ... > > > > > > > Thanks; that was fast! I tried replacing cygwin1.dll with cy= gwin1-20151023.dll . > > > > > > > > > > > > > > The original test case now works. I checked some of my other= tests, > > > > > > > and unfortunately some of them failed, so I extracted out a n= ew test > > > > > > > case, which is attached. My guess is that something is subtl= y different > > > > > > > about the timing on this test. > > > > > > > > > > > > Is this a regression? Did it work with 2.2.1? > > > > > > > > > > Answering myself here, this didn't work at all in 2.2.1. I can > > > > > reproduce the problem and I'm going to take a look. Not sure if > > > > > there's a quick solution, though. This looks like a deadlock > > > > > situation. The signal definitely got send, I just don't see yet > > > > > why it's not handled in wait_sig. > > > > > > > > Yes, test_pending_signal2.c fails in 2.2.1, though the symptoms > > > > differ: there is no Cygwin process left hanging that cannot be > > > > killed by a Cygwin signal. > > > > > > > > But both tests appear to pass in Cygwin 1.7.9, so in that sense > > > > it is a regression, just not a recent one. > > > > > > The regression here was my original fix. It didn't clear pending > > > signals for an exiting thread thorougly enough. I applied a patch now > > > which should fix this issue. I'll upload a new developer snapshot to > > > https://cygwin.com/snapshots/ later today or tomorrow (still looking > > > into the PrlFS issue). Please give it a try. > >=20 > > Snapshot is up. The fix is in the 2.3.0-0.5 test release as well. > > Please give any one of them a try. >=20 > I tested snapshot cygwin1-20151102.dll, and it works in all of my tests. = Nice! One good news, thanks! > I'm glad you figured out how to delete those pending signals. Given that I introduced the regression in the first place, it's a bit embarrassing that it took me so long to understand it 8-} Corinna --=20 Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Maintainer cygwin AT cygwin DOT com Red Hat --gKMricLos+KVdGMg Content-Type: application/pgp-signature Content-length: 819 -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBCAAGBQJWOHeOAAoJEPU2Bp2uRE+gLPMP+wf0uHZ3pRv7HNG2Gy+J1eSE 9+2nkHwoPcJ2SDnUtRDZTl40HeVyaWusfmkGI12Kqkp3NiXvPz34Qa8kK25GrnSH 7FrYAkePfWRx8A/hEiMdWlSu930nqIqiyI/AlBzXb5ZJDeW6J9iHon/M18Y6Ehqd 7p1dNvVbQ1itAzFQO+qr+BdVlqX3H5+qI0E0LVEglwc5a8FaUiY577rwkZw90s6m CE7w296Elw34N1CC4nxqGie+6dhtmR63IhLoR3SKUIsjkKLYnq/SjbeLA+iJi8k8 89InNxo+dGBLrmW/uOEssIIMH4/VIKNxnf6UpZO48UNoLfKnJZKaNg6yomaVudOM Brzi2CycRQtIaZ60XwzId/1pCIbPchEWeKYxdCdywXhgsRt6bKp3CwCztrbQ5mOQ XhbLdsRe38klapRdLowWxeWemdXIqgOIjH3kA1WZysiJYQhtIhR9Tktph5JHnOu9 w8+0QEwbcxqH99j9Q83GVWNq2DcfYDUkqt51r2vClMQgKYR2g1Wvzeg0KYwHJZvk MWbKJ/XPrHkrtPBlfDHLhcI2Fj7UPxIkc0reUsuUl/zeoVjInEByyxjVgj2PbNk6 NYb5n4X8pUfVcRg4ioybM8uo7QiIWYpfj1NDHYSXyozF96kYaE5izoMhMKsK9RAL Nt7/2mCHuC7uZNodYbIr =NEYX -----END PGP SIGNATURE----- --gKMricLos+KVdGMg--