From 5998665f5e202af599a9cffd00aaac430dc0d20b Mon Sep 17 00:00:00 2001
From: Rudolf Cornelissen
Date: Tue, 11 Apr 2006 11:11:38 +0000
Subject: [PATCH] AGP module updated: modified default PCI fastwrites setting
to be disabled, so the busmanager should work (much) more reliable. ATI
graphicscards are known to mess-up when fastwrites are enabled. Updated docs.
Added version number: 0.02.
git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@17071 a95241bf-73f2-0310-859d-f6bbb57e9c96
---
src/add-ons/kernel/bus_managers/agp/UPDATE.html | 7 ++++---
src/add-ons/kernel/bus_managers/agp/agp.cpp | 2 +-
src/add-ons/kernel/bus_managers/agp/agp.settings | 2 +-
src/add-ons/kernel/bus_managers/agp/module.cpp | 4 ++--
4 files changed, 8 insertions(+), 7 deletions(-)
diff --git a/src/add-ons/kernel/bus_managers/agp/UPDATE.html b/src/add-ons/kernel/bus_managers/agp/UPDATE.html
index 2d7dfd89d4..71a6c000a1 100644
--- a/src/add-ons/kernel/bus_managers/agp/UPDATE.html
+++ b/src/add-ons/kernel/bus_managers/agp/UPDATE.html
@@ -4,11 +4,12 @@
Changes done for each module version:
-agp_module 0.02, (Rudolf)
+agp_module 0.02 (Rudolf)
- - Reversed order of programming devices when AGP mode is requested to be disabled. The order of programming devices now adheres to the official AGP specification (missed this item before). This fixes potential coldstart trouble on nVidia cards at least (confirmed a GeForce 4 MX4000).
+
- Reversed order of programming devices when AGP mode is requested to be disabled. The order of programming devices now adheres to the official AGP specification (missed this item before). This fixes potential coldstart trouble on nVidia cards at least (confirmed a GeForce 4 MX4000);
+
- Modified agp.setting for blocking PCI fastwrites (block_fw) to be set to true by default: Fastwrites tend to create trouble on some systems outthere (most notably systems with ATI graphics cards).
-agp_module 0.01, (Rudolf)
+agp_module 0.01 (Rudolf)
diff --git a/src/add-ons/kernel/bus_managers/agp/agp.cpp b/src/add-ons/kernel/bus_managers/agp/agp.cpp
index 978383c691..226fdf321b 100644
--- a/src/add-ons/kernel/bus_managers/agp/agp.cpp
+++ b/src/add-ons/kernel/bus_managers/agp/agp.cpp
@@ -80,7 +80,7 @@ static settings current_settings = { //see comments in agp.settings
0, // max_speed
false, // block_agp
false, // block_sba
- false, // block_fw
+ true, // block_fw
};
diff --git a/src/add-ons/kernel/bus_managers/agp/agp.settings b/src/add-ons/kernel/bus_managers/agp/agp.settings
index 0c30a5da3d..23e4cedebe 100644
--- a/src/add-ons/kernel/bus_managers/agp/agp.settings
+++ b/src/add-ons/kernel/bus_managers/agp/agp.settings
@@ -9,4 +9,4 @@
#max_speed 2 # max AGP speed that may be used (valid are 1,2,4 or 8)
block_agp false # if true blocks use of AGP entirely: so fallback to PCI mode
block_sba false # if true blocks use of 'sideband adressing' if possible
-block_fw false # if true blocks use of 'fastwrites'
+block_fw true # if true blocks use of 'fastwrites'
diff --git a/src/add-ons/kernel/bus_managers/agp/module.cpp b/src/add-ons/kernel/bus_managers/agp/module.cpp
index 50b1691b08..acdf68dbd8 100644
--- a/src/add-ons/kernel/bus_managers/agp/module.cpp
+++ b/src/add-ons/kernel/bus_managers/agp/module.cpp
@@ -36,14 +36,14 @@ bus_std_ops(int32 op, ...)
{
switch(op) {
case B_MODULE_INIT:
- TRACE("agp_man: bus module: init\n");
+ TRACE("agp_man: bus module V0.02: init\n");
if (init() != B_OK)
{
return ENODEV;
}
break;
case B_MODULE_UNINIT:
- TRACE("agp_man: bus module: uninit\n");
+ TRACE("agp_man: bus module V0.02: uninit\n");
uninit();
break;
default: