X-Recipient:	archive-cygwin AT delorie DOT com
DomainKey-Signature:	a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:date:from:to:subject:message-id:in-reply-to
	:references:mime-version:content-type; q=dns; s=default; b=V3N0x
	REV0FZ2RjjxkhDVdVcr4pcHrfZ74i9rSFo27ju57UTHECXZCSZp4uYBKypZwQGpy
	MfpMouya565F0z7W1s7KO/MZ/zkV7T0HEgGT4LBIyTjNatXbpTgck5XbWxUL/PiP
	0xFP6J3K8/vQdvZc0e1zeyAQFIKECJXPmNril0=
DKIM-Signature:	v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:date:from:to:subject:message-id:in-reply-to
	:references:mime-version:content-type; s=default; bh=/XtLgArxYl7
	GlK4nP89Q43K9Fwk=; b=YuHLwbNBk8X0UQfvEYzFsQ9qVuIM7GSIyaEfiYCVZoE
	2mM+zVQSpcNaALUyuq5DLszxLSOPBxZ5XwhpfCV9bWX+UH4mzTQivrbch6hveCFh
	NqtCt7iK/e2hoK3nHlr8yXaw7q8iXldZSQkdaflLxRJKucNPVm3Iv8kS7o9t/PlU
	=
Mailing-List:	contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id:	<cygwin.cygwin.com>
List-Subscribe:	<mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive:	<http://sourceware.org/ml/cygwin/>
List-Post:	<mailto:cygwin AT cygwin DOT com>
List-Help:	<mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender:	cygwin-owner AT cygwin DOT com
Mail-Followup-To:	cygwin AT cygwin DOT com
Delivered-To:	mailing list cygwin AT cygwin DOT com
Authentication-Results:	sourceware.org; auth=none
X-Spam-SWARE-Status:	No, score=-18.8 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=falls, tab, 100000, screen
X-HELO:	conssluserg-06.nifty.com
DKIM-Filter:	OpenDKIM Filter v2.10.3 conssluserg-06.nifty.com x7CDhrWw025815
DKIM-Signature:	v=1; a=rsa-sha256; c=relaxed/relaxed; d=nifty.ne.jp; s=dec2015msa; t=1565617434; bh=Er4c/90CuwQur1lk9vsTkxwpdG00gWKK5JOKi1lWYP0=; h=Date:From:To:Subject:In-Reply-To:References:From; b=VzmD7rXXcQw4IJKcpiE6HC7tT84bs8x77j+yNbvrw8/TShm7wZkKf6ytN+Flk0/oY B5/zi2L06x7s7ZArdKXN5uECpa9H6RvvzjYcvaYHTcKLOH6qGXH8c+9Os3xQtRPVYJ sLhHCHFRu1XzXhfXcJ+yAumP3+A2Y7hEX0sZ/DcJ68fNsJ/QevDUPERhnn9RhfICza KNIF6HrG9544FPFAQdpLstixyQUTf87FfkMoA20mb1etXDeG8+kD4LLNL31uPZxg5v M0SkR7TOnjacuOm89HKSMka2Go+Ei5b7+AdXWpbt9tHB49oVZZZFT+Df1UHAef9pkI aUMLm8z25+PhQ==
Date:	Mon, 12 Aug 2019 22:44:03 +0900
From:	Takashi Yano <takashi DOT yano AT nifty DOT ne DOT jp>
To:	cygwin AT cygwin DOT com
Subject:	Re: [ANNOUNCEMENT] TEST: Cygwin 3.1.0-0.1
Message-Id:	<20190812224403.37e4ccfdcfb7046ed70fef16@nifty.ne.jp>
In-Reply-To:	<announce.20190809185338.GZ11632@calimero.vinschen.de>
References:	<announce DOT 20190809185338 DOT GZ11632 AT calimero DOT vinschen DOT de>
Mime-Version:	1.0
X-IsSubscribed:	yes

--Multipart=_Mon__12_Aug_2019_22_44_03_+0900_R+4i2CLduQOKXFZV
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

Hi Corinna,

On Fri, 9 Aug 2019 20:53:38 +0200
Corinna Vinschen <corinna-cygwin AT cygwin DOT com> wrote:

