This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH]: hurd: Do not hide rtld symbols which need to be preempted [Was: hurd: Hidden symbols in rtld]
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Samuel Thibault <samuel dot thibault at ens-lyon dot org>
- Cc: GNU C Library <libc-alpha at sourceware dot org>
- Date: Sun, 20 Mar 2016 11:23:54 -0700
- Subject: Re: [PATCH]: hurd: Do not hide rtld symbols which need to be preempted [Was: hurd: Hidden symbols in rtld]
- Authentication-results: sourceware.org; auth=none
- References: <20160320164214 dot GA21096 at var dot home> <20160320170528 dot GO3635 at var dot home> <CAMe9rOrdbuCbsFB_zY+1ffiq01rLStYey1-qeHEsv_850-Akzw at mail dot gmail dot com> <20160320173124 dot GQ3635 at var dot home> <CAMe9rOq=S9RJ0LtKGC3W3htGm1u6N82+qMpyCn1Cm9tMSSA7MQ at mail dot gmail dot com> <20160320174056 dot GV3635 at var dot home> <CAMe9rOpe-ANTHnukqKoQHdtih3RmE_D7SA8HTGSVwyVrixq-4A at mail dot gmail dot com> <20160320175426 dot GW3635 at var dot home>
On Sun, Mar 20, 2016 at 10:54 AM, Samuel Thibault
<samuel.thibault@ens-lyon.org> wrote:
> H.J. Lu, on Sun 20 Mar 2016 10:42:11 -0700, wrote:
>> On Sun, Mar 20, 2016 at 10:40 AM, Samuel Thibault
>> <samuel.thibault@ens-lyon.org> wrote:
>> > H.J. Lu, on Sun 20 Mar 2016 10:36:45 -0700, wrote:
>> >> On Sun, Mar 20, 2016 at 10:31 AM, Samuel Thibault
>> >> <samuel.thibault@ens-lyon.org> wrote:
>> >> > H.J. Lu, on Sun 20 Mar 2016 10:14:40 -0700, wrote:
>> >> >> The question should be asked is if __access in dl-sysdep.c can
>> >> >> be used in ld.so after bootstrap on Hurd.
>> >> >
>> >> > Ok. It seems it's only used in dl_main and in process_envvars only
>> >> > called by dl_main, so it should be fine, right?
>> >> >
>> >> > About __open, it's definitely used for dlopen() :)
>> >>
>> >> I don't know since they are Hurd specific.
>> >
>> > ? dl_main, process_envvars, __open and dlopen are not Hurd specific.
>>
>> Whether a function defined in ld.so can be used after bootstrap is
>> Hurd specific.
>
> Sure, but for __open() for instance, its reference from dl-load.c is not
> Hurd-specific, so it is clearly used for dlopen() after bootstrap.
>
If it isn't OK for Hurd, it should be changed.
--
H.J.