Skip to contentSkip to conversation

vesafb: Disable mtrr as the default

PATCH In review Antonino A. Daplas 1 participant 1 message Oct 18, 2005

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 14/17 vesafb: Disable mtrr as the default

vesafb occassionally gets the size wrong when setting the mtrr. When X or DRI attempts to set the mtrr, it will fail due to range overlap significantly affecting their performance. Disable mtrr and let the user explicitly enable it with the mtrr:n option.

---
 Documentation/fb/vesafb.txt |    4 ++--
 drivers/video/vesafb.c      |    2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)
Documentation/fb/vesafb.txt +2 -2
@@ -146,10 +146,10 @@ pmipal	Use the protected mode interface 
 
 mtrr:n	setup memory type range registers for the vesafb framebuffer
 	where n:
-	      0 - disabled (equivalent to nomtrr)
+	      0 - disabled (equivalent to nomtrr) (default)
 	      1 - uncachable
 	      2 - write-back
-	      3 - write-combining (default)
+	      3 - write-combining
 	      4 - write-through
 
 	If you see the following in dmesg, choose the type that matches the
drivers/video/vesafb.c +1 -1
@@ -46,7 +46,7 @@ static struct fb_fix_screeninfo vesafb_f
 };
 
 static int             inverse   = 0;
-static int             mtrr      = 3; /* default to write-combining */
+static int             mtrr      = 0; /* disable mtrr */
 static int	       vram_remap __initdata = 0; /* Set amount of memory to be used */
 static int	       vram_total __initdata = 0; /* Set total amount of memory */
 static int             pmi_setpal = 0;	/* pmi for palette changes ??? */

-------------------------------------------------------
This SF.Net email is sponsored by:
Power Architecture Resource Center: Free content, downloads, discussions, and more. http://solutions.newsforge.com/ibmarch.tmpl