This is the mail archive of the cygwin-patches@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: New stat stuff (was [PATCH] improve performance of stat() ope rations (e.g. ls -lR ))


On Tue, May 28, 2002 at 10:27:23PM -0400, Christopher Faylor wrote:
>On Tue, May 28, 2002 at 10:25:37PM -0400, Christopher Faylor wrote:
>>On Tue, May 28, 2002 at 09:58:52PM -0400, Don Bowman wrote:
>>>
>>>So I've performed a mini-benchmark of Chris' changes.
>>>
>>>I did a ls -lR >/dev/null of the cygwin source tree on my
>>>notebook.
>>>
>>>Baseline (current setup.exe install): 1m14.9s
>>>'statquery' patch I sent earlier: 4.081s
>>>Current CVS tree: 3.718s
>>>Current CVS tree w/ -E switch to mount: 3.711s
>>>Current CVS tree w/ -X switch to mount: 3.716s
>>>
>>>Not all that scientific, I ran each twice, took the 2nd timing.
>>>So, looks good, excellent work. I still don't see any 
>>>difference on the -E or the -X tho'.
>>
>>That has got to mean that there's something wrong in the stat
>>logic.  I didn't do anything to speed up the normal case, AFAIK,
>>unless you're doing this on a FAT/FAT32 partition.
>
>Actually, even in that case, it shouldn't make that big a deal.

Yep.  The stat logic was screwed up.  It was never reading the
file.

I'm rerunning a snapshot now.

cgf


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]