From b7598b1b855b39628104feb9a1f6151a86351a64 Mon Sep 17 00:00:00 2001 From: "H.J. Lu" Date: Fri, 20 May 2016 14:41:14 -0700 Subject: [PATCH] Remove special L2 cache case for Knights Landing 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 | 6 ++++++ sysdeps/x86/cacheinfo.c | 2 -- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 0d9a25cbb9..c5ed4cdbbb 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2016-05-20 H.J. Lu + + [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 * conform/data/ftw.h-data (struct FTW): Do not expect for [XPG3]. diff --git a/sysdeps/x86/cacheinfo.c b/sysdeps/x86/cacheinfo.c index 020d3fd397..182426b2d0 100644 --- a/sysdeps/x86/cacheinfo.c +++ b/sysdeps/x86/cacheinfo.c @@ -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: -- 2.43.5