From: David Anderson <davea@quasar.engr.sgi.com>
To: gdb-patches@sourceware.org
Cc: jimb@red-bean.com, eirik@hackrat.com
Subject: Re: [PATCH] Use mmap for symbol tables
Date: Tue, 31 Jan 2006 04:40:00 -0000 [thread overview]
Message-ID: <200601310437.k0V4bcJ7208753@quasar.engr.sgi.com> (raw)
Jim Blandy
> I understand that it would make your BFD code more complicated, but it
> seems to me you want to map individual sections, not entire files.
> Again, this will still share memory with the block cache, so aside
> from the complexity I don't see the downside.
Eirik Fuller
>I don't see the upside of making the code more complicated. The
>downside of the extra complication is that it makes the patch less
>likely to the point of never actually existing. :-)
>
>Could you be more specific about why multiple mmap regions per file are
>preferable? (It might help to keep in mind that I'm using PROT_READ and
>MAP_SHARED). The only downside I can see is the (relatively small)
>fraction of each symbol table which is not accessed via mmap, but that
>doesn't use memory, just virtual address space (if it does use memory,
>that contradicts the "not accessed" part).
Hope the following is not out of line.
The phrase 'just virtual address space' should not, IMO, be
uttered so cavalierly. Mapping in one glob will in some cases
make gdb useless for some apps (fearless prediction).
Been there, done that, with IRIX/dbx and 'just virtual address
space' was a disaster for a few big apps with large value to
the owners/users (when the apps needed debugging). Meaning we
did exactly what Eirik Fuller proposes and had to drop back to
exactly what Jim Blandy proposes. [The problem was with 32 bit
address space of the debugger as it existed a few years ago.]
And don't mmap in a whole core file for the same reason (not
that such is being proposed, this is just another thought).
David Anderson
next reply other threads:[~2006-01-31 4:40 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-01-31 4:40 David Anderson [this message]
2006-01-31 5:00 ` Eirik Fuller
2006-01-31 5:34 ` Jim Blandy
2006-01-31 14:00 ` Daniel Jacobowitz
2006-01-31 18:39 ` Jim Blandy
2006-02-01 18:11 ` Eirik Fuller
2006-01-31 17:45 ` David Anderson
2006-01-31 18:24 ` Jim Blandy
-- strict thread matches above, loose matches on Subject: below --
2006-01-31 4:53 David Anderson
2006-01-29 23:36 Eirik Fuller
2006-01-30 5:04 ` Jim Blandy
2006-01-30 11:44 ` Eirik Fuller
2006-01-30 18:07 ` Jim Blandy
2006-01-30 18:59 ` Eirik Fuller
2006-01-30 22:11 ` Jim Blandy
2006-01-31 0:38 ` Eirik Fuller
2006-01-31 1:49 ` Jim Blandy
2006-01-31 3:12 ` Eirik Fuller
2006-01-31 21:48 ` Mark Kettenis
2006-02-01 17:52 ` Eirik Fuller
2006-02-01 6:04 ` Michael Snyder
2006-01-30 11:34 ` Andrew STUBBS
2006-01-30 11:42 ` Corinna Vinschen
2006-01-30 11:48 ` Andrew STUBBS
2006-01-31 2:23 ` Daniel Jacobowitz
2006-01-31 3:31 ` Eirik Fuller
2006-01-31 3:38 ` Daniel Jacobowitz
2006-02-07 22:05 ` Eirik Fuller
2006-02-20 15:52 ` Daniel Jacobowitz
2006-01-31 5:28 ` Jim Blandy
2006-01-31 13:59 ` Daniel Jacobowitz
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=200601310437.k0V4bcJ7208753@quasar.engr.sgi.com \
--to=davea@quasar.engr.sgi.com \
--cc=eirik@hackrat.com \
--cc=gdb-patches@sourceware.org \
--cc=jimb@red-bean.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