OcCpuLib: Fix incorrect identification of Xeon E5 and W CPUs

acidanthera · Apr 4, 2021 · 6dcabf0 · 6dcabf0
1 parent cb527e7
commit 6dcabf0
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/Changelog.md b/Changelog.md
@@ -57,6 +57,7 @@ OpenCore Changelog
 - Added dynamic keyboard protocol installation on CrScreenshotDxe
 - Support starting UEFI tools with argument support (e.g. `ControlMsrE2`) without arguments from picker
 - Fixed OpenCanopy font height calculation, may reject previously working fonts and mitigate memory corruption
+- Fixed incorrect identification of Xeon E5XXX/E5-XXXX and Xeon WXXXX/W-XXXX CPUs
 
 #### v0.6.7
 - Fixed ocvalidate return code to be non-zero when issues are found

diff --git a/Library/OcCpuLib/AppleCpuSupport.c b/Library/OcCpuLib/AppleCpuSupport.c
@@ -103,10 +103,10 @@ InternalDetectAppleMajorType (
       }
     }
 
-    if (AsciiStrnCmp (BrandInfix, "E5", L_STR_LEN ("E5")) == 0) {
+    if (AsciiStrnCmp (BrandInfix, "E5-", L_STR_LEN ("E5-")) == 0) {
       return AppleProcessorMajorXeonE5;
     }
-    if (AsciiStrnCmp (BrandInfix, "W", L_STR_LEN ("W")) == 0) {
+    if (AsciiStrnCmp (BrandInfix, "W-", L_STR_LEN ("W-")) == 0) {
       return AppleProcessorMajorXeonW;
     }
     return AppleProcessorMajorXeonNehalem;