This is the mail archive of the
cygwin
mailing list for the Cygwin project.
Re: [PATCH] tests: don't assume getdtablesize () <= 10000000
- From: Christopher Faylor <cgf-use-the-mailinglist-please at cygwin dot com>
- To: cygwin at cygwin dot com
- Date: Wed, 25 Sep 2013 11:04:16 -0400
- Subject: Re: [PATCH] tests: don't assume getdtablesize () <= 10000000
- Authentication-results: sourceware.org; auth=none
- 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>
- Reply-to: cygwin at cygwin dot com
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