This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] Replace __attribute__((visibility("protected")))
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Andreas Schwab <schwab at linux-m68k dot org>
- Cc: GNU C Library <libc-alpha at sourceware dot org>
- Date: Fri, 6 Mar 2015 13:11:45 -0800
- Subject: Re: [PATCH] Replace __attribute__((visibility("protected")))
- Authentication-results: sourceware.org; auth=none
- References: <20150306194153 dot GA30470 at intel dot com> <87pp8lnax0 dot fsf at igel dot home>
On Fri, Mar 6, 2015 at 12:11 PM, Andreas Schwab <schwab@linux-m68k.org> wrote:
> "H.J. Lu" <hongjiu.lu@intel.com> writes:
>
>> diff --git a/elf/ifuncdep2.c b/elf/ifuncdep2.c
>> index 99d1926..599eba7 100644
>> --- a/elf/ifuncdep2.c
>> +++ b/elf/ifuncdep2.c
>> @@ -2,7 +2,8 @@
>>
>> #include "ifunc-sel.h"
>>
>> -int global __attribute__ ((visibility ("protected"))) = -1;
>> +int global = -1;
>> +asm (".protected global");
>
> Some architectures use a symbol prefix.
>
Here is the updated patch to use __SYMBOL_PREFIX. OK
to install?
Thanks.
--
H.J.
From d7648b9bfd8b5dc1857249dc28fddde08a48629e Mon Sep 17 00:00:00 2001
From: "H.J. Lu" <hjl.tools@gmail.com>
Date: Fri, 6 Mar 2015 04:55:56 -0800
Subject: [PATCH 3/3] Replace __attribute__((visibility("protected")))
With copy relocation, address of protected data defined in the shared
library may be external. Compiler shouldn't asssume protected data will
be local. But due to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65248
__attribute__((visibility("protected"))) doesn't work correctly, we need
to use asm (".protected xxx") instead.
* elf/ifuncdep2.c (global): Replace
__attribute__((visibility("protected"))) with
asm (".protected " __SYMBOL_PREFIX "global").
* elf/ifuncmod1.c (global): Likewise.
* elf/ifuncmod5.c (global): Likewise.
---
elf/ifuncdep2.c | 3 ++-
elf/ifuncmod1.c | 3 ++-
elf/ifuncmod5.c | 3 ++-
3 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/elf/ifuncdep2.c b/elf/ifuncdep2.c
index 99d1926..4f3a773 100644
--- a/elf/ifuncdep2.c
+++ b/elf/ifuncdep2.c
@@ -2,7 +2,8 @@
#include "ifunc-sel.h"
-int global __attribute__ ((visibility ("protected"))) = -1;
+int global = -1;
+asm (".protected " __SYMBOL_PREFIX "global");
static int
one (void)
diff --git a/elf/ifuncmod1.c b/elf/ifuncmod1.c
index 2b8195c..fc10b35 100644
--- a/elf/ifuncmod1.c
+++ b/elf/ifuncmod1.c
@@ -6,7 +6,8 @@
*/
#include "ifunc-sel.h"
-int global __attribute__ ((visibility ("protected"))) = -1;
+int global = -1;
+asm (".protected " __SYMBOL_PREFIX "global");
static int
one (void)
diff --git a/elf/ifuncmod5.c b/elf/ifuncmod5.c
index 9a08e8c..1d3529f 100644
--- a/elf/ifuncmod5.c
+++ b/elf/ifuncmod5.c
@@ -1,7 +1,8 @@
/* Test STT_GNU_IFUNC symbols without direct function call. */
#include "ifunc-sel.h"
-int global __attribute__ ((visibility ("protected"))) = -1;
+int global = -1;
+asm (".protected " __SYMBOL_PREFIX "global");
static int
one (void)
--
1.9.3