linux-kernel - Re: [atl2] warn_slowpath in dev

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20090118095231.2c73c448@osprey.hogchain.net>
Date:	Sun, 18 Jan 2009 09:52:31 -0600
From:	Jay Cliburn <jcliburn@...il.com>
To:	Sitsofe Wheeler <sitsofe@...oo.com>
Cc:	linux-kernel@...r.kernel.org, Chris Snook <csnook@...hat.com>,
	jie.yang@...eros.com, atl1-devel@...ts.sourceforge.net,
	netdev@...r.kernel.org
Subject: Re: [atl2] warn_slowpath in dev_watchdog

On Sun, 18 Jan 2009 02:18:33 -0800 (PST)
Sitsofe Wheeler <sitsofe@...oo.com> wrote:

> Hi,
> 
> 
> > From: Jay Cliburn <jcliburn@...il.com>
> > 
> > On Sat, 17 Jan 2009 18:49:18 +0000 (UTC)
> > Sitsofe Wheeler wrote:
> > 
> > > On an ever so slightly modified 2.6.28. I think this only happens
> > > when the upstream router/switch goes out to lunch and packets are
> > > being sent. It is reproducible but only appears on a home
> > > network...
> > > 
> > > [  204.704065] ------------[ cut here ]------------
> > > [  204.704074] WARNING: at net/sched/sch_generic.c:226
> > > dev_watchdog +0x22b/0x240()
> > > [  204.704080] NETDEV WATCHDOG: eth0 (atl2): transmit timed out
> > 
> > Please provide your complete dmesg output.
> 
> Unfortunately I only saved that snippet of the log (I can provide you
> a full dmesg but it won't be from a run where this problem occured).

I'd like to see the full dmesg that includes the warning, if you can
obtain it next time it occurs.  Meanwhile, please see if this patch
helps.

diff --git a/drivers/net/atlx/atl2.c b/drivers/net/atlx/atl2.c
index 8571e8c..376226c 100644
--- a/drivers/net/atlx/atl2.c
+++ b/drivers/net/atlx/atl2.c
@@ -555,14 +555,18 @@ static void atl2_check_for_link(struct atl2_adapter *adapter)
 	atl2_read_phy_reg(&adapter->hw, MII_BMSR, &phy_data);
 	spin_unlock(&adapter->stats_lock);
 
-	/* notify upper layer link down ASAP */
 	if (!(phy_data & BMSR_LSTATUS)) { /* Link Down */
-		if (netif_carrier_ok(netdev)) { /* old link state: Up */
-		printk(KERN_INFO "%s: %s NIC Link is Down\n",
-			atl2_driver_name, netdev->name);
-		adapter->link_speed = SPEED_0;
-		netif_carrier_off(netdev);
-		netif_stop_queue(netdev);
+		if (netif_carrier_ok(netdev)) {
+			/*
+			 * Notify the upper layer and restart the netdev
+			 * watchdog timer while we try to recover the link.
+			 */
+			netif_carrier_off(netdev);
+			netif_stop_queue(netdev);
+			netdev->trans_start = jiffies;
+			printk(KERN_INFO "%s: %s NIC Link is Down\n",
+				atl2_driver_name, netdev->name);
+			adapter->link_speed = SPEED_0;
 		}
 	}
 	schedule_work(&adapter->link_chg_task);
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/