Re: last snapshot is not better than previous version

[Corinna Vinschen <corinna-cygwin@cygwin.com>]

[-- Attachment #1: Type: text/plain, Size: 2405 bytes --]

Hi Denis,

On May 26 19:21, Denis Excoffier wrote:
> Hello,
> 
> I tried the snapshot dated 2016-05-20. Okay, this one was not announced.
> 
> Functionally it seems ok. But some simple tasks now take a lot of time compared to the last non-snapshot version (2.5.1):
> 
> 1) searching into a small text file
> A simple text file (created with "seq 10000") takes several seconds to load in vim. Lookup for the string "9999" then takes 5 seconds. Previously, searching "9999999" in a bigger file (created with "seq 10000000") used to take 2 seconds only.

I can't reproduce this.

> Note: The "seq 10000000 > myfile" itself makes no notable difference.
> 
> 2) ls in a large directory
> My /usr/bin contains 8098 files. Previously, a plain "ls -F" in this folder took some initialization time (2 seconds), then the display of the files was quite instantaneous. With the new snapshot, the initialization (stat and sort) takes more or less the same time, but afterwards, each single line is displayed at a visible pace, a clear screen is executed after each set of ~24 lines and the whole directory is listed in about 10 minutes.

I can't reproduce this either.

> Note: "ls | wc -l" itself makes no notable difference.
> 
> This occurs under X-Windows (xterm) or not (login window). Under x86, Windows 7. I switched back to 2.5.1.
> 
> This message just in case it could be useful.

The problem is, there's no change at all in the snapshot which would
explain a slowdown of such basic functions:

- Assorted select(2) improvements:

  commits a23e6a35d896a075640db714b28ce74bb6b8d7ff
          e5665d8c930485d5ac6d8913573e27b9e5043d92

- There's a change to the sched_* functions, which is used only rarely
  and which doesn't introduce any waits or loops.

  commits 450f557feee5dd4900e3fd4f16cbb6cabe89fbc6
          4b51e4c142445c133d6075d2a68ce98e8ec6058b

- Fix a size restriction to 2 GB areas

  commit 0aa738220bb9dea2ad479e484560767b36701947

- Replace toupper with explicit tests of character values in the rare
  case TZ isn't set

  commit 030d86d04dbd6ae878ff894256c5a465890d5970

And that's it.  All other changes since 2.5.1 only affect non-Cygwin
targets in newlib and libgloss.


Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]