From 9b5a06c242e640d4af84a031e96ca47bcf595caf Mon Sep 17 00:00:00 2001 From: "H.J. Lu" Date: Thu, 3 Jan 2013 15:02:33 -0800 Subject: [PATCH] Make __get_cpu_features hidden Add static-init-arch.oS to libc_nonshared.a so that __get_cpu_features can be used in other shared libraries. --- ChangeLog.bz14955 | 25 +++++++++++++++++++ include/libc-symbols.h | 12 +-------- sysdeps/i386/i686/multiarch/Makefile | 2 ++ sysdeps/i386/i686/multiarch/Versions | 5 ---- .../i386/i686/multiarch/static-init-arch.c | 1 + sysdeps/x86/Makefile | 9 +++++++ sysdeps/x86/tst-ifunc-cpu1-main.c | 10 ++++++++ sysdeps/x86/tst-ifunc-cpu1-mod.c | 24 ++++++++++++++++++ sysdeps/x86_64/multiarch/Makefile | 2 ++ sysdeps/x86_64/multiarch/Versions | 5 ---- sysdeps/x86_64/multiarch/init-arch.c | 14 +++-------- sysdeps/x86_64/multiarch/init-arch.h | 25 +++++++++---------- sysdeps/x86_64/multiarch/static-init-arch.c | 3 +++ 13 files changed, 92 insertions(+), 45 deletions(-) create mode 100644 ChangeLog.bz14955 delete mode 100644 sysdeps/i386/i686/multiarch/Versions create mode 100644 sysdeps/i386/i686/multiarch/static-init-arch.c create mode 100644 sysdeps/x86/tst-ifunc-cpu1-main.c create mode 100644 sysdeps/x86/tst-ifunc-cpu1-mod.c delete mode 100644 sysdeps/x86_64/multiarch/Versions create mode 100644 sysdeps/x86_64/multiarch/static-init-arch.c diff --git a/ChangeLog.bz14955 b/ChangeLog.bz14955 new file mode 100644 index 0000000000..81cdb3b7f8 --- /dev/null +++ b/ChangeLog.bz14955 @@ -0,0 +1,25 @@ +2013-01-03 H.J. Lu + + [BZ #14955] + * include/libc-symbols.h (libc_ifunc): Remove INIT_ARCH. + (libm_ifunc): Defined to libc_ifunc. + * sysdeps/i386/i686/multiarch/Makefile (sysdep_routines): Add + static-init-arch. + (static-only-routines): Likewise. + * sysdeps/x86_64/multiarch/Makefile (sysdep_routines): Likewise. + (static-only-routines): Likewise. + * sysdeps/i386/i686/multiarch/Versions: Removed. + * sysdeps/x86_64/multiarch/Versions: Likewise. + * sysdeps/i386/i686/multiarch/static-init-arch.c: New file. + * sysdeps/x86_64/multiarch/static-init-arch.c: Likewise. + * sysdeps/x86/Makefile (tests): Add tst-ifunc-cpu1-main. + (modules-names): Add tst-ifunc-cpu1-mod. + (tst-ifunc-cpu1-mod.so-no-z-defs): New macro. + ($(objpfx)tst-ifunc-cpu1-main): New dependency. + * sysdeps/x86/tst-ifunc-cpu1-main.c: New file. + * sysdeps/x86/tst-ifunc-cpu1-mod.c: Likewise. + * sysdeps/x86_64/multiarch/init-arch.c (__always_inline): Defined + to empty. + (__get_cpu_features): Removed. + * sysdeps/x86_64/multiarch/init-arch.h (INIT_ARCH): Removed. + (__get_cpu_features): Defined as an inline function. diff --git a/include/libc-symbols.h b/include/libc-symbols.h index 0608320afc..3e4c5abf06 100644 --- a/include/libc-symbols.h +++ b/include/libc-symbols.h @@ -751,22 +751,12 @@ for linking") extern void *name##_ifunc (void) __asm__ (#name); \ void *name##_ifunc (void) \ { \ - INIT_ARCH (); \ __typeof (name) *res = expr; \ return res; \ } \ __asm__ (".type " #name ", %gnu_indirect_function"); -/* The body of the function is supposed to use __get_cpu_features - which will, if necessary, initialize the data first. */ -#define libm_ifunc(name, expr) \ - extern void *name##_ifunc (void) __asm__ (#name); \ - void *name##_ifunc (void) \ - { \ - __typeof (name) *res = expr; \ - return res; \ - } \ - __asm__ (".type " #name ", %gnu_indirect_function"); +#define libm_ifunc(name, expr) libc_ifunc (name, (expr)) #ifdef HAVE_ASM_SET_DIRECTIVE # define libc_ifunc_hidden_def1(local, name) \ diff --git a/sysdeps/i386/i686/multiarch/Makefile b/sysdeps/i386/i686/multiarch/Makefile index 8946bfa586..e3c97245b1 100644 --- a/sysdeps/i386/i686/multiarch/Makefile +++ b/sysdeps/i386/i686/multiarch/Makefile @@ -1,5 +1,7 @@ ifeq ($(subdir),csu) aux += init-arch +sysdep_routines += static-init-arch +static-only-routines += static-init-arch tests += test-multiarch gen-as-const-headers += ifunc-defines.sym endif diff --git a/sysdeps/i386/i686/multiarch/Versions b/sysdeps/i386/i686/multiarch/Versions deleted file mode 100644 index 59b185ac8d..0000000000 --- a/sysdeps/i386/i686/multiarch/Versions +++ /dev/null @@ -1,5 +0,0 @@ -libc { - GLIBC_PRIVATE { - __get_cpu_features; - } -} diff --git a/sysdeps/i386/i686/multiarch/static-init-arch.c b/sysdeps/i386/i686/multiarch/static-init-arch.c new file mode 100644 index 0000000000..6d8100d0a9 --- /dev/null +++ b/sysdeps/i386/i686/multiarch/static-init-arch.c @@ -0,0 +1 @@ +#include diff --git a/sysdeps/x86/Makefile b/sysdeps/x86/Makefile index 3e87a77069..79a44061e9 100644 --- a/sysdeps/x86/Makefile +++ b/sysdeps/x86/Makefile @@ -6,4 +6,13 @@ tests: $(objpfx)tst-xmmymm.out $(objpfx)tst-xmmymm.out: ../sysdeps/x86/tst-xmmymm.sh $(objpfx)ld.so @echo "Checking ld.so for SSE register use. This will take a few seconds..." $(SHELL) $< $(objpfx) '$(NM)' '$(OBJDUMP)' '$(READELF)' > $@ + +ifeq (yesyes,$(build-shared)$(multi-arch)) +tests += tst-ifunc-cpu1-main +modules-names += tst-ifunc-cpu1-mod + +tst-ifunc-cpu1-mod.so-no-z-defs = yes + +$(objpfx)tst-ifunc-cpu1-main: $(objpfx)tst-ifunc-cpu1-mod.so +endif endif diff --git a/sysdeps/x86/tst-ifunc-cpu1-main.c b/sysdeps/x86/tst-ifunc-cpu1-main.c new file mode 100644 index 0000000000..37c16e2e1e --- /dev/null +++ b/sysdeps/x86/tst-ifunc-cpu1-main.c @@ -0,0 +1,10 @@ +/* Test function pointer to local STT_GNU_IFUNC function. */ + +extern void (*foo_ptr) (void); + +int +main (void) +{ + foo_ptr (); + return 0; +} diff --git a/sysdeps/x86/tst-ifunc-cpu1-mod.c b/sysdeps/x86/tst-ifunc-cpu1-mod.c new file mode 100644 index 0000000000..08ff30f41e --- /dev/null +++ b/sysdeps/x86/tst-ifunc-cpu1-mod.c @@ -0,0 +1,24 @@ +#include + +static void +one (void) +{ +} + +static void +two (void) +{ +} + +void * foo_ifunc (void) __asm__ ("foo") attribute_hidden; +__asm__(".type foo, %gnu_indirect_function"); + +void * +foo_ifunc (void) +{ + const struct cpu_features * cpu = __get_cpu_features (); + return cpu->max_cpuid > 1 ? two : one; +} + +extern void foo (void) attribute_hidden; +void (*foo_ptr) (void) = foo; diff --git a/sysdeps/x86_64/multiarch/Makefile b/sysdeps/x86_64/multiarch/Makefile index dd6c27d0b4..5a91a395e2 100644 --- a/sysdeps/x86_64/multiarch/Makefile +++ b/sysdeps/x86_64/multiarch/Makefile @@ -1,5 +1,7 @@ ifeq ($(subdir),csu) aux += init-arch +sysdep_routines += static-init-arch +static-only-routines += static-init-arch tests += test-multiarch gen-as-const-headers += ifunc-defines.sym endif diff --git a/sysdeps/x86_64/multiarch/Versions b/sysdeps/x86_64/multiarch/Versions deleted file mode 100644 index 59b185ac8d..0000000000 --- a/sysdeps/x86_64/multiarch/Versions +++ /dev/null @@ -1,5 +0,0 @@ -libc { - GLIBC_PRIVATE { - __get_cpu_features; - } -} diff --git a/sysdeps/x86_64/multiarch/init-arch.c b/sysdeps/x86_64/multiarch/init-arch.c index 992cbfb754..9ee968e2ab 100644 --- a/sysdeps/x86_64/multiarch/init-arch.c +++ b/sysdeps/x86_64/multiarch/init-arch.c @@ -19,6 +19,9 @@ #include #include +/* Define __get_cpu_features. */ +#undef __always_inline +#define __always_inline #include "init-arch.h" @@ -177,14 +180,3 @@ __init_cpu_features (void) atomic_write_barrier (); __cpu_features.kind = kind; } - -#undef __get_cpu_features - -const struct cpu_features * -__get_cpu_features (void) -{ - if (__cpu_features.kind == arch_kind_unknown) - __init_cpu_features (); - - return &__cpu_features; -} diff --git a/sysdeps/x86_64/multiarch/init-arch.h b/sysdeps/x86_64/multiarch/init-arch.h index 0aece18de3..cba10cf3f1 100644 --- a/sysdeps/x86_64/multiarch/init-arch.h +++ b/sysdeps/x86_64/multiarch/init-arch.h @@ -109,19 +109,18 @@ extern struct cpu_features extern void __init_cpu_features (void) attribute_hidden; -# define INIT_ARCH() \ - do \ - if (__cpu_features.kind == arch_kind_unknown) \ - __init_cpu_features (); \ - while (0) - -/* Used from outside libc.so to get access to the CPU features structure. */ -extern const struct cpu_features *__get_cpu_features (void) - __attribute__ ((const)); - -# ifndef NOT_IN_libc -# define __get_cpu_features() (&__cpu_features) -# endif + +/* Implicitly call __init_cpu_features before accessing the CPU features + structure. */ + +extern __always_inline attribute_hidden +const struct cpu_features * +__get_cpu_features (void) +{ + if (__cpu_features.kind == arch_kind_unknown) + __init_cpu_features (); + return &__cpu_features; +} # define HAS_CPU_FEATURE(idx, reg, bit) \ ((__get_cpu_features ()->cpuid[idx].reg & (bit)) != 0) diff --git a/sysdeps/x86_64/multiarch/static-init-arch.c b/sysdeps/x86_64/multiarch/static-init-arch.c new file mode 100644 index 0000000000..a82d978218 --- /dev/null +++ b/sysdeps/x86_64/multiarch/static-init-arch.c @@ -0,0 +1,3 @@ +#ifdef LIBC_NONSHARED +# include "init-arch.c" +#endif -- 2.43.5