From: Samuel Thibault <samuel.thibault@gnu.org>
To: "H.J. Lu" <hjl.tools@gmail.com>
Cc: Adhemerval Zanella <adhemerval.zanella@linaro.org>,
GNU C Library <libc-alpha@sourceware.org>,
bug-hurd@gnu.org, Thomas Schwinge <thomas@codesourcery.com>,
Joseph Myers <joseph@codesourcery.com>
Subject: Re: [PATCH] Hurd: Add rtld-strncpy-c.c
Date: Tue, 19 Jan 2021 13:44:00 +0100 [thread overview]
Message-ID: <20210119124400.jj535wvskksgvkx7@begin> (raw)
In-Reply-To: <CAMe9rOoQMpDeFzUerscXmc9t7Fj9Wroh9DucXRS9uyLGwYkspw@mail.gmail.com>
H.J. Lu via Libc-alpha, le mar. 19 janv. 2021 04:35:09 -0800, a ecrit:
> On Tue, Jan 19, 2021 at 4:26 AM Adhemerval Zanella
> <adhemerval.zanella@linaro.org> wrote:
> > On 19/01/2021 09:21, H.J. Lu via Libc-alpha wrote:
> > > On Tue, Jan 19, 2021 at 1:02 AM Samuel Thibault <samuel.thibault@gnu.org> wrote:
> > >> H.J. Lu via Libc-alpha, le lun. 18 janv. 2021 18:24:48 -0800, a ecrit:
> > >>> On Mon, Jan 18, 2021 at 4:40 PM H.J. Lu <hjl.tools@gmail.com> wrote:
> > >>>> On Mon, Jan 18, 2021 at 4:24 PM Samuel Thibault <samuel.thibault@gnu.org> wrote:
> > >>>>> More precisely, I don't see why hurd should be doing anything special
> > >>>>> while everything is already taken care of by elf/Makefile's librtld.map
> > >>>>> file scripts.
> > >>>>
> > >>>> I am taking a look.
> > >>>
> > >>> Try this. I have no idea if it really works on hurd.
> > >>
> > >> It is working indeed.
> > >
> > > Should I check it in?
> > >
> >
> > Yes please, the loader should not use ifunc on its own string usage and
> > it is a build issue.
>
> Done.
Thanks!
prev parent reply other threads:[~2021-01-19 12:44 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20201108225251.3195591-1-samuel.thibault@ens-lyon.org>
[not found] ` <87h7nl1425.fsf@dem-tschwing-1.ger.mentorg.com>
2021-01-18 20:05 ` [PATCH] Hurd: Enable ifunc by default Joseph Myers
2021-01-18 20:25 ` Samuel Thibault
2021-01-19 0:04 ` Samuel Thibault
2021-01-19 0:17 ` H.J. Lu
2021-01-19 0:21 ` Samuel Thibault
2021-01-19 0:24 ` Samuel Thibault
2021-01-19 0:40 ` H.J. Lu
2021-01-19 2:24 ` [PATCH] Hurd: Add rtld-strncpy-c.c H.J. Lu
2021-01-19 9:02 ` Samuel Thibault
2021-01-19 12:21 ` H.J. Lu
2021-01-19 12:25 ` Adhemerval Zanella
2021-01-19 12:35 ` H.J. Lu
2021-01-19 12:44 ` Samuel Thibault [this message]
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=20210119124400.jj535wvskksgvkx7@begin \
--to=samuel.thibault@gnu.org \
--cc=adhemerval.zanella@linaro.org \
--cc=bug-hurd@gnu.org \
--cc=hjl.tools@gmail.com \
--cc=joseph@codesourcery.com \
--cc=libc-alpha@sourceware.org \
--cc=thomas@codesourcery.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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).