x86: Set Prefer_No_VZEROUPPER if AVX512ER is available

AVX512ER won't be implemented in any Xeon processors and will be in
all Xeon Phi processors.  Don't check CPU model number when setting
Prefer_No_VZEROUPPER for Xeon Phi.  Instead, set Prefer_No_VZEROUPPER
if AVX512ER is available.  It works with current and future Xeon Phi
and non-Xeon Phi processors.

	* sysdeps/x86/cpu-features.c (init_cpu_features): Set
	Prefer_No_VZEROUPPER if AVX512ER is available.
	* sysdeps/x86/cpu-features.h
	(bit_cpu_AVX512PF): New.
	(bit_cpu_AVX512ER): Likewise.
	(bit_cpu_AVX512CD): Likewise.
	(bit_cpu_AVX512BW): Likewise.
	(bit_cpu_AVX512VL): Likewise.
	(index_cpu_AVX512PF): Likewise.
	(index_cpu_AVX512ER): Likewise.
	(index_cpu_AVX512CD): Likewise.
	(index_cpu_AVX512BW): Likewise.
	(index_cpu_AVX512VL): Likewise.
	(reg_AVX512PF): Likewise.
	(reg_AVX512ER): Likewise.
	(reg_AVX512CD): Likewise.
	(reg_AVX512BW): Likewise.
	(reg_AVX512VL): Likewise.

diff --git a/ChangeLog b/ChangeLog
index ce6f13d17963c57f0ce2b7335cc68f280c0e2740..51ae9c18b23646b141007a67a8452f2ac22c349a 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,24 @@
+2017-04-18  H.J. Lu  <hongjiu.lu@intel.com>
+
+       * sysdeps/x86/cpu-features.c (init_cpu_features): Set
+       Prefer_No_VZEROUPPER if AVX512ER is available.
+       * sysdeps/x86/cpu-features.h
+       (bit_cpu_AVX512PF): New.
+       (bit_cpu_AVX512ER): Likewise.
+       (bit_cpu_AVX512CD): Likewise.
+       (bit_cpu_AVX512BW): Likewise.
+       (bit_cpu_AVX512VL): Likewise.
+       (index_cpu_AVX512PF): Likewise.
+       (index_cpu_AVX512ER): Likewise.
+       (index_cpu_AVX512CD): Likewise.
+       (index_cpu_AVX512BW): Likewise.
+       (index_cpu_AVX512VL): Likewise.
+       (reg_AVX512PF): Likewise.
+       (reg_AVX512ER): Likewise.
+       (reg_AVX512CD): Likewise.
+       (reg_AVX512BW): Likewise.
+       (reg_AVX512VL): Likewise.
+
 2017-04-18  Florian Weimer  <fweimer@redhat.com>
 
        * elf/dl-misc.c (_dl_sysdep_read_whole_file): Assume that

diff --git a/sysdeps/x86/cpu-features.c b/sysdeps/x86/cpu-features.c
index 33788ed32373bb26074382a76eaa78087667678f..ae7f844f8b6b593987c82d2b7d2a8d1f2d1555d1 100644 (file)
--- a/sysdeps/x86/cpu-features.c
+++ b/sysdeps/x86/cpu-features.c
@@ -138,8 +138,6 @@ init_cpu_features (struct cpu_features *cpu_features)
 
            case 0x57:
              /* Knights Landing.  Enable Silvermont optimizations.  */
-             cpu_features->feature[index_arch_Prefer_No_VZEROUPPER]
-               |= bit_arch_Prefer_No_VZEROUPPER;
 
            case 0x5c:
            case 0x5f:
@@ -225,6 +223,12 @@ init_cpu_features (struct cpu_features *cpu_features)
        cpu_features->feature[index_arch_AVX_Fast_Unaligned_Load]
          |= bit_arch_AVX_Fast_Unaligned_Load;
 
+      /* Since AVX512ER is unique to Xeon Phi, set Prefer_No_VZEROUPPER
+         if AVX512ER is available.  */
+      if (CPU_FEATURES_CPU_P (cpu_features, AVX512ER))
+       cpu_features->feature[index_arch_Prefer_No_VZEROUPPER]
+         |= bit_arch_Prefer_No_VZEROUPPER;
+
       /* To avoid SSE transition penalty, use _dl_runtime_resolve_slow.
          If XGETBV suports ECX == 1, use _dl_runtime_resolve_opt.  */
       cpu_features->feature[index_arch_Use_dl_runtime_resolve_slow]

diff --git a/sysdeps/x86/cpu-features.h b/sysdeps/x86/cpu-features.h
index 8ec1562fe7f9b701a590b37bb627734af981c849..1583d653457feb9a235b86fe82e9302a01d0bc7e 100644 (file)
--- a/sysdeps/x86/cpu-features.h
+++ b/sysdeps/x86/cpu-features.h
@@ -63,6 +63,11 @@
 #define bit_cpu_AVX2           (1 << 5)
 #define bit_cpu_AVX512F                (1 << 16)
 #define bit_cpu_AVX512DQ       (1 << 17)
+#define bit_cpu_AVX512PF       (1 << 26)
+#define bit_cpu_AVX512ER       (1 << 27)
+#define bit_cpu_AVX512CD       (1 << 28)
+#define bit_cpu_AVX512BW       (1 << 30)
+#define bit_cpu_AVX512VL       (1u << 31)
 
 /* XCR0 Feature flags.  */
 #define bit_XMM_state          (1 << 1)
@@ -239,6 +244,11 @@ extern const struct cpu_features *__get_cpu_features (void)
 # define index_cpu_AVX2                COMMON_CPUID_INDEX_7
 # define index_cpu_AVX512F     COMMON_CPUID_INDEX_7
 # define index_cpu_AVX512DQ    COMMON_CPUID_INDEX_7
+# define index_cpu_AVX512PF    COMMON_CPUID_INDEX_7
+# define index_cpu_AVX512ER    COMMON_CPUID_INDEX_7
+# define index_cpu_AVX512CD    COMMON_CPUID_INDEX_7
+# define index_cpu_AVX512BW    COMMON_CPUID_INDEX_7
+# define index_cpu_AVX512VL    COMMON_CPUID_INDEX_7
 # define index_cpu_ERMS                COMMON_CPUID_INDEX_7
 # define index_cpu_RTM         COMMON_CPUID_INDEX_7
 # define index_cpu_FMA         COMMON_CPUID_INDEX_1
@@ -258,6 +268,11 @@ extern const struct cpu_features *__get_cpu_features (void)
 # define reg_AVX2              ebx
 # define reg_AVX512F           ebx
 # define reg_AVX512DQ          ebx
+# define reg_AVX512PF          ebx
+# define reg_AVX512ER          ebx
+# define reg_AVX512CD          ebx
+# define reg_AVX512BW          ebx
+# define reg_AVX512VL          ebx
 # define reg_ERMS              ebx
 # define reg_RTM               ebx
 # define reg_FMA               ecx