Re: PATCH: BZ #14557: __kernel_vsyscall unwind info is incorrect

["H.J. Lu" <hjl dot tools at gmail dot com>]

On Mon, Sep 24, 2012 at 5:41 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Mon, Sep 24, 2012 at 1:45 PM, Roland McGrath <roland@hack.frob.com> wrote:
>> Perhaps it would be cleaner just to put the vDSO entry onto the _ns_loaded
>> list in the static case as the dynamic linker does.  I'm not sure off hand
>> what other bits of functionality this might enable, but it seems like there
>> might be some.  I can vaguely imagine there might be some downside too, but
>> I don't know what it would be.
>
> I have thought about and couldn't find any other benefits of putting the
> vDSO entry onto the _ns_loaded.  To properly populate _ns_loaded,
> we need to duplicate rtld.c's GLRO(dl_sysinfo_dso) handling in
> dl-support.c, which calls other dynamic linker functions.  I don't know
> if they are all supported in libc.a.  I can give it a try.
>

The _ns_loaded list is for ld.so and doesn't support libc.a.  For example, ld.so
calls its own malloc routines to manage memory before entering main.
We can't easily do that in libc.a since we need to set up a few things
for malloc
first.  Unless there is a specific issue we need to address, I'd like to use the
existing patch.  OK to install?

Thanks.


-- 
H.J.