Enable Gen6Accelerator name fix by default

lvs1974 · Apr 8, 2018 · 5d4528e · 5d4528e
1 parent 13c64b0
commit 5d4528e
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 7 deletions.
diff --git a/Changelog.md b/Changelog.md
@@ -67,7 +67,7 @@ IntelGraphicsFixup Changelog
 
 #### v1.2.7
 - Add `igfxgl=1` boot argument (and `disable-metal` property) to disable Metal support
-- Add `igfxsnb=1` boot argument (and `modern-sandy` property) to rename Gen6Accelerator with IntelAccelerator to fix GVA warnings
+- Implement Gen6Accelerator with IntelAccelerator rename to fix GVA warnings (use `igfxsnb=0` boot argument to disable)
 - Harden solved symbol verification to avoid panics with broken kext cache
 - Replace connector-less Sandy Bridge framebuffer for unsupported macmodels from 0x30030 to 0x50000
 

diff --git a/IntelGraphicsFixup/kern_igfx.cpp b/IntelGraphicsFixup/kern_igfx.cpp
@@ -389,7 +389,6 @@ bool IGFX::intelGraphicsStart(IOService *that, IOService *provider) {
 	}
 
 	if (CPUInfo::getGeneration() == CPUInfo::CpuGeneration::SandyBridge) {
-		callbackIgfx->moderniseAccelerator = provider->getProperty("modern-sandy") != nullptr;
 		PE_parse_boot_argn("igfxsnb", &callbackIgfx->moderniseAccelerator, sizeof(callbackIgfx->moderniseAccelerator));
 		if (callbackIgfx->moderniseAccelerator)
 			that->setName("IntelAccelerator");

diff --git a/IntelGraphicsFixup/kern_igfx.hpp b/IntelGraphicsFixup/kern_igfx.hpp
@@ -324,7 +324,7 @@ class IGFX {
 	/**
 	 *  Set modern name to Sandy Accelerator
 	 */
-	int moderniseAccelerator {0};
+	int moderniseAccelerator {1};
 
 	/**
 	 *  Loaded vinfo

diff --git a/README.md b/README.md
@@ -14,8 +14,8 @@ An open source kernel extension providing patches to select Intel GPUs.
 - Allows booting in VESA mode with Intel HD graphics (via `-igfxvesa` boot argument)
 - Performs basic framebuffer id injection when none is specified (connector-less frames will be used with discrete GPUs)
 - Allows GuC microcode loading with Intel Skylake/Kaby Lake Graphics in 10.13 (via `igfxfw=1` boot argument)
-- Renames Gen6Accelerator with IntelAccelerator for Sandy Bridge CPUs for GVA support (via `igfxsnb=1` boot argument or `modern-sandy` property)
-- Allows booting with OpenGL-only acceleration (via `ngfxgl=0` boot argument or `disable-metal` property)
+- Renames Gen6Accelerator with IntelAccelerator for Sandy Bridge CPUs for GVA support
+- Allows booting with OpenGL-only acceleration (via `igfxgl=0` boot argument or `disable-metal` property)
 
 #### Configuration
 Add `-igfxdbg` to enable debug printing (available in DEBUG binaries).  
@@ -25,8 +25,8 @@ Add `-igfxvesa` to boot Intel graphics without hardware acceleration (VESA mode)
 Add `igfxfw=1` to enable GuC microcode loading in 10.13 or newer.  
 Add `igfxrst=1` to prefer drawing Apple logo at 2nd boot stage instead of framebuffer copying.  
 Add `igfxframe=frame` to inject a dedicated framebuffer identifier into IGPU (only for TESTING purposes).  
-Add `igfxsnb=1` to ensure proper IntelAccelerator name for Sandy Bridge CPUs for GVA support  
-Add `ngfxgl=0` to disable Metal support  
+Add `igfxsnb=0` to disable IntelAccelerator name fix for Sandy Bridge CPUs  
+Add `igfxgl=0` to disable Metal support  
 
 #### Credits
 - [Apple](https://www.apple.com) for macOS