From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mout.kundenserver.de (mout.kundenserver.de [212.227.17.13]) by sourceware.org (Postfix) with ESMTPS id AE98E3858414 for ; Wed, 1 Sep 2021 08:46:26 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org AE98E3858414 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 (mreue109 [212.227.15.183]) with ESMTPSA (Nemesis) id 1MuDPf-1n8lW13MIn-00uYT6 for ; Wed, 01 Sep 2021 10:46:24 +0200 Received: by calimero.vinschen.de (Postfix, from userid 500) id 3284EA80D60; Wed, 1 Sep 2021 10:46:24 +0200 (CEST) Date: Wed, 1 Sep 2021 10:46:24 +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: <20210831204541.2b56702cdcd2fae5e91ba8e2@nifty.ne.jp> <583ca127-02e7-6b3c-3732-6478c0f862e3@cornell.edu> <20210901080220.ee4a5bfbea62cc1ae0a9598e@nifty.ne.jp> <20210901091652.6bf3cccbcaed4a22f6ffa6b0@nifty.ne.jp> <20210901172339.1039604b7067e0492534a20f@nifty.ne.jp> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20210901172339.1039604b7067e0492534a20f@nifty.ne.jp> X-Provags-ID: V03:K1:c3PlhjNlTXI5lHyvHbRXodFPc6ofT2NyneAbmuZIRLDTgRUDZC6 ZaoRSPrxbcfOr9Ysoym8MY+TgPlJfiMrkUeL/5bimjMfr8c5k7f7/dmBOHWenfENEPZhD4L bhw9iQjFhj2GzMIHsU6OwXwNm4338WbvJlCqkMsP54udG2CjOfBpzMg6pz33ObQtypXS9mq p22BlzQdtoTKmWLg/acRg== X-UI-Out-Filterresults: notjunk:1;V03:K0:qipB0koAJQ4=:w6rNl0/Gn7EuIX3nEqtRY6 BLAaJUF+wzNcwIGaoruwVJbBSlyefI1Rt7Ce8uyfBjxEQTWdSn/MrZTpsiGl2xKf/yrM9JuHe tL086jX9fib/uto03ibeZTJuZmEFm6DhFCd4X9ORWyg8rLFOnHLgAzRXV3AXMZZqlNQFoCPKi hI9mbE9MtPOfiss4Dpx83NwhOW5pXuDj9DmNi5PeOgjDuV9CXyNfSs7+K0cmZrbUUbPuoKnPG wyWe13AjLEsse+2ACj2dDldWktxIgetUO5U+GhPn1gfIDrfTqcJGNCyqT3x8aYJpoAPqgdy0+ o8bsoTHwStajisuUPCj6tGUyNnmUF/WbLBGrMb6bYmQ8XeEpVQf2v15j880Zmbdfu6L1k1hQ2 ZsfKUzko4TDvOw/hn1PmamAd63jYVMEEaIbvHs1axP48YwcrykFpZefW/W48JDy7BNtobLH0J Fbx46IxUuiQsJGdTwxbSP4Z9CPOtTn6gJtI918b76yW8dXlVRIG0eyJ2/9RPKnYZXuxQzi5GL kKDqwjLpBBbMtM0uoyAIZoYCoLkpDiWgg3hSJPBour6PgMSUTKsQzwVU/pqE3p87d4MC8wb3D wM8zenYMtZzkvbipxUcH8P0T68CWPfLkR73FphTiBDMNwUGYrnZNrdZerUpG8kRRRK4vuiJCo DhysJJ4uRTQJiPZ9fFfHTI9bDWBD9fk/I7eFrZfzoaV770fX8Fz58cVTdWVn++sxnCkYGp0u7 dqckw+AA2QrIrGJ7dMYxdTnNEDxm+vjGFeo1TvOKZXxis+jx0w1nBRbSNGU= X-Spam-Status: No, score=-100.1 required=5.0 tests=BAYES_00, GOOD_FROM_CORINNA_CYGWIN, KAM_DMARC_NONE, KAM_DMARC_STATUS, RCVD_IN_MSPIKE_H3, 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, 01 Sep 2021 08:46:29 -0000 On Sep 1 17:23, Takashi Yano wrote: > On Wed, 1 Sep 2021 10:07:48 +0200 > Corinna Vinschen wrote: > > On Sep 1 09:16, Takashi Yano wrote: > > > On Wed, 1 Sep 2021 08:02:20 +0900 > > > Takashi Yano wrote: > > > > On Tue, 31 Aug 2021 17:50:14 +0200 > > > > Corinna Vinschen wrote: > > > > > So for the time being I suggest the below patch on top of topic/pipe. > > > > > It contains everything we discussed so far. > > > > > > > > One more thing. 'git log' cannot stop normally with 'q' with your patch. > > > > > > The same happes with 'yes |less'. > > > > > > The cause is that write side cannot detect closing read side because > > > query_hdl (read handle) is still opened. > > > > Oh > > > > my > > > > god. > > > > > > That kills the entire idea of keeping the read handle :( > > One idea is: > > Count read handle and write handle opned using NtQueryObject(). > If the numbers of opened handle are equal each other, only > the write side (pair of write handle and query_hdl) is alive. > In this case, write() returns error. > If read side is alive, number of read handles is greater than > number of write handles. Interesting idea. But where do you do the count? The event object will not get signalled, so WFMO will not return when performing a blocking write. Corinna