From: Ulrich Drepper <drepper@redhat.com>
Date: Sat, 25 Aug 2007 17:24:23 +0000 (+0000)
Subject: * sysdeps/x86_64/cacheinfo.c (handle_amd): Fix computation of
X-Git-Tag: cvs/fedora-glibc-20070825T1839
X-Git-Url: https://sourceware.org/git/?a=commitdiff_plain;h=76fca9f14a5fac1ab7a7b51e9bdff062fc364576;p=glibc.git

* sysdeps/x86_64/cacheinfo.c (handle_amd): Fix computation of

	associativity for fully-associative caches.
---

diff --git a/ChangeLog b/ChangeLog
index 7de59d8f68..9374c90f9d 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
 2007-08-25  Ulrich Drepper  <drepper@redhat.com>
 
+	* sysdeps/x86_64/cacheinfo.c (handle_amd): Fix computation of
+	associativity for fully-associative caches.
+
 	* sysdeps/x86_64/cacheinfo.c (handle_amd): Handle L3 cache
 	requests.  Fill on more associativity values for L2.
 	Patch mostly by Evandro Menezes.
diff --git a/sysdeps/x86_64/cacheinfo.c b/sysdeps/x86_64/cacheinfo.c
index eef7fa8726..793dc2d357 100644
--- a/sysdeps/x86_64/cacheinfo.c
+++ b/sysdeps/x86_64/cacheinfo.c
@@ -296,14 +296,13 @@ handle_amd (int name)
       return (ecx & 0xf000) == 0 ? 0 : (ecx >> 6) & 0x3fffc00;
 
     case _SC_LEVEL2_CACHE_ASSOC:
-      ecx >>= 12;
-      switch (ecx & 0xf)
+      switch ((ecx >> 12) & 0xf)
         {
         case 0:
         case 1:
         case 2:
         case 4:
-	  return ecx & 0xf;
+	  return (ecx >> 12) & 0xf;
 	case 6:
 	  return 8;
 	case 8:
@@ -319,7 +318,7 @@ handle_amd (int name)
 	case 14:
 	  return 128;
 	case 15:
-	  return (ecx << 6) & 0x3fffc00;
+	  return ((ecx >> 6) & 0x3fffc00) / (ecx & 0xff);
 	default:
 	  return 0;
         }
@@ -332,14 +331,13 @@ handle_amd (int name)
       return (edx & 0xf000) == 0 ? 0 : (edx & 0x3ffc0000) << 1;
 
     case _SC_LEVEL3_CACHE_ASSOC:
-      edx >>= 12;
-      switch (edx & 0xf)
+      switch ((edx >> 12) & 0xf)
 	{
 	case 0:
 	case 1:
 	case 2:
 	case 4:
-	  return edx & 0xf;
+	  return (edx >> 12) & 0xf;
 	case 6:
 	  return 8;
 	case 8:
@@ -355,7 +353,7 @@ handle_amd (int name)
 	case 14:
 	  return 128;
 	case 15:
-	  return (edx & 0x3ffc0) << 13;
+	  return ((edx & 0x3ffc0000) << 1) / (edx & 0xff);
 	default:
 	  return 0;
 	}