diff mbox series

x86: Add x86_64 to x86-64 HWCAP [BZ #22093]

Message ID 20170906180229.GA32425@gmail.com
State New
Headers show
Series x86: Add x86_64 to x86-64 HWCAP [BZ #22093] | expand

Commit Message

H.J. Lu Sept. 6, 2017, 6:02 p.m. UTC
Before glibc 2.26, ld.so set dl_platform to "x86_64" and searched the
"x86_64" subdirectory when loading a shared library.  ld.so in glibc
2.26 was changed to set dl_platform to "haswell" or "xeon_phi", based
on supported ISAs.  This led to shared library loading failure for
shared libraries placed under the "x86_64" subdirectory.

This patch adds "x86_64" to x86-64 dl_hwcap so that ld.so will always
search the "x86_64" subdirectory when loading a shared library.

NB: We can't set x86-64 dl_platform to "x86-64" since ld.so will skip
the "haswell" and "xeon_phi" subdirectories on "haswell" and "xeon_phi"
machines.

Tested on i686 and x86-64.  OK for master?


H.J.
--
	[BZ #22093]
	* sysdeps/x86/cpu-features.c (init_cpu_features): Initialize
	GLRO(dl_hwcap) to HWCAP_X86_64 for x86-64.
	* sysdeps/x86/dl-hwcap.h (HWCAP_COUNT): Updated.
	(HWCAP_IMPORTANT): Likewise.
	(HWCAP_X86_64): New enum.
	(HWCAP_X86_AVX512_1): Updated.
	* sysdeps/x86/dl-procinfo.c (_dl_x86_hwcap_flags): Add "x86_64".
	* sysdeps/x86_64/Makefile (tests): Add tst-x86_64-1.
	(modules-names): Add x86_64/tst-x86_64mod-1.
	(LDFLAGS-tst-x86_64mod-1.so): New.
	($(objpfx)tst-x86_64-1): Likewise.
	($(objpfx)x86_64/tst-x86_64mod-1.os): Likewise.
	(tst-x86_64-1-clean): Likewise.
	* sysdeps/x86_64/tst-x86_64-1.c: New file.
	* sysdeps/x86_64/tst-x86_64mod-1.c: Likewise.
---
 sysdeps/x86/cpu-features.c       |  3 ++-
 sysdeps/x86/dl-hwcap.h           | 12 +++++++-----
 sysdeps/x86/dl-procinfo.c        |  4 ++--
 sysdeps/x86_64/Makefile          | 17 +++++++++++++++++
 sysdeps/x86_64/tst-x86_64-1.c    | 26 ++++++++++++++++++++++++++
 sysdeps/x86_64/tst-x86_64mod-1.c | 22 ++++++++++++++++++++++
 6 files changed, 76 insertions(+), 8 deletions(-)
 create mode 100644 sysdeps/x86_64/tst-x86_64-1.c
 create mode 100644 sysdeps/x86_64/tst-x86_64mod-1.c

Comments

Andreas Schwab Sept. 11, 2017, 9:32 a.m. UTC | #1
On Sep 06 2017, "H.J. Lu" <hongjiu.lu@intel.com> wrote:

> 	[BZ #22093]
> 	* sysdeps/x86/cpu-features.c (init_cpu_features): Initialize
> 	GLRO(dl_hwcap) to HWCAP_X86_64 for x86-64.
> 	* sysdeps/x86/dl-hwcap.h (HWCAP_COUNT): Updated.
> 	(HWCAP_IMPORTANT): Likewise.
> 	(HWCAP_X86_64): New enum.
> 	(HWCAP_X86_AVX512_1): Updated.
> 	* sysdeps/x86/dl-procinfo.c (_dl_x86_hwcap_flags): Add "x86_64".
> 	* sysdeps/x86_64/Makefile (tests): Add tst-x86_64-1.
> 	(modules-names): Add x86_64/tst-x86_64mod-1.
> 	(LDFLAGS-tst-x86_64mod-1.so): New.
> 	($(objpfx)tst-x86_64-1): Likewise.
> 	($(objpfx)x86_64/tst-x86_64mod-1.os): Likewise.
> 	(tst-x86_64-1-clean): Likewise.
> 	* sysdeps/x86_64/tst-x86_64-1.c: New file.
> 	* sysdeps/x86_64/tst-x86_64mod-1.c: Likewise.

Ok.

Andreas.
H.J. Lu Oct. 19, 2017, 9:53 p.m. UTC | #2
On Mon, Sep 11, 2017 at 2:32 AM, Andreas Schwab <schwab@suse.de> wrote:
> On Sep 06 2017, "H.J. Lu" <hongjiu.lu@intel.com> wrote:
>
>>       [BZ #22093]
>>       * sysdeps/x86/cpu-features.c (init_cpu_features): Initialize
>>       GLRO(dl_hwcap) to HWCAP_X86_64 for x86-64.
>>       * sysdeps/x86/dl-hwcap.h (HWCAP_COUNT): Updated.
>>       (HWCAP_IMPORTANT): Likewise.
>>       (HWCAP_X86_64): New enum.
>>       (HWCAP_X86_AVX512_1): Updated.
>>       * sysdeps/x86/dl-procinfo.c (_dl_x86_hwcap_flags): Add "x86_64".
>>       * sysdeps/x86_64/Makefile (tests): Add tst-x86_64-1.
>>       (modules-names): Add x86_64/tst-x86_64mod-1.
>>       (LDFLAGS-tst-x86_64mod-1.so): New.
>>       ($(objpfx)tst-x86_64-1): Likewise.
>>       ($(objpfx)x86_64/tst-x86_64mod-1.os): Likewise.
>>       (tst-x86_64-1-clean): Likewise.
>>       * sysdeps/x86_64/tst-x86_64-1.c: New file.
>>       * sysdeps/x86_64/tst-x86_64mod-1.c: Likewise.
>
> Ok.
>
> Andreas.
>

I'd like to backport it to 2.26 branch.  Any objections?
H.J. Lu Oct. 22, 2017, 11:10 a.m. UTC | #3
On Thu, Oct 19, 2017 at 2:53 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Mon, Sep 11, 2017 at 2:32 AM, Andreas Schwab <schwab@suse.de> wrote:
>> On Sep 06 2017, "H.J. Lu" <hongjiu.lu@intel.com> wrote:
>>
>>>       [BZ #22093]
>>>       * sysdeps/x86/cpu-features.c (init_cpu_features): Initialize
>>>       GLRO(dl_hwcap) to HWCAP_X86_64 for x86-64.
>>>       * sysdeps/x86/dl-hwcap.h (HWCAP_COUNT): Updated.
>>>       (HWCAP_IMPORTANT): Likewise.
>>>       (HWCAP_X86_64): New enum.
>>>       (HWCAP_X86_AVX512_1): Updated.
>>>       * sysdeps/x86/dl-procinfo.c (_dl_x86_hwcap_flags): Add "x86_64".
>>>       * sysdeps/x86_64/Makefile (tests): Add tst-x86_64-1.
>>>       (modules-names): Add x86_64/tst-x86_64mod-1.
>>>       (LDFLAGS-tst-x86_64mod-1.so): New.
>>>       ($(objpfx)tst-x86_64-1): Likewise.
>>>       ($(objpfx)x86_64/tst-x86_64mod-1.os): Likewise.
>>>       (tst-x86_64-1-clean): Likewise.
>>>       * sysdeps/x86_64/tst-x86_64-1.c: New file.
>>>       * sysdeps/x86_64/tst-x86_64mod-1.c: Likewise.
>>
>> Ok.
>>
>> Andreas.
>>
>
> I'd like to backport it to 2.26 branch.  Any objections?
>

I am checking it in.
diff mbox series

Patch

diff --git a/sysdeps/x86/cpu-features.c b/sysdeps/x86/cpu-features.c
index 6f900840d4..c267f17b76 100644
--- a/sysdeps/x86/cpu-features.c
+++ b/sysdeps/x86/cpu-features.c
@@ -336,7 +336,6 @@  no_cpuid:
 
   /* Reuse dl_platform, dl_hwcap and dl_hwcap_mask for x86.  */
   GLRO(dl_platform) = NULL;
-  GLRO(dl_hwcap) = 0;
 #if !HAVE_TUNABLES && defined SHARED
   /* The glibc.tune.hwcap_mask tunable is initialized already, so no need to do
      this.  */
@@ -344,6 +343,7 @@  no_cpuid:
 #endif
 
 #ifdef __x86_64__
+  GLRO(dl_hwcap) = HWCAP_X86_64;
   if (cpu_features->kind == arch_kind_intel)
     {
       if (CPU_FEATURES_ARCH_P (cpu_features, AVX512F_Usable)
@@ -374,6 +374,7 @@  no_cpuid:
 	GLRO(dl_platform) = "haswell";
     }
 #else
+  GLRO(dl_hwcap) = 0;
   if (CPU_FEATURES_CPU_P (cpu_features, SSE2))
     GLRO(dl_hwcap) |= HWCAP_X86_SSE2;
 
diff --git a/sysdeps/x86/dl-hwcap.h b/sysdeps/x86/dl-hwcap.h
index c95668415a..38627e9eef 100644
--- a/sysdeps/x86/dl-hwcap.h
+++ b/sysdeps/x86/dl-hwcap.h
@@ -24,15 +24,16 @@ 
 # define HWCAP_PLATFORMS_START	0
 # define HWCAP_PLATFORMS_COUNT	4
 # define HWCAP_START		0
-# define HWCAP_COUNT		2
-# define HWCAP_IMPORTANT	(HWCAP_X86_SSE2 | HWCAP_X86_AVX512_1)
+# define HWCAP_COUNT		3
+# define HWCAP_IMPORTANT \
+  (HWCAP_X86_SSE2 | HWCAP_X86_64 | HWCAP_X86_AVX512_1)
 #elif defined __x86_64__
 /* For 64 bit, only cover x86-64 platforms and capabilities.  */
 # define HWCAP_PLATFORMS_START	2
 # define HWCAP_PLATFORMS_COUNT	4
 # define HWCAP_START		1
-# define HWCAP_COUNT		2
-# define HWCAP_IMPORTANT	(HWCAP_X86_AVX512_1)
+# define HWCAP_COUNT		3
+# define HWCAP_IMPORTANT	(HWCAP_X86_64 | HWCAP_X86_AVX512_1)
 #else
 /* For 32 bit, only cover i586, i686 and SSE2.  */
 # define HWCAP_PLATFORMS_START	0
@@ -45,7 +46,8 @@ 
 enum
 {
   HWCAP_X86_SSE2		= 1 << 0,
-  HWCAP_X86_AVX512_1		= 1 << 1
+  HWCAP_X86_64			= 1 << 1,
+  HWCAP_X86_AVX512_1		= 1 << 2
 };
 
 static inline const char *
diff --git a/sysdeps/x86/dl-procinfo.c b/sysdeps/x86/dl-procinfo.c
index 43ab8fe25b..0192feb850 100644
--- a/sysdeps/x86/dl-procinfo.c
+++ b/sysdeps/x86/dl-procinfo.c
@@ -58,11 +58,11 @@  PROCINFO_CLASS struct cpu_features _dl_x86_cpu_features
 #if !defined PROCINFO_DECL && defined SHARED
   ._dl_x86_hwcap_flags
 #else
-PROCINFO_CLASS const char _dl_x86_hwcap_flags[2][9]
+PROCINFO_CLASS const char _dl_x86_hwcap_flags[3][9]
 #endif
 #ifndef PROCINFO_DECL
 = {
-    "sse2", "avx512_1"
+    "sse2", "x86_64", "avx512_1"
   }
 #endif
 #if !defined SHARED || defined PROCINFO_DECL
diff --git a/sysdeps/x86_64/Makefile b/sysdeps/x86_64/Makefile
index 7f3ffe34ba..1514805f4a 100644
--- a/sysdeps/x86_64/Makefile
+++ b/sysdeps/x86_64/Makefile
@@ -52,6 +52,12 @@  $(objpfx)tst-quad2pie: $(objpfx)tst-quadmod2pie.o
 CFLAGS-tst-quad1pie.c = $(PIE-ccflag)
 CFLAGS-tst-quad2pie.c = $(PIE-ccflag)
 
+tests += tst-x86_64-1
+modules-names += x86_64/tst-x86_64mod-1
+LDFLAGS-tst-x86_64mod-1.so = -Wl,-soname,tst-x86_64mod-1.so
+
+$(objpfx)tst-x86_64-1: $(objpfx)x86_64/tst-x86_64mod-1.so
+
 tests += tst-audit3 tst-audit4 tst-audit5 tst-audit6 tst-audit7 \
 	 tst-audit10 tst-sse tst-avx tst-avx512
 test-extras += tst-audit4-aux tst-audit10-aux \
@@ -124,3 +130,14 @@  endif
 ifeq ($(subdir),csu)
 gen-as-const-headers += tlsdesc.sym rtld-offsets.sym
 endif
+
+$(objpfx)x86_64/tst-x86_64mod-1.os: $(objpfx)tst-x86_64mod-1.os
+	$(make-target-directory)
+	rm -f $@
+	ln $< $@
+
+do-tests-clean common-mostlyclean: tst-x86_64-1-clean
+
+.PHONY: tst-x86_64-1-clean
+tst-x86_64-1-clean:
+	-rm -rf $(objpfx)x86_64
diff --git a/sysdeps/x86_64/tst-x86_64-1.c b/sysdeps/x86_64/tst-x86_64-1.c
new file mode 100644
index 0000000000..ba1a55cdaf
--- /dev/null
+++ b/sysdeps/x86_64/tst-x86_64-1.c
@@ -0,0 +1,26 @@ 
+/* Test searching the "x86_64" directory for shared libraries.
+   Copyright (C) 2017 Free Software Foundation, Inc.
+   This file is part of the GNU C Library.
+
+   The GNU C Library is free software; you can redistribute it and/or
+   modify it under the terms of the GNU Lesser General Public
+   License as published by the Free Software Foundation; either
+   version 2.1 of the License, or (at your option) any later version.
+
+   The GNU C Library is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+   Lesser General Public License for more details.
+
+   You should have received a copy of the GNU Lesser General Public
+   License along with the GNU C Library; if not, see
+   <http://www.gnu.org/licenses/>.  */
+
+extern void foo (void);
+
+int
+main (void)
+{
+  foo ();
+  return 0;
+}
diff --git a/sysdeps/x86_64/tst-x86_64mod-1.c b/sysdeps/x86_64/tst-x86_64mod-1.c
new file mode 100644
index 0000000000..83dfafb5a8
--- /dev/null
+++ b/sysdeps/x86_64/tst-x86_64mod-1.c
@@ -0,0 +1,22 @@ 
+/* Test searching the "x86_64" directory for shared libraries.
+   Copyright (C) 2017 Free Software Foundation, Inc.
+   This file is part of the GNU C Library.
+
+   The GNU C Library is free software; you can redistribute it and/or
+   modify it under the terms of the GNU Lesser General Public
+   License as published by the Free Software Foundation; either
+   version 2.1 of the License, or (at your option) any later version.
+
+   The GNU C Library is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+   Lesser General Public License for more details.
+
+   You should have received a copy of the GNU Lesser General Public
+   License along with the GNU C Library; if not, see
+   <http://www.gnu.org/licenses/>.  */
+
+void
+foo (void)
+{
+}