X-Recipient:	archive-cygwin AT delorie DOT com
X-Spam-Check-By:	sourceware.org
Date:	Fri, 27 Mar 2009 12:59:43 +0100
From:	Corinna Vinschen <corinna-cygwin AT cygwin DOT com>
To:	cygwin AT cygwin DOT com
Subject:	Re: [1.7] makeinfo : too many open files
Message-ID:	<20090327115943.GW12738@calimero.vinschen.de>
Reply-To:	cygwin AT cygwin DOT com
Mail-Followup-To:	cygwin AT cygwin DOT com
References:	<667599 DOT 46574 DOT qm AT web25002 DOT mail DOT ukl DOT yahoo DOT com>
MIME-Version:	1.0
In-Reply-To:	<667599.46574.qm@web25002.mail.ukl.yahoo.com>
User-Agent:	Mutt/1.5.19 (2009-02-20)
Mailing-List:	contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id:	<cygwin.cygwin.com>
List-Unsubscribe:	<mailto:cygwin-unsubscribe-archive-cygwin=delorie DOT com AT cygwin DOT 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 Marco,

On Mar 26 22:09, Marco Atzeri wrote:
> Hi Corinna,
> rebase had no effect. 
> 
> I packed the test case in
> http://matzeri.altervista.org/strace/
> as makeinfo_test.tar.xz
> 
> 
> $ cd doc/interpreter
> $ makeinfo -I.. octave.texi

Thanks for your testcase.

I got the same problem in cygheap.cc as you saw.  makeinfo was unable to
raise the internal Cygwin heap size.  And it was no wonder, really,
because the cygheap had already been raised so much that it hit the
memory location of the next DLL in memory at some 0x67xxxxxx address.
This is *very* unlikely for the cygheap, unless...

What happens is that makeinfo calls pathconf(3) on all input files.  At
one point I had to rewrite pathconf for 1.7 and while doing that I missed
to free memory I allocated for a temporary file handler structure.  This
happens in every single call to pathconf(3).  Ouch!

I fixed this bug in CVS.  A new Cygwin 1.7 test release is due soon.


Thanks again for your report and the testcase,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Project Co-Leader          cygwin AT cygwin DOT com
Red Hat

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Problem reports:       http://cygwin.com/problems.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/

- Raw text -