X-Recipient: archive-cygwin@delorie.com X-SWARE-Spam-Status: No, hits=-1.4 required=5.0 tests=AWL,BAYES_00 X-Spam-Check-By: sourceware.org Message-ID: <7a1785a5f0a1d0a8956cd10e573f2e53.squirrel@www.webmail.wingert.org> In-Reply-To: <4C092F96.5040800@cygwin.com> References: <4C03D6C5.4050004@x-ray.at> <80373222dd5d43b134a5ede7036e7674.squirrel@www.webmail.wingert.org> <20100602080626.GV16885@calimero.vinschen.de> <383c8b44a088dad09a0b77d3299feda7.squirrel@www.webmail.wingert.org> <20100602174848.GA14172@ednor.casa.cgf.cx> <20100603235944.GA12167@ednor.casa.cgf.cx> <20100604024422.GB12167@ednor.casa.cgf.cx> <78e7b77657c0cfcd63dc22ad9679bc85.squirrel@www.webmail.wingert.org> <20100604045807.GC12167@ednor.casa.cgf.cx> <2570318aabfa537bf24c8d3a55f9dcd4.squirrel@www.webmail.wingert.org> <4C092F96.5040800@cygwin.com> Date: Fri, 4 Jun 2010 10:07:58 -0700 Subject: Re: Cygwin Performance and stat() From: "Christopher Wingert" To: cygwin@cygwin.com User-Agent: SquirrelMail/1.4.20 MIME-Version: 1.0 Content-Type: text/plain;charset=iso-8859-1 Content-Transfer-Encoding: 8bit Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: cygwin-owner@cygwin.com Mail-Followup-To: cygwin@cygwin.com Delivered-To: mailing list cygwin@cygwin.com Again, not production, this was to highlight a point and to enhance (I would assume) the most common use case (file to file rsync). Plenty of solutions for a good patch, complete the Windows version of stat() or call the Cygwin version if the Window's version fails. Since I don't have any confidence that the "Cygwin organization" will accept any patch I write (per Faylor), I really have no incentive to do the job right. Chris > On 6/4/2010 12:37 PM, Christopher Wingert wrote: >> So here is an example of a performance gain by not using cygwin stat(). >> I >> did this patch in about an hour (with the help of some git code), so I >> wouldn't recommend it for any production use. >> >> On a dry run rsync from my local drive to my NAS (105GB, 34k files, 4k >> directories). The current release cygwin rsync did it in [36m:43s], >> with >> the patch (below) applied, it dropped to [04m:48s]. > > But your patch only does a portion of the work that stat() is supposed to > do. I don't think there's any question that ignoring the need to fill in > certain stat fields will save time. But that's not really a workable > alternative either, if that's what you're implying. > > -- > Larry Hall http://www.rfk.com > RFK Partners, Inc. (508) 893-9779 - RFK Office > 216 Dalton Rd. (508) 893-9889 - FAX > Holliston, MA 01746 > > _____________________________________________________________________ > > A: Yes. >> Q: Are you sure? >>> A: Because it reverses the logical flow of conversation. >>>> Q: Why is top posting annoying in email? > > -- > 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 > > -- 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