public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* 2.6.9-rc1-bk11+ and 2.6.9-rc1-mm3,4 r8169: freeze during boot (FIX included)
@ 2004-09-12 22:35 Hans-Frieder Vogt
  2004-09-12 23:26 ` Francois Romieu
  0 siblings, 1 reply; 20+ messages in thread
From: Hans-Frieder Vogt @ 2004-09-12 22:35 UTC (permalink / raw)
  To: linux-kernel

Hi,

2.6.9-rc1-bk11 introduced a patch for the Realtek 8169 network chip driver, 
that leads to a freeze of my system during bootup.
My system:
Athlon 64, VIA K8T800 chipset, RTL8110S-32 (equiv. to RTL8169), running in 
64bit mode

no unusual messages are on the console prior to the freeze.

I traced the problem back to the patch which was introduced in 2.6.9-rc1-bk11 
and is also part of 2.6.9-rc1-mm3 and -mm4:
--- a/drivers/net/r8169.c       2004-07-02 11:51:44 -07:00
+++ b/drivers/net/r8169.c       2004-08-31 00:15:35 -07:00
@@ -983,7 +983,7 @@

        tp->cp_cmd = PCIMulRW | RxChkSum;

-       if ((sizeof(dma_addr_t) > 32) &&
+       if ((sizeof(dma_addr_t) > 4) &&
            !pci_set_dma_mask(pdev, DMA_64BIT_MASK))
                tp->cp_cmd |= PCIDAC;
        else {

which now, on my 64bit-system, enables DAC. For whatever reason this freezes 
my system (I do not understand why, because the r8169 seems to understand DAC 
according to the available documentation, perhaps a VIA K8T800 bug?).
Until somebody comes up with a proper solution for this problem, I suggest as 
a work-around to introduce a parameter so that the DAC can be simply 
unselected if necessary, as outlined in the patch below.

Thanks for any suggestions as to what the problem might be.
Hans-Frieder

--- linux-2.6.9-rc1-mm4.orig/drivers/net/r8169.c 2004-09-08 15:43:31.525119800 
+0200
+++ linux-2.6.9-rc1-mm4/drivers/net/r8169.c 2004-09-11 12:54:53.910456828 
+0200
@@ -167,6 +167,7 @@
 MODULE_DEVICE_TABLE(pci, rtl8169_pci_tbl);
 
 static int rx_copybreak = 200;
+static int use_dac = 1;
 
 enum RTL8169_registers {
  MAC0 = 0,  /* Ethernet hardware address. */
@@ -398,6 +399,8 @@
 MODULE_DESCRIPTION("RealTek RTL-8169 Gigabit Ethernet driver");
 MODULE_PARM(media, "1-" __MODULE_STRING(MAX_UNITS) "i");
 MODULE_PARM(rx_copybreak, "i");
+MODULE_PARM(use_dac, "i");
+MODULE_PARM_DESC(use_dac, "Use DAC addressing for DMA transfers on 64bit 
machines");
 MODULE_LICENSE("GPL");
 
 static int rtl8169_open(struct net_device *dev);
@@ -1152,7 +1155,7 @@
 
  dev->features |= NETIF_F_HW_VLAN_TX | NETIF_F_HW_VLAN_RX;
 
- if ((sizeof(dma_addr_t) > 4) && !pci_set_dma_mask(pdev, DMA_64BIT_MASK)) {
+ if ((sizeof(dma_addr_t) > 4) && use_dac && !pci_set_dma_mask(pdev, 
DMA_64BIT_MASK)) {
   tp->cp_cmd |= PCIDAC;
   dev->features |= NETIF_F_HIGHDMA;
  } else {

-- 
--
Hans-Frieder Vogt                 e-mail: hfvogt (at) arcor (dot) de

^ permalink raw reply	[flat|nested] 20+ messages in thread
end of thread, other threads:[~2004-09-20 21:16 UTC | newest]

Thread overview: 20+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-09-12 22:35 2.6.9-rc1-bk11+ and 2.6.9-rc1-mm3,4 r8169: freeze during boot (FIX included) Hans-Frieder Vogt
2004-09-12 23:26 ` Francois Romieu
2004-09-13 12:43   ` Hans-Frieder Vogt
2004-09-13 22:02     ` Francois Romieu
2004-09-13 23:31       ` Hans-Frieder Vogt
2004-09-15 22:40         ` Hans-Frieder Vogt
2004-09-15 23:09           ` Francois Romieu
2004-09-15 23:32             ` Hans-Frieder Vogt
2004-09-16  7:02               ` Francois Romieu
2004-09-16 18:20                 ` Jon Mason
2004-09-17 15:43                   ` Jon Mason
2004-09-17 16:01                     ` Francois Romieu
2004-09-19 21:17                       ` Andy Lutomirski
2004-09-19 21:39                         ` Francois Romieu
2004-09-19 23:51                           ` Andy Lutomirski
2004-09-20  2:56                           ` Jeff Garzik
2004-09-20  7:17                             ` Francois Romieu
2004-09-20 17:59                               ` Jeff Garzik
2004-09-20 21:15                                 ` Francois Romieu
2004-09-20 16:06                         ` Jon Mason

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox