lkml.org 
[lkml]   [2004]   [Jun]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
SubjectRe: [PATCH] nForce2 C1halt fixup, again
Date
On Friday 04 of June 2004 04:26, Aric Cyr wrote:
> With so many people reporting that 2.6.6 fixed the nForce2 hard lock
> problem I soon started using it. However, on my system it would still
> lock up! I tried disabling APIC, IO-APIC, ACPI, PREEMPT, etc. to no
> avail.
>
> After looking into the pci_fixup_nforce2() function, I saw that it was
> expecting one of two cases for the PCI config value: 0x1F0FFF01 or
> 0x9F0FFF01, then, depending on the PCI revision ID it would set the
> config value to 0x1F01FF01 or 0x9F01FF01 resp. I looked at the value
> of my nForce2 board and saw that it was actually 0x8F0FFF01.
>
> So the current 2.6.6 fixup was inadvertently flipping the high nibble
> to 0x9 in my case. Since the fixup is actually idependent of the PCI
> revision ID (the 5th nibble is changed from 0xF to 0x1 in either
> case), I tried explicitly changing only that part of the config value.
> Sure enough, for the first time in a while my system is finally stable
> again.
>
> My patch is attached. Comments please!

If bit 0x10000000 is not set then C1 Halt Disconnect is disabled.
I have reports that it is unsupported on some boards.

So what about patch below instead?


[PATCH] apply nForce2 fixup only if C1 Halt Disconnect is enabled

Some boards don't support C1 Halt Disconnect.

Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@elka.pw.edu.pl>

linux-2.6.7-rc2-bk2-bzolnier/arch/i386/pci/fixup.c | 7 ++++++-
1 files changed, 6 insertions(+), 1 deletion(-)

diff -puN arch/i386/pci/fixup.c~nForce2_C1Halt_fixup arch/i386/pci/fixup.c
--- linux-2.6.7-rc2-bk2/arch/i386/pci/fixup.c~nForce2_C1Halt_fixup 2004-06-03 16:52:46.556771744 +0200
+++ linux-2.6.7-rc2-bk2-bzolnier/arch/i386/pci/fixup.c 2004-06-03 16:58:17.275494856 +0200
@@ -226,7 +226,12 @@ static void __init pci_fixup_nforce2(str
fixed_val = rev < 0xC1 ? 0x1F01FF01 : 0x9F01FF01;

pci_read_config_dword(dev, 0x6c, &val);
- if (val != fixed_val) {
+
+ /*
+ * Apply fixup only if C1 Halt Disconnect is enabled
+ * (bit28) because it is not supported on some boards.
+ */
+ if ((val & (1 << 28)) && val != fixed_val) {
printk(KERN_WARNING "PCI: nForce2 C1 Halt Disconnect fixup\n");
pci_write_config_dword(dev, 0x6c, fixed_val);
}
_

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 14:03    [W:0.066 / U:0.580 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site