Remove special L2 cache case for Knights Landing
authorH.J. Lu <hjl.tools@gmail.com>
Fri, 20 May 2016 21:41:14 +0000 (14:41 -0700)
committerH.J. Lu <hjl.tools@gmail.com>
Fri, 20 May 2016 21:42:00 +0000 (14:42 -0700)
L2 cache is shared by 2 cores on Knights Landing, which has 4 threads
per core:

https://en.wikipedia.org/wiki/Xeon_Phi#Knights_Landing

So L2 cache is shared by 8 threads on Knights Landing as reported by
CPUID.  We should remove special L2 cache case for Knights Landing.

[BZ #18185]
* sysdeps/x86/cacheinfo.c (init_cacheinfo): Don't limit threads
sharing L2 cache to 2 for Knights Landing.

ChangeLog
sysdeps/x86/cacheinfo.c

index 0d9a25c..c5ed4cd 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2016-05-20  H.J. Lu  <hongjiu.lu@intel.com>
+
+       [BZ #18185]
+       * sysdeps/x86/cacheinfo.c (init_cacheinfo): Don't limit threads
+       sharing L2 cache to 2 for Knights Landing.
+
 2016-05-20  Joseph Myers  <joseph@codesourcery.com>
 
        * conform/data/ftw.h-data (struct FTW): Do not expect for [XPG3].
index 020d3fd..182426b 100644 (file)
@@ -573,8 +573,6 @@ init_cacheinfo (void)
                {
                  switch (model)
                    {
-                   case 0x57:
-                     /* Knights Landing has L2 cache shared by 2 cores.  */
                    case 0x37:
                    case 0x4a:
                    case 0x4d: