Re: r8169, 2.6.2-rc2, Sager 4780 laptop

[Francois Romieu <romieu@xxxxxxxxxxxxx>]

bhartin@xxxxxxxxxxxxxxxx <bhartin@xxxxxxxxxxxxxxxx> :
[...]
> I got my laptop back up and found the messages logged:
> 
> kernel: r8169: eth0: Reset RTL8169s PHY
> 
> I didn't see exactly at what point this started happening during the 
> shutdown, but the message repeated about 500 times before the system 
> logger was stopped.

It implies a minimum of 6000 invocations of rtl8169_phy_timer(). 

*doh*

Please apply patch below.



mod_timer() expects an absolute time, not a relative offset.


 drivers/net/r8169.c |    2 +-
 1 files changed, 1 insertion(+), 1 deletion(-)

diff -puN drivers/net/r8169.c~r8169-mod_timer drivers/net/r8169.c
--- linux-2.6.5/drivers/net/r8169.c~r8169-mod_timer     2004-04-07 
01:30:14.000000000 +0200
+++ linux-2.6.5-fr/drivers/net/r8169.c  2004-04-07 01:32:53.000000000 +0200
@@ -617,7 +617,7 @@ static void rtl8169_phy_timer(unsigned l
                }
        }
 
-       mod_timer(timer, RTL8169_PHY_TIMEOUT);
+       mod_timer(timer, jiffies + RTL8169_PHY_TIMEOUT);
 }
 
 static inline void rtl8169_delete_timer(struct net_device *dev)

_