Re: [PATCH] On-demand loading of shlib's debuginfo

Pedro Alves
2011-07-21T10:59:39Z

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