public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
From: Corinna Vinschen <corinna-cygwin@cygwin.com>
To: cygwin@cygwin.com
Subject: Re: [1.7] makeinfo : too many open files
Date: Fri, 27 Mar 2009 13:15:00 -0000	[thread overview]
Message-ID: <20090327115943.GW12738@calimero.vinschen.de> (raw)
In-Reply-To: <667599.46574.qm@web25002.mail.ukl.yahoo.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/

  reply	other threads:[~2009-03-27 12:00 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-27  1:31 Marco Atzeri
2009-03-27 13:15 ` Corinna Vinschen [this message]
  -- strict thread matches above, loose matches on Subject: below --
2009-03-26 21:40 Marco Atzeri
2009-03-26 15:23 Marco Atzeri
2009-03-26 16:01 ` Corinna Vinschen
2009-03-26 17:56   ` Christopher Faylor
2009-02-21  3:18 [1.7] makeinfo: " Charles Wilson
2009-02-24  9:21 ` Corinna Vinschen
2009-02-24 12:49   ` Eric Blake

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20090327115943.GW12738@calimero.vinschen.de \
    --to=corinna-cygwin@cygwin.com \
    --cc=cygwin@cygwin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).