From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mout.kundenserver.de (mout.kundenserver.de [212.227.126.187]) by sourceware.org (Postfix) with ESMTPS id DACDA3858402 for ; Tue, 14 Sep 2021 11:03:57 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org DACDA3858402 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 (mreue012 [212.227.15.167]) with ESMTPSA (Nemesis) id 1MT7ip-1mLXgC3EUB-00UXU4 for ; Tue, 14 Sep 2021 13:03:55 +0200 Received: by calimero.vinschen.de (Postfix, from userid 500) id 41B5BA80D65; Tue, 14 Sep 2021 13:03:55 +0200 (CEST) Date: Tue, 14 Sep 2021 13:03:55 +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: <20210911113517.f74fc3ac1971bbf04c7a9bd1@nifty.ne.jp> <695ce1f4-4f7d-f3f3-6dd3-087467d67b28@cornell.edu> <20210912174849.3d38107568065a95aeb19c7c@nifty.ne.jp> <20210912200423.667e40eb1adc52461bbefa20@nifty.ne.jp> <20210914043718.f420491c6723f3dc2e2d9753@nifty.ne.jp> <20210914170838.6393026ac3eaa4cd4e3d9618@nifty.ne.jp> <20210914185623.0488635bf6d1e925baee5fe8@nifty.ne.jp> <20210914191924.5092f63762d62c88431b5313@nifty.ne.jp> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20210914191924.5092f63762d62c88431b5313@nifty.ne.jp> X-Provags-ID: V03:K1:DgQSLGSwWOBu2GzXlvw23VHScjHVkU89/sp8+I/hyv0anPloHrK bYj2TlRXJCoUKu8O/FXlBpNORy4txujCG/eLuPqmgQOl+94v5QT0xltvToiHP8dRwKq8E0Y i6gswGwPqgbg2uARQO1QSkYYFFq1G57nCmzUNYxAYWD+WGUJNO6npxFOEEcILhSo3gT6zcJ KKwC3LhMbIw1f72yHga2A== X-UI-Out-Filterresults: notjunk:1;V03:K0:fvtbYc50CEE=:AwI+f4nz3COEku2LbTkCwt 8DSqg/6vtjKi5qgXBoqlBDRrbxf4dwO2v8zyeLYTqqw3kxhHfc8oEeJWF3SbYi+r9+SWkFSEL aAvJO1IDIQ4tQpf8fxOO8XapbQ+mdn4A2NabId+wKvadn8l9U2NUjesMPllugwRDi6lkJa5qz Bw0MHIQLpGk7iTbnrcJlQCO5FDI9Ab3cXafzHWDKHPh1ReMsLGlAR6BqwqKu/AhIf2+fViuJl ughXHgnoB8fPtJu/2b+9Lt2oEGKEndXija0jE0EidmU+JnhlQulF5XQ3JwS8BeCAFbTg2Flch rjYdMmkOlYy9WZrS+HRtrMraXRMQwiyAM1+UfgQ8AGcOpmukFA1T+acI/HnmQ7tsJac/AX5Z+ OOyXvlyibNiD8Fv9086UFMjtCRDJxYyV6NCJmqiUIOY8mztx6VaA9hW0BGsWQmYNDxdN9zMp1 C2wvRJWIvaQYf6hzkpbKgLurhAlWNyVZWBdKNxWefiiFIV5eHptv8y+BAk7u9Xh2nQ38DJXFW HUVN/WXO8uAFz6kFi3dKx5nFapVuxbAgTQKnKpHngSWj8In73qr+SdOK/OVWhb+Tx85Tm19wz 6/o/22EwY2V5s4XyPtKjRbvtGVlRbn+bDiLKFkGaJnIk4Fy3pCpNZ/DFAGeMllMnFVvqcp2Vr lV4EhlxeF1/sHY3o30lMUGUqySOpCtqlYqSWcdkPXNmdu7TbypkzawmUOPGLZvTpuYGb+x4Jm UjsNzUaV6RVHXoTxSvoajTMI1ZPbWtVw+CB23heASeTbPCYhxMUwiGr22tg= 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_H2, 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: Tue, 14 Sep 2021 11:03:59 -0000 On Sep 14 19:19, Takashi Yano wrote: > On Tue, 14 Sep 2021 18:56:23 +0900 > Takashi Yano wrote: > > On Tue, 14 Sep 2021 11:03:39 +0200 > > Corinna Vinschen wrote: > > > I don't quite follow the argument. Blocking pipes are using > > > asynchronous IO, so they are in fact not blocking calls on the > > > OS level. After calling NtWriteFile, the blocking variation > > > will go into the subsequent > > > > > > waitret = cygwait (evt, INFINITE, cw_cancel | cw_sig); > > > > > > So, wouldn't you get the same effect by keeping the pipe in > > > FILE_PIPE_QUEUE_OPERATION mode and just add a timeout to the above > > > cygwait call and handle select_sem in a not yet existing WAIT_TIMEOUT > > > conditional? > > > > Sounds reasonable. I revised the patches. Do you mean something like > > patch attached? > > Revised a bit. Pushed. I added a bit of text to the commit message in terms of setting pipes to blocking when exec'ing a non-Cygwin tool. Thanks, Corinna