X-Recipient:	archive-cygwin AT delorie DOT com
DomainKey-Signature:	a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:date:from:to:subject:message-id:reply-to
	:references:mime-version:content-type:in-reply-to; q=dns; s=
	default; b=fcVHlA92JRKm6mGh3eTf5Cma3lOXGEcYCTr95FVlST6ysXLfM1WQa
	Ts1r1babLDGbgDm7yvnsxWizzncaHlgv7IuEk+p92tu40UYoAN4RyRx2NlvW7/JA
	MXIKkrHh1thbq1GJQRU1hjPX5wKrZBNgnBkNyEHVOlT0GDX8azppMo=
DKIM-Signature:	v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:date:from:to:subject:message-id:reply-to
	:references:mime-version:content-type:in-reply-to; s=default;
	bh=3vY+RqqdGMzz7r7EvwGWMkTwmeY=; b=Pph6VJM38sxr76AqduPWAzMo0Nv7
	T3F/Zza4yYURTtDkH3b1WGDlMGEd8dmiXAcyeG+XVcRrCK5iINYJEkHnYGQ5OYnB
	rfENNMIPDsYizPtGOqI1E4ofPxXmfzhgMoCJ4XRkA7Yu4AX4a827IC9/xPpPqKhG
	ywPpPOSz9xZRWag=
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
Authentication-Results:	sourceware.org; auth=none
X-Virus-Found:	No
X-Spam-SWARE-Status:	Yes, score=5.2 required=5.0 tests=AWL,BAYES_00,FSL_HELO_NON_FQDN_1,HELO_LOCALHOST,RCVD_IN_PBL,RCVD_IN_RP_RNBL,RCVD_IN_SORBS_DUL,RP_MATCHES_RCVD autolearn=no version=3.3.2
X-HELO:	mho-02-ewr.mailhop.org
X-Mail-Handler:	Dyn Standard SMTP by Dyn
X-Report-Abuse-To:	abuse AT dyndns DOT com (see http://www.dyndns.com/services/sendlabs/outbound_abuse.html for abuse reporting information)
X-MHO-User:	U2FsdGVkX19YnUFI+CetFJp/Fyie2DS2
Date:	Wed, 25 Sep 2013 00:32:25 -0400
From:	Christopher Faylor <cgf-use-the-mailinglist-please AT cygwin DOT com>
To:	cygwin AT cygwin DOT com
Subject:	Re: [PATCH] tests: don't assume getdtablesize () <= 10000000
Message-ID:	<20130925043225.GB1192@ednor.casa.cgf.cx>
Reply-To:	cygwin AT cygwin DOT com
Mail-Followup-To:	cygwin AT cygwin DOT com
References:	<517EF2F1 DOT 2030802 AT cs DOT ucla DOT edu> <5241DBE6 DOT 4000007 AT redhat DOT com>
MIME-Version:	1.0
In-Reply-To:	<5241DBE6.4000007@redhat.com>
User-Agent:	Mutt/1.5.20 (2009-06-14)

On Tue, Sep 24, 2013 at 12:37:26PM -0600, Eric Blake wrote:
>[adding cygwin]
>
>On 04/29/2013 04:23 PM, Paul Eggert wrote:
>> ---
>
>> +
>> +	tests: don't assume getdtablesize () <= 10000000
>> +	* modules/cloexec-tests:
>> +	* modules/dup2-tests:
>> +	* modules/dup3-tests:
>> +	* modules/nonblocking-tests:
>> +	* modules/posix_spawn_file_actions_addclose-tests:
>> +	* modules/posix_spawn_file_actions_adddup2-tests:
>> +	* modules/posix_spawn_file_actions_addopen-tests:
>> +	* modules/unistd-safer-tests:
>> +	Depend on the getdtablesize module.
>> +	* tests/test-cloexec.c:
>> +	* tests/test-dup-safer.c:
>> +	* tests/test-dup2.c:
>> +	* tests/test-dup3.c:
>> +	* tests/test-fcntl.c:
>> +	* tests/test-nonblocking.c:
>> +	* tests/test-posix_spawn_file_actions_addclose.c:
>> +	* tests/test-posix_spawn_file_actions_adddup2.c:
>> +	* tests/test-posix_spawn_file_actions_addopen.c:
>> +	Don't assume getdtablesize () <= 10000000.
>
>This patch causes failures on at least test-fcntl and test-dup2 on
>cygwin (both 32-bit and 64-bit); there, getdtablesize() currently
>returns the current runtime value, but this value starts at 256, and
>automatically expands as needed at runtime up to 3200.  I think cygwin
>should be patched to make getdtablesize() return a constant 3200 (rather
>than the current runtime value);

Why?  What does "3200" have to do with anything?  There is not supposed
to be a hard-coded upper limit.

cgf

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