From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 63783 invoked by alias); 2 Nov 2015 17:58:05 -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 63775 invoked by uid 89); 2 Nov 2015 17:58:04 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-4.1 required=5.0 tests=AWL,BAYES_50,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; Mon, 02 Nov 2015 17:58:03 +0000 Received: by calimero.vinschen.de (Postfix, from userid 500) id B760FA80617; Mon, 2 Nov 2015 18:58:00 +0100 (CET) Date: Mon, 02 Nov 2015 17:58:00 -0000 From: Corinna Vinschen To: John Carey Cc: cygwin@cygwin.com Subject: Re: pthread_kill: signals remain pending after target thread exits Message-ID: <20151102175800.GA13010@calimero.vinschen.de> Reply-To: cygwin@cygwin.com Mail-Followup-To: John Carey , cygwin@cygwin.com References: <28F5B565B6F6424C87E4AC0DCC84316575D71070@S1P5DAG5C.EXCHPROD.USA.NET> <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> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="3V7upXqbjpZ4EhLz" Content-Disposition: inline In-Reply-To: <20151102141838.GB963@calimero.vinschen.de> User-Agent: Mutt/1.5.23 (2014-03-12) X-SW-Source: 2015-11/txt/msg00043.txt.bz2 --3V7upXqbjpZ4EhLz Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Content-length: 1921 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 cygwin= 1-20151023.dll . > > > > > > > > > > The original test case now works. I checked some of my other tes= ts, > > > > > and unfortunately some of them failed, so I extracted out a new t= est > > > > > case, which is attached. My guess is that something is subtly di= fferent > > > > > about the timing on this test. > > > > > > > > Is this a regression? Did it work with 2.2.1? > > >=20 > > > 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. > >=20 > > 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. > >=20 > > But both tests appear to pass in Cygwin 1.7.9, so in that sense > > it is a regression, just not a recent one. >=20 > 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. 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. Thanks, Corinna --=20 Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Maintainer cygwin AT cygwin DOT com Red Hat --3V7upXqbjpZ4EhLz Content-Type: application/pgp-signature Content-length: 819 -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBCAAGBQJWN6QoAAoJEPU2Bp2uRE+gZbgP/3YA0vCQ2GbGIKM6oYcqMTbm kJea92h1QCQctVxOF6+JbjKkunLQsDHsGK/RXwg0jK3pUBg4qOYdLJlf34H8HHz8 8jTmjQwF/Wd1LyLBGz5Yl2iOzqXhlVFppn8qgOeh3mMYRNlKb8aSjwZ9TY1ugFUU PcGnrgBtsUfkqSVY8w5kBC7Nj/f0G6KNICerLygPjIFA7eT8d3RZFs0H+tooweay nIfwMTwHX9GoaZcflU9ss9gnDAPO9u1Em+dAdJ/nsQJWNDA5bRtlaMVZMJpQjaGr RB9Q/g73bFm3pLNb5Kez4cPpAYHua4kLsC66Fp7S7Ow5OoGL74xatro0WoZUWgcm tew0odbII9zyd/BdmxxhZ8oh9Gtqpgc4thjHubVAHu7wZ+rMczrx74Rz6B9tEO5Q +XSA1NxrK6A7KnW/duj5Gz2bE4sW+MSSRW5rh5rifdKcpFnQts1vrhVM/wDJgZzx d2eLOiec+DbWSQj5Ch06iyiX5Lbu6emVaPPmJOvMXVk45pIKuoDOsR/pDQxC4Blb bILSLOF1a8+mq5Zl3ZUSd/2dZcEkP+O05GDzhvU98ooQcjVXpIwJPm4XDaPhIfWX +cOptlPRfpSSpIKB8u1zIiCFyoNnDt8hgQFUTzf60XN70y8cQjuYKb7bFtP3Y3gz NUJDUQAJZw+t0kbMpjJw =E8xA -----END PGP SIGNATURE----- --3V7upXqbjpZ4EhLz--