Re: kernel BUG at tg3.c:1557

["David S. Miller" <davem@redhat.com>]

   From: Roland Kuhn <rkuhn@e18.physik.tu-muenchen.de>
   Date: Thu, 8 Aug 2002 00:10:31 +0200 (CEST)

   Since the insertion of a dummy write solved the problem, I would say it's 
   the chipset's PCI reordering, which is malfunctioning in the 2466.

Roland can you retry using this patch?  The difference from
my previous one is that when we use the indirect register
writing of the mailbox registers, we offset into the GRCMBOX
area of the chip registers.

This seems to be how Broadcom's driver does indirect accesses
to mailbox registers.

--- drivers/net/tg3.c.~1~	Wed Aug  7 15:42:34 2002
+++ drivers/net/tg3.c	Wed Aug  7 15:45:56 2002
@@ -173,8 +173,23 @@
 	}
 }
 
+static void tg3_write_indirect_mbox(struct tg3 *tp, u32 off, u32 val)
+{
+	if (/*XXX*/ 1 || (tp->tg3_flags & TG3_FLAG_PCIMEM_ORDERING_HWBUG) != 0) {
+		unsigned long flags;
+
+		spin_lock_irqsave(&tp->indirect_lock, flags);
+		pci_write_config_dword(tp->pdev, TG3PCI_REG_BASE_ADDR,
+				       off + 0x5600);
+		pci_write_config_dword(tp->pdev, TG3PCI_REG_DATA, val);
+		spin_unlock_irqrestore(&tp->indirect_lock, flags);
+	} else {
+		writel(val, tp->regs + off);
+	}
+}
+
 #define tw32(reg,val)		tg3_write_indirect_reg32(tp,(reg),(val))
-#define tw32_mailbox(reg, val)	writel(((val) & 0xffffffff), tp->regs + (reg))
+#define tw32_mailbox(reg, val)	tg3_write_indirect_mbox(tp,(reg),(val))
 #define tw16(reg,val)		writew(((val) & 0xffff), tp->regs + (reg))
 #define tw8(reg,val)		writeb(((val) & 0xff), tp->regs + (reg))
 #define tr32(reg)		readl(tp->regs + (reg))
--- drivers/net/tg3.h.~1~	Wed Aug  7 15:44:04 2002
+++ drivers/net/tg3.h	Wed Aug  7 15:44:30 2002
@@ -1740,7 +1740,8 @@
 #define TG3_FLAG_POLL_SERDES		0x00000080
 #define TG3_FLAG_PHY_RESET_ON_INIT	0x00000100
 #define TG3_FLAG_PCIX_TARGET_HWBUG	0x00000200
-#define TG3_FLAG_WOL_SPEED_100MB	0x00000400
+#define TG3_FLAG_PCIMEM_ORDERING_HWBUG	0x00000400
+#define TG3_FLAG_WOL_SPEED_100MB	0x00000800
 #define TG3_FLAG_WOL_ENABLE		0x00001000
 #define TG3_FLAG_NVRAM			0x00002000
 #define TG3_FLAG_NVRAM_BUFFERED		0x00004000