> Hi folks,
> 
> 
> I uploaded a new Cygwin test release 3.1.0-0.1
> 
> This release comes with a couple of new features and quite a few
> bug fixes.
> 
> The most interesting change, courtesy Ken Brown, is a revamp of the
> old FIFO code.  It should now be possible to open FIFOs multiple times
> for writing, something the old code failed on.
> 
> 
> Please test.

I tested this test release, and found a few problems.

(1) The cursor position is not restored correctly after screen
    alternation in the case of xterm compatible mode is enabled.
(2) fork() on console ocasionally falls into deadlock.
(3) Horizontal tab setting on console is broken after resizing
    console window if xterm compatible mode is enabled.
(4) Segmentation fault occurs in some cases regarding signalfd.

I will propose patches for (1),(2) and (3) in the other posts.

However, I can not find out the cause of problem (4). This seems
to affect only 32bit version of cygwin.

To reproduce (4), use a simple test case attached (signalfd-chk.c).
Compile it and execute, then type 'q' or '^C' to stop it.
This causes segmentation fault.

I am not sure why, but, the patch attached (signalfd-segfault.diff)
resolves the problem (4).

Could you please have a look?

--Multipart=_Mon__12_Aug_2019_22_44_03_+0900_R+4i2CLduQOKXFZV
Content-Type: text/x-csrc;
 name="signalfd-chk.c"
Content-Disposition: attachment;
 filename="signalfd-chk.c"
Content-Transfer-Encoding: 7bit

#include <stdio.h>
#include <unistd.h>
#include <poll.h>
#include <termios.h>
#include <sys/signalfd.h>
#include <sys/wait.h>

int run()
{
	struct termios tt;
	sigset_t sigset;
	pid_t pid;
	struct pollfd pfd[2];
	int running = 1;

	tcgetattr(STDIN_FILENO, &tt);
	tt.c_lflag &= ~(ECHO|ICANON|ISIG);
	tcsetattr(STDIN_FILENO, TCSANOW, &tt);

	pid = fork();
	if (pid == 0) {
		/* Child */
		usleep(100000);
		return 0;
	}
	printf("Start: %d\n", pid);

	sigemptyset(&sigset);
	sigaddset(&sigset, SIGCHLD);
	sigprocmask(SIG_BLOCK, &sigset, NULL);

	pfd[0].fd = STDIN_FILENO;
	pfd[0].events = POLLIN;
	pfd[1].fd = signalfd(-1, &sigset, 0);
	pfd[1].events = POLLIN;

	do {
		poll(pfd, 2, -1);
		if (pfd[0].revents & POLLIN) {
			char c;
			int ret = read(pfd[0].fd, &c, 1);
			if (ret == 0 || c == 'q' || c == '\003') {
				running = 0;
				break;
			}
		}
		if (pfd[1].revents & POLLIN) {
			struct signalfd_siginfo info;
			int ret = read(pfd[1].fd, &info, sizeof(info));
			if (ret) printf("Signal: %d\r\n", info.ssi_signo);
			if (info.ssi_signo == SIGCHLD) break;
		}
	} while (1);
	waitpid(pid, NULL, 0);
	printf("End: %d\n", pid);

	tcgetattr(STDIN_FILENO, &tt);
	tt.c_lflag |= ECHO|ICANON|ISIG;
	tcsetattr(STDIN_FILENO, TCSANOW, &tt);

	return running;
}

int main()
{
	while (run());
	return 0;
}

--Multipart=_Mon__12_Aug_2019_22_44_03_+0900_R+4i2CLduQOKXFZV
Content-Type: text/plain;
 name="signalfd-segfault.diff"
Content-Disposition: attachment;
 filename="signalfd-segfault.diff"
Content-Transfer-Encoding: 7bit

diff --git a/winsup/cygwin/select.cc b/winsup/cygwin/select.cc
index 1bb143a23..91c0bcc53 100644
--- a/winsup/cygwin/select.cc
+++ b/winsup/cygwin/select.cc
@@ -1887,7 +1887,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:


--Multipart=_Mon__12_Aug_2019_22_44_03_+0900_R+4i2CLduQOKXFZV
Content-Type: text/plain; charset=us-ascii


--
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
--Multipart=_Mon__12_Aug_2019_22_44_03_+0900_R+4i2CLduQOKXFZV--

- Raw text -