From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mout.kundenserver.de (mout.kundenserver.de [212.227.126.131]) by sourceware.org (Postfix) with ESMTPS id 51C6F3858403 for ; Wed, 15 Sep 2021 13:43:01 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 51C6F3858403 Authentication-Results: sourceware.org; dmarc=fail (p=none dis=none) header.from=cygwin.com Authentication-Results: sourceware.org; spf=fail smtp.mailfrom=cygwin.com Received: from calimero.vinschen.de ([24.134.7.25]) by mrelayeu.kundenserver.de (mreue009 [212.227.15.167]) with ESMTPSA (Nemesis) id 1MMXYN-1mADUS36Pv-00JYeC for ; Wed, 15 Sep 2021 15:42:59 +0200 Received: by calimero.vinschen.de (Postfix, from userid 500) id 31140A80BAE; Wed, 15 Sep 2021 15:42:59 +0200 (CEST) Date: Wed, 15 Sep 2021 15:42:59 +0200 From: Corinna Vinschen To: cygwin-developers@cygwin.com Subject: Re: cygrunsrv + sshd + rsync = 20 times too slow -- throttled? Message-ID: Reply-To: cygwin-developers@cygwin.com Mail-Followup-To: cygwin-developers@cygwin.com References: <20210915092100.796985ae33039f3184483450@nifty.ne.jp> <20210915094454.42ef1d085d9bf92a1af8d39a@nifty.ne.jp> <20210915095911.415d174ea38fb2d2ea8e5432@nifty.ne.jp> <20210915194858.845bcd54c0c63246b40703a8@nifty.ne.jp> <20210915205433.014102a9c1204687135e1417@nifty.ne.jp> <20210915220450.a96397d8b8159d9adfbdab07@nifty.ne.jp> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20210915220450.a96397d8b8159d9adfbdab07@nifty.ne.jp> X-Provags-ID: V03:K1:hVp5RM7WN5tGNil3DwBeUWeKfDIZUlgr5PRC+ClMjutW9z08hhn /ZYcag80npQpjeAjB265ZDmc/GxC+iHVyOZCT4zafq1VfR2m9cDPMAMOFLetqmjgmUZ3gf1 hzzNZCscANCkohyZTI1e++tUzNlavt6068KbhhCCL4iJogxq9PgCPZl683UJVLQnWG8AGpT OdcDTaySXP1YK8USmjmEw== X-UI-Out-Filterresults: notjunk:1;V03:K0:GHiyofcTZ/w=:yD4mdv7uz0Humab+lD1mfs G40lgsmxPzy+ECohLYuvw4mYMItBFjVTy7lD+xAP30f2bLsP6ObGa6QO93QaLrWzDouZFS9Ii OXkIrekslOBCRxTE1XCbKteSH4z9ujGJbeg2wjLh86qvYujhSJHswW5sLWSy0JssAnGriwFqp L/5WgJ0YenScEOJrwTk7zV1Gk3Jr8ECNuVsDsLS2sR8QmxUxYnEIHyIp4w40qwSF200klXC3f 4BopsXHQHiFqQdcy/woQ/39/xWXHjhugSRz+xPLfOSywtgbhGzHcRy/6gdJ+ikwfihjRAbjuy dYl5MtOMNPfaaFps2/Y+X747HNuUfPRbuYS2oulfoo6LCsYc3glELLIikQ8M0GOmM1LVQTd0t FhTQ6lkTWAtfzyNC/dxjvLPd/CLWBnTdGpAoN+b/ffoO+yEeIEWP6bM2Jairkq+Vrgd62YKgl 6JOYK8Z64skFezFp2qPh2vHBuUGKKpa2zvkCncHZotdmghCVmCp0kGxHQdpf6HSEzftCOCWHT s9B4A0Ej127N7U9NssV/zm+4sn8u5th7SKflHDJTYj1EcHBBHjyGToikEOBgKXCwAp/l0B71j 9KZVpkKGG1O54iAbleUeuaBzLN1y/UgNBvSN1rtJSIUdM/1N4sPEvYX91J+4Kjii5tXs3HmEP GqZkRgMQP40n1cOX+ayuHreM5jrIWp9m4nThzTsIFfhbUGNQuZguyJkmP/WbUvLe+KoI0PqXP ggec0DpimhGhLHtigmsxnMukTun5BMrKnhHYm7EuF7AGfxPhN/MzCQSbt9I= X-Spam-Status: No, score=-99.9 required=5.0 tests=BAYES_00, GOOD_FROM_CORINNA_CYGWIN, KAM_DMARC_NONE, KAM_DMARC_STATUS, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H4, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_NEUTRAL, TXREP autolearn=ham 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-developers@cygwin.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Cygwin core component developers mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 15 Sep 2021 13:43:02 -0000 On Sep 15 22:04, Takashi Yano wrote: > On Wed, 15 Sep 2021 14:20:48 +0200 > Corinna Vinschen wrote: > > After Ken's comment, I was going to suggest this one: > > > > From 34b14470406cb9551f98707bf63175811a506523 Mon Sep 17 00:00:00 2001 > > From: Corinna Vinschen > > Date: Wed, 15 Sep 2021 14:17:59 +0200 > > Subject: [PATCH] Cygwin: pipes: don't call NtQueryInformationFile on read side > > of pipes > > > > NtQueryInformationFile hangs if it's called on the read side handle of > > a pipe while another thread or process is performing a blocking read. > > > > Avoid select potentially hanging by calling NtQueryInformationFile > > only on the write side of the pipe and using PeekNamedPipe otherwise. > > > > --- > > winsup/cygwin/select.cc | 8 ++++++++ > > 1 file changed, 8 insertions(+) > > [...] > > LGTM. > I also confirmed a several test cases are all passed with this patch. Thanks for reviewing. Pushed. Corinna