From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 113350 invoked by alias); 14 Aug 2019 19:21:26 -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 113301 invoked by uid 89); 14 Aug 2019 19:21:26 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-18.0 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_0,GIT_PATCH_1,GIT_PATCH_2,GIT_PATCH_3,KAM_NUMSUBJECT,RCVD_IN_DNSWL_NONE autolearn=ham version=3.3.1 spammy=Cause, takashi.yano@nifty.ne.jp, sk:takashi, D*ne.jp X-HELO: conssluserg-05.nifty.com Received: from conssluserg-05.nifty.com (HELO conssluserg-05.nifty.com) (210.131.2.90) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 14 Aug 2019 19:21:24 +0000 Received: from Express5800-S70 (ntsitm268057.sitm.nt.ngn.ppp.infoweb.ne.jp [125.1.110.57]) (authenticated) by conssluserg-05.nifty.com with ESMTP id x7EJLFfI029770 for ; Thu, 15 Aug 2019 04:21:16 +0900 DKIM-Filter: OpenDKIM Filter v2.10.3 conssluserg-05.nifty.com x7EJLFfI029770 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nifty.ne.jp; s=dec2015msa; t=1565810476; bh=0aW3csaeBWiEM+RA3HTb8Uu2vHSTwWN8jLZw1EPCFHk=; h=Date:From:To:Subject:In-Reply-To:References:From; b=XviYEsstRr51nYbbbOjPsj4jF+bv3/EgOVTd2sCYE8i4XAv5krjlU6h6N6MNu8Rph NIMsVoGJaY7MCeyFo57951WYgEhkkTgIKl+tjl63D/x/Gv0cj73vUL4oOPtcSXR7D0 Bv6y2ooufK7BeGBzxvmRTzlWL7Jt28ID/jSlAc30bliNTYozLrSwc2CkU9HAgzsHBd JDdghePrPnMC/14NASUOTvZArTzgGWl5kth2qTprOq6ShX7BYoHcDwgvYFqMawgFsk QEj0l1MmsX9QfG2Jyg4cIhT+R1NKGgX+REOlQ84rf0D/VaVrmeD5dRvgDQV5/zTZVc Ja5PDleDhD03w== Date: Wed, 14 Aug 2019 19:21:00 -0000 From: Takashi Yano To: cygwin@cygwin.com Subject: Re: [ANNOUNCEMENT] TEST: Cygwin 3.1.0-0.1 Message-Id: <20190815042126.7c2f0baf57b4a82f7d013f74@nifty.ne.jp> In-Reply-To: <20190814134900.GY11632@calimero.vinschen.de> References: <20190812224403.37e4ccfdcfb7046ed70fef16@nifty.ne.jp> <20190812153613.GN11632@calimero.vinschen.de> <20190813104753.GU11632@calimero.vinschen.de> <20190814204100.659fe40d928eae15338198a7@nifty.ne.jp> <20190814204717.caf6884b1216bbeee2f586d6@nifty.ne.jp> <20190814134900.GY11632@calimero.vinschen.de> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-IsSubscribed: yes X-SW-Source: 2019-08/txt/msg00181.txt.bz2 On Wed, 14 Aug 2019 15:49:00 +0200 Corinna Vinschen wrote: > The only reason I can see is if sigwait_common() returns EINTR because > it was interrupted by an unrelated signal. This in turn lets the read() > call fail with EINTR and that should be expected by the callers, in > theory. Strangely, this problem also disappears with this patch. diff --git a/winsup/cygwin/select.cc b/winsup/cygwin/select.cc index 9cf892801..82ac0674f 100644 --- a/winsup/cygwin/select.cc +++ b/winsup/cygwin/select.cc @@ -1869,7 +1869,7 @@ thread_signalfd (void *arg) switch (WaitForSingleObject (si->evt, INFINITE)) { case WAIT_OBJECT_0: - tls->signalfd_select_wait = NULL; + //tls->signalfd_select_wait = NULL; event = true; break; default: Moreover, this EINTR is not set by sigwait_common() but set by select_stuff::wait(). With following debug print, the message shows: 1 [main] script 1082 select_stuff::wait: signal = 20 script: poll failed: Interrupted system call Script done, file is typescript however, the signal 20 (SIGCHLD) is registered to signalfd by script and should be caught. diff --git a/winsup/cygwin/select.cc b/winsup/cygwin/select.cc index 9cf892801..adcc75ad3 100644 --- a/winsup/cygwin/select.cc +++ b/winsup/cygwin/select.cc @@ -446,6 +446,7 @@ next_while:; unconditionally, ignoring any SA_RESTART detection by call_signal_handler(). */ _my_tls.call_signal_handler (); + system_printf ("signal = %d\n", _my_tls.infodata.si_signo); set_sig_errno (EINTR); res = select_signalled; /* Cause loop exit in cygwin_select */ break; -- Takashi Yano -- Problem reports: http://cygwin.com/problems.html FAQ: http://cygwin.com/faq/ Documentation: http://cygwin.com/docs.html Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple