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=evYjfk1HjQ26JmG7/BkdThmjzriHf2SVdPX08N7W4PoRs6ZUIpA+T E0yAop5VlwXlBjwMwq1R959sbg5ZLZ2U05IJyVznKd+2pOg4I7oKQnMtL0afrwb0 4e++0o2+QsGJsTNkARb3Wyi5jxb0hHZEEwE+28qoCkITPg6RUi/3c0= 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=f3gCkodCHzAFkcndCLhqw11MxHA=; b=V6Vb5AnAN0hREzKLrtYifmyKbKr+ Yg6Enl9bv/Idiz2kmtGhEhZS6Jdg209zeOiwwhbHzXkKI+rJXhHVr1qhDdem+JpK z0/ujLCn5gyE0ZCM9lAnSIUyev0GM0EJKjOPN8wY4kra5VCDKzzczriAz+KywFy/ jIj394qdZJIpO4M= Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , 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.1 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: U2FsdGVkX1+ngePu71njtUjfu52cRy/V Date: Wed, 25 Sep 2013 11:04:16 -0400 From: Christopher Faylor To: cygwin AT cygwin DOT com Subject: Re: [PATCH] tests: don't assume getdtablesize () <= 10000000 Message-ID: <20130925150416.GB3361@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> <20130925043225 DOT GB1192 AT ednor DOT casa DOT cgf DOT cx> <5242D173 DOT 2020901 AT redhat DOT com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5242D173.2020901@redhat.com> User-Agent: Mutt/1.5.20 (2009-06-14) On Wed, Sep 25, 2013 at 06:05:07AM -0600, Eric Blake wrote: >On 09/24/2013 10:32 PM, Christopher Faylor wrote: > >>> 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. >> > >But there IS a hard-coded limit: > >http://cygwin.com/cgi-bin/cvsweb.cgi/src/winsup/cygwin/dtable.h?rev=1.59&content-type=text/x-cvsweb-markup&cvsroot=src Sorry, you're right. I should have checked before talking. And, it apparently you were expecting me to check. Since you've previously made modifications in this area, why not supply a simple patch? 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