From patchwork Fri May 23 23:15:36 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Richard Henderson X-Patchwork-Id: 1121 Received: (qmail 20145 invoked by alias); 23 May 2014 23:15:50 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 20063 invoked by uid 89); 23 May 2014 23:15:48 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.2 required=5.0 tests=AWL, BAYES_00, FREEMAIL_ENVFROM_END_DIGIT, FREEMAIL_FROM, RCVD_IN_DNSWL_NONE, SPF_PASS autolearn=no version=3.3.2 X-HELO: mail-qg0-f46.google.com X-Received: by 10.140.91.5 with SMTP id y5mr11116537qgd.12.1400886945676; Fri, 23 May 2014 16:15:45 -0700 (PDT) From: Richard Henderson To: libc-alpha@sourceware.org Cc: roland@hack.frob.com Subject: [PATCH 2/2] Only provide non-default symbols in libpthread for vfork Date: Fri, 23 May 2014 16:15:36 -0700 Message-Id: <1400886936-16338-3-git-send-email-rth@twiddle.net> In-Reply-To: <1400886936-16338-1-git-send-email-rth@twiddle.net> References: <1400886936-16338-1-git-send-email-rth@twiddle.net> * nptl/pt-vfork.c (vfork_resolve): Rename from vfork_ifunc. (vfork_ifunc): Define via attribute ifunc. (DEFINE_VFORK): Remove. (vfork, __vfork): Define via compat_symbol. --- nptl/pt-vfork.c | 23 +++++++---------------- 1 file changed, 7 insertions(+), 16 deletions(-) diff --git a/nptl/pt-vfork.c b/nptl/pt-vfork.c index 8beb121..78c0781 100644 --- a/nptl/pt-vfork.c +++ b/nptl/pt-vfork.c @@ -42,30 +42,21 @@ extern __typeof (vfork) __libc_vfork; /* Defined in libc. */ -attribute_hidden __attribute__ ((used)) -__typeof (vfork) * -vfork_ifunc (void) +static __typeof (vfork) * +vfork_resolve (void) { return &__libc_vfork; } -# ifdef HAVE_ASM_SET_DIRECTIVE -# define DEFINE_VFORK(name) \ - asm (".set " #name ", vfork_ifunc\n" \ - ".globl " #name "\n" \ - ".type " #name ", %gnu_indirect_function"); -# else -# define DEFINE_VFORK(name) \ - asm (#name " = vfork_ifunc\n" \ - ".globl " #name "\n" \ - ".type " #name ", %gnu_indirect_function"); -# endif +void *vfork_ifunc(void *, const void *, size_t) + __attribute__ ((ifunc ("vfork_resolve"))); #endif #if SHLIB_COMPAT (libpthread, GLIBC_2_0, GLIBC_2_20) -DEFINE_VFORK (vfork) +compat_symbol (libpthread, vfork_ifunc, vfork, GLIBC_2_0); #endif #if SHLIB_COMPAT (libpthread, GLIBC_2_1_2, GLIBC_2_20) -DEFINE_VFORK (__vfork) +strong_alias (vfork_ifunc, vfork_ifunc2) +compat_symbol (libpthread, vfork_ifunc2, __vfork, GLIBC_2_1_2); #endif