public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
From: "Vivek Das Mohapatra" <vivek@collabora.com>
To: Joseph Myers <joseph@codesourcery.com>
Cc: libc-alpha@sourceware.org
Subject: Re: [RFC][PATCH v13 0/9] Implementation of RTLD_SHARED for dlmopen
Date: Wed, 29 Sep 2021 18:32:38 +0100 (BST)	[thread overview]
Message-ID: <alpine.DEB.2.21.2109291829230.21196@noise.cbg.collabora.co.uk> (raw)
In-Reply-To: <alpine.DEB.2.22.394.2109291636230.2573743@digraph.polyomino.org.uk>

On Wed, 29 Sep 2021, Joseph Myers wrote:

> On Wed, 29 Sep 2021, Vivek Das Mohapatra via Libc-alpha wrote:
>
>>  - The following DSOs no longer have the new flag since they are stubs:
>>    {htl,nptl}/libpthread, dlfcn/libdl, login/libutil, resolv/libanl
>
> The HTL libpthread implementation hasn't been merged into libc, unlike
> NPTL.  Probably it *should* be, but it hasn't happened yet.  (And, thus,
> on Hurd, libanl hasn't been merged into libc because it depends on
> libpthread.)

Hm. Ok. Thanks for the heads up.

I can restore the flag for htl/pthread easily enough.

I'm less certain how to do this properly conditionaly for anl but I'll
look into it. Possibly I'll just restore the flag for libanl (see below).

> On the other hand, librt has been merged into libc on Linux (not Hurd, for
> the same reason of dependency on libpthread), despite that not being
> mentioned in the NEWS entry for 2.34.

It's harmless for a stub library to have the flag if it doesn't need it,
so I'll probably leave librt tagged until it's fully merged (unless it's
easy to to do conditionally given the current setup).

Not worth introducing a ton of complexity into the build system for imo.

  reply	other threads:[~2021-09-29 17:32 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-29 16:26 Vivek Das Mohapatra
2021-09-29 16:26 ` [RFC][PATCH v13 1/9] Define a new dynamic section tag - DT_GNU_FLAGS_1 (bug 22745) Vivek Das Mohapatra
2021-09-29 16:26 ` [RFC][PATCH v13 2/9] Abstract loaded-DSO search code into a helper function Vivek Das Mohapatra
2021-09-29 16:26 ` [RFC][PATCH v13 3/9] Use the new DSO finder " Vivek Das Mohapatra
2021-09-29 16:26 ` [RFC][PATCH v13 4/9] Add DT_GNU_FLAGS_1/DF_GNU_1_UNIQUE to glibc DSOs (bug 22745) Vivek Das Mohapatra
2021-09-29 16:26 ` [RFC][PATCH v13 5/9] Implement dlmopen RTLD_SHARED flag " Vivek Das Mohapatra
2021-09-29 16:26 ` [RFC][PATCH v13 6/9] Add dlmopen / RTLD_SHARED tests Vivek Das Mohapatra
2021-09-29 16:26 ` [RFC][PATCH v13 7/9] Restore separate libc loading for the TLS/namespace storage test Vivek Das Mohapatra
2021-09-29 16:26 ` [RFC][PATCH v13 8/9] Drop DT_GNU_FLAGS_1/DF_GNU_1_UNIQUE from merged libraries Vivek Das Mohapatra
2021-09-29 16:26 ` [RFC][PATCH v13 9/9] NEWS for RTLD_SHARED, RTLD_ISOLATE & DT_GNU_FLAGS_1/DF_GNU_1_UNIQUE Vivek Das Mohapatra
2021-09-29 16:43 ` [RFC][PATCH v13 0/9] Implementation of RTLD_SHARED for dlmopen Joseph Myers
2021-09-29 17:32   ` Vivek Das Mohapatra [this message]
2021-09-29 18:33 ` Adhemerval Zanella
2021-09-29 19:30   ` Vivek Das Mohapatra

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=alpine.DEB.2.21.2109291829230.21196@noise.cbg.collabora.co.uk \
    --to=vivek@collabora.com \
    --cc=joseph@codesourcery.com \
    --cc=libc-alpha@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).