X-Recipient:	archive-cygwin AT delorie DOT com
X-SWARE-Spam-Status:	No, hits=-0.3 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,SARE_RECV_IP_080178
X-Spam-Check-By:	sourceware.org
Message-ID:	<4C84D12A.2080506@graphtech.co.il>
Date:	Mon, 06 Sep 2010 14:31:54 +0300
From:	Sagi Ben-Akiva <sagi AT graphtech DOT co DOT il>
User-Agent:	Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.11) Gecko/20100714 SUSE/2.0.6-3.1 SeaMonkey/2.0.6
MIME-Version:	1.0
To:	cygwin AT cygwin DOT com
Subject:	Re: Cygwin slow on x64 systems
References:	<4C7B9327 DOT 9030204 AT graphtech DOT co DOT il> <loom DOT 20100831T222419-914 AT post DOT gmane DOT org> <loom DOT 20100901T190726-553 AT post DOT gmane DOT org> <4C7E8A40 DOT 4090102 AT redhat DOT com> <loom DOT 20100901T192451-70 AT post DOT gmane DOT org>
In-Reply-To:	<loom.20100901T192451-70@post.gmane.org>
X-IsSubscribed:	yes
Mailing-List:	contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id:	<cygwin.cygwin.com>
List-Subscribe:	<mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive:	<http://sourceware.org/ml/cygwin/>
List-Post:	<mailto:cygwin AT cygwin DOT com>
List-Help:	<mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender:	cygwin-owner AT cygwin DOT com
Mail-Followup-To:	cygwin AT cygwin DOT com
Delivered-To:	mailing list cygwin AT cygwin DOT com

Hi Magnus,

I applied your patch but I don't notice for any improvement in performance with 
my test case (I still get only 7 lines/sec).

I tried it with several cygwin 1.7.7-1 revisions
Which version of cygwin sources do you use ?
Did you try it with the latest snapshot ?

Thanks,
   Sagi.

Magnus Holmgren wrote:
>>>> I did some testing on my 64-bit Vista system, and it appears that
>>>> CreateThread is the main cause.
>>>
>>> To test this, I removed the call to sigproc_init in dll_crt0_0 and made sure
>>> it was always called in dll_crt0_1 instead. Suddenly the sigp thread started
>>> executing immediately, and its initialization was complete long before
>>> wait_for_sigthread was called.
>>
>> Since you obviously have a patch, would you mind sharing it, rather than
>> just your conclusions from said patch?
>
> Not quite ready for commit as is, but here it is:
>
> Index: src/winsup/cygwin/dcrt0.cc
> ===================================================================
> RCS file: /cvs/src/src/winsup/cygwin/dcrt0.cc,v
> retrieving revision 1.382
> diff -r1.382 dcrt0.cc
> 746,747c746,747
> <    if (!dynamically_loaded)
> <      sigproc_init ();
> ---
>>    //if (!dynamically_loaded)
>>      //sigproc_init ();
> 792c792
> <    if (dynamically_loaded)
> ---
>>    //if (dynamically_loaded)
>
>    Magnus
>



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

- Raw text -