From: Adhemerval Zanella <adhemerval.zanella@linaro.org>
To: libc-alpha@sourceware.org
Subject: Re: [RFC][PATCH v8 09/20] Define a new dynamic section tag - DT_GNU_FLAGS_1
Date: Mon, 15 Feb 2021 15:42:30 -0300 [thread overview]
Message-ID: <37a13aab-54f6-c105-f914-3ed14aa58cef@linaro.org> (raw)
In-Reply-To: <20210209171839.7911-10-vivek@collabora.com>
On 09/02/2021 14:18, Vivek Das Mohapatra via Libc-alpha wrote:
> Define a new flags section DT_GNU_FLAGS_1 (no more bits are available
> in DT_GNU_FLAGS).
>
> One flag is currently defined: DF_GNU_1_UNIQUE.
>
> libc and its companion DSOs (libpthread et al) should have this
> section and flag set.
> ---
> elf/elf.h | 7 ++++++-
> elf/get-dynamic-info.h | 12 ++++++++++++
> include/elf.h | 2 ++
> include/link.h | 1 +
> 4 files changed, 21 insertions(+), 1 deletion(-)
>
> diff --git a/elf/elf.h b/elf/elf.h
> index 4f838d4af2..9f9b8e4291 100644
> --- a/elf/elf.h
> +++ b/elf/elf.h
> @@ -895,6 +895,7 @@ typedef struct
> Dyn.d_un.d_val field of the Elf*_Dyn structure. This follows Sun's
> approach. */
> #define DT_VALRNGLO 0x6ffffd00
> +#define DT_GNU_FLAGS_1 0x6ffffdf4 /* Open DSO once across all namespaces */
> #define DT_GNU_PRELINKED 0x6ffffdf5 /* Prelinking timestamp */
> #define DT_GNU_CONFLICTSZ 0x6ffffdf6 /* Size of conflict section */
> #define DT_GNU_LIBLISTSZ 0x6ffffdf7 /* Size of library list */
Ok, it matches binutils definition.
> @@ -909,7 +910,7 @@ typedef struct
> #define DT_SYMINENT 0x6ffffdff /* Entry size of syminfo */
> #define DT_VALRNGHI 0x6ffffdff
> #define DT_VALTAGIDX(tag) (DT_VALRNGHI - (tag)) /* Reverse order! */
> -#define DT_VALNUM 12
> +#define DT_VALNUM 13
>
Ok.
> /* DT_* entries which fall between DT_ADDRRNGHI & DT_ADDRRNGLO use the
> Dyn.d_un.d_ptr field of the Elf*_Dyn structure.
> @@ -998,6 +999,10 @@ typedef struct
> #define DF_1_WEAKFILTER 0x20000000
> #define DF_1_NOCOMMON 0x40000000
>
> +/* State flags selectable in the `d_un.d_val' element of the DT_GNU_FLAGS_1
> + entry in the dynamic section. */
> +#define DF_GNU_1_UNIQUE 0x00000001
> +
> /* Flags for the feature selection in DT_FEATURE_1. */
> #define DTF_1_PARINIT 0x00000001
> #define DTF_1_CONFEXP 0x00000002
Ok, it matches binutils definition.
> diff --git a/elf/get-dynamic-info.h b/elf/get-dynamic-info.h
> index d8ec32377d..d441b39576 100644
> --- a/elf/get-dynamic-info.h
> +++ b/elf/get-dynamic-info.h
> @@ -175,6 +175,18 @@ elf_get_dynamic_info (struct link_map *l, ElfW(Dyn) *temp)
> if (l->l_flags_1 & DF_1_NOW)
> info[DT_BIND_NOW] = info[VERSYMIDX (DT_FLAGS_1)];
> }
> + if (info[DT_VALTAGIDX (DT_GNU_FLAGS_1)
> + + DT_NUM + DT_THISPROCNUM + DT_VERSIONTAGNUM + DT_EXTRANUM] != NULL)
> + {
> + l->l_gnu_flags_1 = info[DT_VALTAGIDX (DT_GNU_FLAGS_1)
> + + DT_NUM + DT_THISPROCNUM
> + + DT_VERSIONTAGNUM + DT_EXTRANUM]->d_un.d_val;
> +
> + if (__builtin_expect (GLRO(dl_debug_mask) & DL_DEBUG_FILES, 0)
> + && l->l_gnu_flags_1 & ~DT_GNU_1_SUPPORTED_MASK)
Use __glibc_likely.
> + _dl_debug_printf ("\nWARNING: Unsupported flag value(s) of 0x%x in DT_GNU_FLAGS_1.\n",
> + l->l_gnu_flags_1 & ~DT_GNU_1_SUPPORTED_MASK);
> + }
> if (info[DT_RUNPATH] != NULL)
> /* If both RUNPATH and RPATH are given, the latter is ignored. */
> info[DT_RPATH] = NULL;
Ok.
> diff --git a/include/elf.h b/include/elf.h
> index 14ed67ff67..5eee37c294 100644
> --- a/include/elf.h
> +++ b/include/elf.h
> @@ -25,5 +25,7 @@
> (DF_1_NOW | DF_1_NODELETE | DF_1_INITFIRST | DF_1_NOOPEN \
> | DF_1_ORIGIN | DF_1_NODEFLIB | DF_1_PIE)
>
> +#define DT_GNU_1_SUPPORTED_MASK DF_GNU_1_UNIQUE
> +
> #endif /* !_ISOMAC */
> #endif /* elf.h */
Ok.
> diff --git a/include/link.h b/include/link.h
> index b51fa0cbdf..55e0cad71d 100644
> --- a/include/link.h
> +++ b/include/link.h
> @@ -285,6 +285,7 @@ struct link_map
> unsigned int l_used;
>
> /* Various flag words. */
> + ElfW(Word) l_gnu_flags_1;
> ElfW(Word) l_feature_1;
> ElfW(Word) l_flags_1;
> ElfW(Word) l_flags;
>
Ok.
next prev parent reply other threads:[~2021-02-15 18:42 UTC|newest]
Thread overview: 78+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-02-09 17:18 [RFC][PATCH v8 00/20] Implementation of RTLD_SHARED for dlmopen Vivek Das Mohapatra
2021-02-09 17:18 ` [RFC][PATCH v8 01/20] Declare and describe the dlmopen RTLD_SHARED flag Vivek Das Mohapatra
2021-02-15 13:08 ` Adhemerval Zanella
2021-02-15 13:29 ` Andreas Schwab
2021-02-15 13:31 ` Adhemerval Zanella
2021-02-09 17:18 ` [RFC][PATCH v8 02/20] include/link.h: Update the link_map struct to allow proxies Vivek Das Mohapatra
2021-02-15 13:11 ` Adhemerval Zanella
2021-02-09 17:18 ` [RFC][PATCH v8 03/20] elf/dl-object.c: Implement a helper function to proxy link_map entries Vivek Das Mohapatra
2021-02-15 13:30 ` Adhemerval Zanella
2021-02-09 17:18 ` [RFC][PATCH v8 04/20] elf/dl-load.c, elf-dl-open.c: Implement RTLD_SHARED dlmopen proxying Vivek Das Mohapatra
2021-02-15 14:53 ` Adhemerval Zanella
2021-02-09 17:18 ` [RFC][PATCH v8 05/20] elf/dl-fini.c: Handle proxy link_map entries in the shutdown path Vivek Das Mohapatra
2021-02-15 17:08 ` Adhemerval Zanella
2021-02-09 17:18 ` [RFC][PATCH v8 06/20] elf/dl-init.c: Skip proxied link map entries in the dl init path Vivek Das Mohapatra
2021-02-15 17:52 ` Adhemerval Zanella
2021-02-09 17:18 ` [RFC][PATCH v8 07/20] elf/dl-open.c: Don't try libc linit in namespaces with no libc mapping Vivek Das Mohapatra
2021-02-15 17:54 ` Adhemerval Zanella
2021-02-17 15:39 ` Vivek Das Mohapatra
2021-02-17 16:17 ` Adhemerval Zanella
2021-02-17 18:32 ` Vivek Das Mohapatra
2021-02-17 20:52 ` Adhemerval Zanella
2021-02-09 17:18 ` [RFC][PATCH v8 08/20] elf/dl-open.c: when creating a proxy check the libc_map in NS 0 Vivek Das Mohapatra
2021-02-15 17:55 ` Adhemerval Zanella
2021-02-09 17:18 ` [RFC][PATCH v8 09/20] Define a new dynamic section tag - DT_GNU_FLAGS_1 Vivek Das Mohapatra
2021-02-15 18:42 ` Adhemerval Zanella [this message]
2021-02-09 17:18 ` [RFC][PATCH v8 10/20] Abstract the loaded-DSO search code into a private helper function Vivek Das Mohapatra
2021-02-15 19:25 ` Adhemerval Zanella
2021-02-09 17:18 ` [RFC][PATCH v8 11/20] Compare loaded DSOs by file ID and check for DF_GNU_1_UNIQUE Vivek Das Mohapatra
2021-02-18 20:45 ` Adhemerval Zanella
2021-02-09 17:18 ` [RFC][PATCH v8 12/20] Use the new DSO finder helper function since we have it Vivek Das Mohapatra
2021-02-19 14:39 ` Adhemerval Zanella
2021-02-19 19:50 ` Adhemerval Zanella
2021-02-22 18:44 ` Vivek Das Mohapatra
2021-02-22 18:51 ` Adhemerval Zanella
2021-02-22 23:50 ` Vivek Das Mohapatra
2021-02-09 17:18 ` [RFC][PATCH v8 13/20] Use the DSO search helper to check for preloaded DT_GNU_UNIQUE DSOs Vivek Das Mohapatra
2021-02-19 17:26 ` Adhemerval Zanella
2021-02-09 17:18 ` [RFC][PATCH v8 14/20] When loading DSOs into alternate namespaces check for DT_GNU_UNIQUE Vivek Das Mohapatra
2021-02-19 18:11 ` Adhemerval Zanella
2021-02-09 17:18 ` [RFC][PATCH v8 15/20] Suppress audit calls when a (new) namespace is empty Vivek Das Mohapatra
2021-02-19 19:45 ` Adhemerval Zanella
2021-02-09 17:18 ` [RFC][PATCH v8 16/20] Suppress inter-namespace DSO sharing for audit libraries Vivek Das Mohapatra
2021-02-19 20:39 ` Adhemerval Zanella
2021-02-09 17:18 ` [RFC][PATCH v8 17/20] dlsym, dlvsym should be able to look up symbols via DSO proxies Vivek Das Mohapatra
2021-02-22 18:51 ` Adhemerval Zanella
2021-02-09 17:18 ` [RFC][PATCH v8 18/20] Add DT_GNU_FLAGS_1/DF_GNU_1_UNIQUE dynamic section+flag to glibc DSOs Vivek Das Mohapatra
2021-02-22 20:39 ` Adhemerval Zanella
2021-02-22 21:19 ` Florian Weimer
2021-02-22 21:53 ` Adhemerval Zanella
2021-02-23 3:26 ` Alan Modra
2021-02-23 11:57 ` Adhemerval Zanella
2021-02-23 10:00 ` Florian Weimer
2021-02-23 11:58 ` Adhemerval Zanella
2021-02-23 12:12 ` Florian Weimer
2021-02-23 0:17 ` Vivek Das Mohapatra
2021-02-23 12:02 ` Adhemerval Zanella
2021-02-23 13:24 ` Adhemerval Zanella
2021-02-23 18:46 ` Vivek Das Mohapatra
2021-02-23 18:48 ` Adhemerval Zanella
2021-02-23 19:01 ` Florian Weimer
2021-02-23 19:06 ` Adhemerval Zanella
2021-02-09 17:18 ` [RFC][PATCH v8 19/20] Add dlmopen / RTLD_SHARED tests Vivek Das Mohapatra
2021-02-26 19:50 ` Adhemerval Zanella
2021-02-26 20:26 ` Andreas Schwab
2021-02-09 17:18 ` [RFC][PATCH v8 20/20] Restore separate libc loading for the TLS/namespace storage test Vivek Das Mohapatra
2021-02-09 19:01 ` [RFC][PATCH v8 00/20] Implementation of RTLD_SHARED for dlmopen Joseph Myers
2021-02-10 12:25 ` Vivek Das Mohapatra
2021-02-10 18:32 ` Joseph Myers
2021-02-12 18:52 ` Adhemerval Zanella
2021-02-12 18:56 ` Florian Weimer
2021-02-12 19:01 ` Adhemerval Zanella
2021-02-12 19:29 ` Florian Weimer
2021-02-18 14:41 ` Vivek Das Mohapatra
2021-02-18 15:02 ` Florian Weimer
2021-02-18 15:05 ` Vivek Das Mohapatra
2021-02-19 8:57 ` Florian Weimer
2021-02-19 15:08 ` Vivek Das Mohapatra
2021-03-04 18:27 ` Adhemerval Zanella
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=37a13aab-54f6-c105-f914-3ed14aa58cef@linaro.org \
--to=adhemerval.zanella@linaro.org \
--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).