This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] On-demand loading of shlib's debuginfo
- From: Pedro Alves <pedro at codesourcery dot com>
- To: gdb-patches at sourceware dot org
- Cc: Jan Kratochvil <jan dot kratochvil at redhat dot com>, Sergio Durigan Junior <sergiodj at redhat dot com>
- Date: Thu, 21 Jul 2011 11:58:44 +0100
- Subject: Re: [PATCH] On-demand loading of shlib's debuginfo
- References: <m3tyajfcsq.fsf@redhat.com> <20110720205313.GA2611@host1.jankratochvil.net>
On Wednesday 20 July 2011 21:53:14, Jan Kratochvil wrote:
> On Tue, 19 Jul 2011 00:23:49 +0200, Sergio Durigan Junior wrote:
> > With that in mind, we decided to tackle this problem progressively, and the
> > first part of the solution is ready for submission.
>
> That is currently it still touches the solib files on disk for the purpose of
> solib_map_sections so that will be a different patch, looking forward.
I had skimmed the patch only, and reserved commenting until I had
a chance of reading the patch carefuly and figuring it out
myself, but since you raise this...
Is that the reason then that svr4_match_pc_solist goes through
the link map to map a PC to a so_list, instead of having a
generic implementation that matches the PC to the so_list's
loaded (bfd) sections (that is, just call solib_contains_address_p) ?
If so, it'd make more sense IMO to remove that from this patch,
and add it only along with a change that lazies mapping in the bfd
and its sections as well. I'm not sure how safe that will be.
--
Pedro Alves