From: "Andreas K. Huettel" <dilfridge@gentoo.org>
To: Florian Weimer <fweimer@redhat.com>,
libc-alpha@sourceware.org,
Siddhesh Poyarekar <siddhesh@sourceware.org>
Subject: Re: [PATCH v2 3/4] configure: Disable lazy binding by default
Date: Tue, 18 Jul 2023 18:25:27 +0200 [thread overview]
Message-ID: <15049364.O9o76ZdvQC@pinacolada> (raw)
In-Reply-To: <064ff58c-9d2d-144e-2915-597593bfe66a@sourceware.org>
[-- Attachment #1: Type: text/plain, Size: 1846 bytes --]
Am Montag, 17. Juli 2023, 17:55:35 CEST schrieb Siddhesh Poyarekar via Libc-alpha:
> On 2023-07-17 11:30, Florian Weimer via Libc-alpha wrote:
> > * Siddhesh Poyarekar:
> >
> >> On 2023-07-13 05:38, Florian Weimer via Libc-alpha wrote:
> >>> * Siddhesh Poyarekar via Libc-alpha:
> >>>
> >>>> +‘--disable-bind-now’
> >>>> + Enable lazy binding for installed shared objects and programs.
> >>>> + Lazy binding may improve program load times but it will disable
> >>>> + security hardening that enables full RELRO and a read-only global
> >>>> + offset table (GOT).
> >>> This description is not correct because as far as I can see,
> >>> --disable-bind-now does not enable lazy binding if the toolchain or the
> >>> build flags default to it.
> >>> I think we should remove this flag, rather than making
> >>> --disable-bind-now work as documented.
> >>
> >> I feel like this needs more thought either way then, no point blocking
> >> 2.38 for it. What's the use case for lazy binding? Are people
> >> actually using this in the wild and would we be breaking things for
> >> them?
> >
> > I don't think it matters. We have very few symbols that can be bound
> > lazily. Most symbolic bindings are against data symbols, which cannot
> > be bound lazily, so the performance win for glibc itself is minuscule.
> >
> > I think it may have been beneficial before we had the libc_hidden
> > mechanism. I suppose libpthread integration also helped to decreasing
> > binding overhead.
>
> Fair enough, I'll respin this by dropping the option and let Andreas
> decide if it's suitable for 2.38. If not, I'll just queue it up for 2.39.
Let's queue it for 2.39 then.
--
Andreas K. Hüttel
dilfridge@gentoo.org
Gentoo Linux developer
(council, toolchain, base-system, perl, libreoffice)
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 981 bytes --]
next prev parent reply other threads:[~2023-07-18 16:25 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-06-29 18:41 [PATCH 0/4] Update default build configuration Siddhesh Poyarekar
2023-06-29 18:41 ` [PATCH 1/4] configure: Use autoconf 2.71 Siddhesh Poyarekar
2023-06-29 19:48 ` Joseph Myers
2023-06-29 19:56 ` Siddhesh Poyarekar
2023-06-29 20:03 ` Siddhesh Poyarekar
2023-06-29 20:07 ` Joseph Myers
2023-06-29 22:16 ` Zack Weinberg
2023-06-29 18:41 ` [PATCH 2/4] configure: Default --enable-stack-protector to strong Siddhesh Poyarekar
2023-06-29 18:41 ` [PATCH 3/4] Disable lazy binding by default Siddhesh Poyarekar
2023-06-29 21:54 ` Michael Hudson-Doyle
2023-06-29 22:16 ` Siddhesh Poyarekar
2023-06-29 18:41 ` [PATCH 4/4] configure: Disable building libcrypt " Siddhesh Poyarekar
2023-06-29 18:45 ` Dmitry V. Levin
2023-06-29 19:02 ` Siddhesh Poyarekar
2023-07-14 6:21 ` Sam James
2023-07-17 15:24 ` Siddhesh Poyarekar
2023-07-17 17:26 ` Sam James
2023-06-30 1:42 ` [PATCH v2 0/4] Update default build configuration Siddhesh Poyarekar
2023-06-30 1:42 ` [PATCH v2 1/4] configure: Use autoconf 2.71 Siddhesh Poyarekar
2023-07-14 21:00 ` Carlos O'Donell
2023-07-17 14:21 ` [PATCH v3] " Siddhesh Poyarekar
2023-07-17 14:46 ` Siddhesh Poyarekar
2023-06-30 1:42 ` [PATCH v2 2/4] configure: Default --enable-stack-protector to strong Siddhesh Poyarekar
2023-07-13 9:51 ` Florian Weimer
2023-07-17 14:55 ` Siddhesh Poyarekar
2023-07-17 15:45 ` Florian Weimer
2023-07-17 15:52 ` Siddhesh Poyarekar
2023-08-03 10:06 ` Florian Weimer
2023-06-30 1:42 ` [PATCH v2 3/4] configure: Disable lazy binding by default Siddhesh Poyarekar
2023-07-13 9:38 ` Florian Weimer
2023-07-17 14:57 ` Siddhesh Poyarekar
2023-07-17 15:30 ` Florian Weimer
2023-07-17 15:55 ` Siddhesh Poyarekar
2023-07-18 16:25 ` Andreas K. Huettel [this message]
2023-06-30 1:42 ` [PATCH v2 4/4] configure: Disable building libcrypt " Siddhesh Poyarekar
2023-07-13 9:40 ` Florian Weimer
2023-07-14 4:16 ` Mark Harris
2023-07-17 15:13 ` Siddhesh Poyarekar
2023-07-17 16:44 ` [PATCH v3] " Siddhesh Poyarekar
2023-07-17 17:18 ` Andreas Schwab
2023-07-17 18:17 ` Siddhesh Poyarekar
2023-07-20 18:05 ` Joseph Myers
2023-07-20 18:39 ` Andreas K. Huettel
2023-07-07 10:31 ` [ping][PATCH v2 0/4] Update default build configuration Siddhesh Poyarekar
2023-07-12 21:51 ` [ping2][PATCH " Siddhesh Poyarekar
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=15049364.O9o76ZdvQC@pinacolada \
--to=dilfridge@gentoo.org \
--cc=fweimer@redhat.com \
--cc=libc-alpha@sourceware.org \
--cc=siddhesh@sourceware.org \
/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).