public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
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 --]

  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).