X-Recipient: archive-cygwin@delorie.com
X-Spam-Check-By: sourceware.org
Date: Mon, 29 Jun 2009 10:42:12 -0400
From: Christopher Faylor <cgf-use-the-mailinglist-please@cygwin.com>
To: cygwin@cygwin.com
Subject: Re: [1.7.0-50] scp progress counter flies through first 175 MB or 	so
Message-ID: <20090629144212.GD19123@ednor.casa.cgf.cx>
Reply-To: cygwin@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
References: <2ul445d2cfjj1q2t2viropiikoj70slglb@4ax.com> <20090625101039.GP7289@calimero.vinschen.de> <20090625140508.GA17040@ednor.casa.cgf.cx> <20090625143651.GR7289@calimero.vinschen.de> <20090625145005.GB18338@ednor.casa.cgf.cx> <20090628193155.GA10407@ednor.casa.cgf.cx> <20090629092857.GZ30864@calimero.vinschen.de> <20090629095322.GD30864@calimero.vinschen.de> <20090629142348.GB19123@ednor.casa.cgf.cx> <20090629143840.GI30864@calimero.vinschen.de>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20090629143840.GI30864@calimero.vinschen.de>
User-Agent: Mutt/1.5.19 (2009-01-05)
Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm
Precedence: bulk
List-Id: <cygwin.cygwin.com>
List-Unsubscribe: <mailto:cygwin-unsubscribe-archive-cygwin=delorie.com@cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe@cygwin.com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin@cygwin.com>
List-Help: <mailto:cygwin-help@cygwin.com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
Delivered-To: mailing list cygwin@cygwin.com

On Mon, Jun 29, 2009 at 04:38:40PM +0200, Corinna Vinschen wrote:
>On Jun 29 10:23, Christopher Faylor wrote:
>> On Mon, Jun 29, 2009 at 11:53:22AM +0200, Corinna Vinschen wrote:
>> >On Jun 29 11:28, Corinna Vinschen wrote:
>> >>On Jun 28 15:31, Christopher Faylor wrote:
>> >>>Btw, Corinna, were you proposing turning the "FIXME" code in peek_pipe
>> >>>back on?
>> >>[...]
>> >>I tested sftp with big files (Gigs) and it appears to work fine with
>> >>the FIXME code switched back to life.  I tested the unison command
>> >>using Cygwin's unison 2.31.4-2 under tcsh and it worked fine.  I tested
>> >>various combinations with pipes created by a native application and
>> >>with native applications on send and receive side of a Cygwin pipe and
>> >>all my testcases worked as expected.
>> >>
>> >>So, maybe we should really give it a try again.
>> >
>> >Btw., I also tried the rsync of 9000 files with the FIXME code active
>> >and it worked a couple of times in a row without fail.
>> 
>> I did make a change subsequent to your turning off this code.  Maybe
>> that was enough to get things working.  Either that or, if you are using
>> the trunk code, maybe you aren't hitting this at all anymore since there
>> are overlapped I/O accomodations.
>> 
>> Anyway, I wouldn't mind giving the code another try if you are up for
>> it.  Although I didn't write the code originally, it has always bothered
>> me that it didn't work because it seems like it should have.
>> 
>> Shall I turn it on, create a snapshot, and then ask our legion of
>> testers to try it?
>
>Sure, go ahead.  I would even go so far as to create a new -51 test
>release to get some more testing of this.

How about if we do that after seeing what happens with the snapshot
(especially since I already sent the email :-|)?

If we get good preliminary reports then a new version is certainly
called for.

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

