public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
From: John Hood <cgull@glup.org>
To: cygwin@cygwin.com
Subject: Re: last snapshot is not better than previous version
Date: Sat, 04 Jun 2016 03:52:00 -0000	[thread overview]
Message-ID: <5752506F.3040602@glup.org> (raw)
In-Reply-To: <20160531091651.GB24015@calimero.vinschen.de>

On 5/31/2016 5:16 AM, Corinna Vinschen wrote:
> On May 30 19:00, Denis Excoffier wrote:
>>> On 2016-05-30 10:25, Corinna Vinschen wrote:
>>>
>>>
>>> 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
>> Thank you for your answer. However, i do insist.
>> I use again the file created with "seq 10000 > 10k" and still
>> look-up the string "9999" inside it.
>>
>> I perform:
>> strace -o vi.out /usr/bin/vim 10k
>> and inside vi:
>> :0
>> /9999
>> :q
>>
>> Without snapshot, vi.out contains 15133 lines
>> With snapshot, vi.out contains 426553 lines. Huh?
> I tested with 2.5.1, with the latest snapshot and with the latest from
> git master, on 32 and 64 bit.  In all cases, the above results in about
> 1000 'select_stuff::cleanup: calling cleanup routines' lines in vi.out,
> give or take, depending on my typing speed.
>
>> Without snapshot, the 6 lines:
>> ------------------------------
>> select: sel.always_ready 0
>> select: sel.wait returns 0
>> select_stuff::cleanup: calling cleanup routines
>> select_stuff::destroy: deleting select records
>> select: recalculating ms
>> select: ms now 0
>> ------------------------------
>> occur about 450 times
>>
>> With snapshot, the 6 lines:
>> ------------------------------
>> select: sel.always_ready 0
>> select: sel.wait returns 0
>> select_stuff::cleanup: calling cleanup routines
>> select_stuff::destroy: deleting select records
>> select: recalculating us
>> select: us now 0
>> ------------------------------
>> occur more than 69000 times
>>
>> Something is probably wrong near here. Please tell me what i should experiment next.
>>
>> Al least i have found a wrong line in select.cc: line 4, the copyright year is missing. ;-)
> Yeah, no worries.
>
> Again, I can *not* reproduce this.  Maybe we should start at the basics,
> a cygcheck output.  What's your OS?  I'm testing on a 64 bit W10.
> What's your terminal?  I tried from Mintty as well as from a console.
Corinna asked me to look at this since I coded the select()-related 
changes she mentioned.  I can't reproduce either of your cases.

Some things I'll mention:

1) Your Cygwin install seems to be in your Windows home directory. 
There's nothing wrong with that but it is unusual.  Is there some reason 
the usual location doesn't work?

2) The cygcheck output shows you being a member of over 50 groups, is 
this system in a domain?  Are any network shares involved?  (I tried 
mounting a local network share, then using ls.  It was 10 times slower-- 
but that was still only 4 seconds to list 11000 files.)

3) You have many thousands of packages installed.  It's not the complete 
Cygwin set but it is a substantial fraction.  It might be worth trying 
an install with a smaller number of packages.  Are you running these 
tests on the same Cygwin install but with different DLLs installed, or 
on different Cygwin installs?

4) You've given an interpretation of the strace output that you see, but 
it might be useful to see the actual strace output for your two test 
cases.  Perhaps the noisy logging for select() is an effect of the 
problem, rather than its cause.  Could you run them again (also with the 
-d flag, which will help us see where the time goes), and send them in 
email?

Regards,

   --jh

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

  reply	other threads:[~2016-06-04  3:52 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-26 19:26 Denis Excoffier
2016-05-30  9:01 ` Corinna Vinschen
2016-05-30 22:41   ` Denis Excoffier
2016-05-31 16:38     ` Corinna Vinschen
2016-06-04  3:52       ` John Hood [this message]
2016-06-04 19:15         ` Denis Excoffier
2016-06-06 14:55           ` Corinna Vinschen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5752506F.3040602@glup.org \
    --to=cgull@glup.org \
    --cc=cygwin@cygwin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).