X-Recipient:	archive-cygwin AT delorie DOT com
X-Original-To:	cygwin AT cygwin DOT com
Delivered-To:	cygwin AT cygwin DOT com
DMARC-Filter:	OpenDMARC Filter v1.4.1 sourceware.org BE3233858D28
Authentication-Results:	sourceware.org;
	dmarc=pass (p=none dis=none) header.from=ispras.ru
Authentication-Results:	sourceware.org; spf=pass smtp.mailfrom=ispras.ru
MIME-Version:	1.0
Date:	Mon, 18 Apr 2022 18:07:16 +0300
From:	Alexey Izbyshev <izbyshev AT ispras DOT ru>
To:	Takashi Yano <takashi DOT yano AT nifty DOT ne DOT jp>
Subject:	Re: Deadlock when pressing Ctrl-C at startup of a native console
	application
In-Reply-To:	<20220418214823.a01ed743ceb15d3708ee3fd9@nifty.ne.jp>
References:	<355a1f23bf39b0670110a840075224a2 AT ispras DOT ru>
	<20220416181650 DOT cdb31ec9c75189724bf2f498 AT nifty DOT ne DOT jp>
	<20220418214823 DOT a01ed743ceb15d3708ee3fd9 AT nifty DOT ne DOT jp>
User-Agent:	Roundcube Webmail/1.4.4
Message-ID:	<df1dabbfc4ceb32e8e51449b6fec2afc@ispras.ru>
X-Sender:	izbyshev AT ispras DOT ru
X-Spam-Status:	No, score=-0.6 required=5.0 tests=BAYES_00, DOS_RCVD_IP_TWICE_B,
	KAM_DMARC_STATUS, SPF_HELO_NONE, SPF_PASS, TXREP,
	T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.4
X-Spam-Checker-Version:	SpamAssassin 3.4.4 (2020-01-24) on
	server2.sourceware.org
X-BeenThere:	cygwin AT cygwin DOT com
X-Mailman-Version:	2.1.29
List-Id:	General Cygwin discussions and problem reports <cygwin.cygwin.com>
List-Unsubscribe:	<https://cygwin.com/mailman/options/cygwin>,
	<mailto:cygwin-request AT cygwin DOT com?subject=unsubscribe>
List-Archive:	<https://cygwin.com/pipermail/cygwin/>
List-Post:	<mailto:cygwin AT cygwin DOT com>
List-Help:	<mailto:cygwin-request AT cygwin DOT com?subject=help>
List-Subscribe:	<https://cygwin.com/mailman/listinfo/cygwin>,
	<mailto:cygwin-request AT cygwin DOT com?subject=subscribe>
Cc:	cygwin AT cygwin DOT com
Errors-To:	cygwin-bounces+archive-cygwin=delorie DOT com AT cygwin DOT com
Sender:	"Cygwin" <cygwin-bounces+archive-cygwin=delorie DOT com AT cygwin DOT com>

On 2022-04-18 15:48, Takashi Yano wrote:
> On Sat, 16 Apr 2022 18:16:50 +0900
> Takashi Yano wrote:
>> On Fri, 15 Apr 2022 21:09:05 +0300
>> Alexey Izbyshev wrote:
>> > Hi,
>> >
>> > I've discovered that pressing Ctrl-C in a Cygwin ssh session running the
>> > following command can cause a deadlock:
>> >
>> > $ while python -c ''; do :; done
>> >
>> > The exact native application doesn't matter here, e.g. "ping -n 1
>> > localhost >/dev/null" instead of "python -c ''" also "works".
>> >
>> > The deadlock is most easily triggered when CPU is fully loaded. It takes
>> > no more than 20 tries on my machine to hit it (and when I first saw it,
>> > it was on the first try).
>> >
>> > When the deadlock is triggered, there are conhost.exe and
>> > cygwin-console-helper.exe processes, but there is no python.
>> >
>> > The exact process tree varies depending on Cygwin version. With the
>> > vanilla 3.3.4 (the root is bash spawned by sshd):
>> >
>> > bash---bash-+-conhost.exe
>> >              `-cygwin-console-helper.exe
>> >
>> > With 20220301 snapshot
>> > (https://cygwin.com/snapshots/x86/cygwin1-20220301.dll.xz), I get one of
>> > the two following trees with an extra conhost.exe process:
>> >
>> > bash---bash-+-conhost.exe
>> >              |-conhost.exe
>> >              `-cygwin-console-helper.exe
>> >
>> > or
>> >
>> > bash---bash-+-conhost.exe
>> >       |      `-cygwin-console-helper.exe
>> >       `-conhost.exe
>> >
>> > I was testing with 32-bit Cygwin on x64 Windows 10 21H2.
>> 
>> Thanks for the report. I could reproduce the issue.
>> I also found the current git master does not have this issue,
>> while cygwin-3_3-branch does.
>> 
>> Now I am identifying which patch solved the issue.
>> One is obviously:
>> https://cygwin.com/pipermail/cygwin-patches/2022q2/011870.html
>> however, this was applied also for cygwin-3_3-branch. Therefore,
>> another patch also should affect this issue.
> 
> I identified the cause and applied a workaround patch.
> The countermeasure version is available at:
> https://tyan0.yr32.net/cygwin/x86/test/cygwin1-20220418.dll.xz
> https://tyan0.yr32.net/cygwin/x86_64/test/cygwin1-20220418.dll.xz
> 
> Could you please test?

I've tested the 32-bit version, and Ctrl-C in my original test works as 
expected (no deadlock or other issues after multiple tries). Thank you 
for the fix!

Alexey

-- 
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

- Raw text -