hollalinux
/
linux-sg2042
mirror of https://github.com/sophgo/linux-riscv.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[PATCH] lockdep: fix smc91x 

When booting using root-nfs, I'm seeing (independently) two lockdep dumps
in the smc91x driver.  The patch below fixes both.  Both dumps look like
real locking issues.

Nico - please review and ack if you think the patch is correct.

Dump 1:

Sending DHCP requests .
=================================
[ INFO: inconsistent lock state ]

Signed-off-by: Jeff Garzik <jeff@garzik.org>
This commit is contained in:
Russell King 2006-08-14 23:00:20 -07:00 committed by Jeff Garzik
parent ab1976682f
commit 76cb4fe7c0
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
drivers/net/smc91x.c
View File

@ -321,12 +321,12 @@ static void smc_reset(struct net_device *dev)
DBG(2, "%s: %s\n", dev->name, __FUNCTION__); DBG(2, "%s: %s\n", dev->name, __FUNCTION__);
/* Disable all interrupts, block TX tasklet */ /* Disable all interrupts, block TX tasklet */
spin_lock(&lp->lock); spin_lock_irq(&lp->lock);
SMC_SELECT_BANK(2); SMC_SELECT_BANK(2);
SMC_SET_INT_MASK(0); SMC_SET_INT_MASK(0);
pending_skb = lp->pending_tx_skb; pending_skb = lp->pending_tx_skb;
lp->pending_tx_skb = NULL; lp->pending_tx_skb = NULL;
spin_unlock(&lp->lock); spin_unlock_irq(&lp->lock);
/* free any pending tx skb */ /* free any pending tx skb */
if (pending_skb) { if (pending_skb) {
@ -448,12 +448,12 @@ static void smc_shutdown(struct net_device *dev)
DBG(2, "%s: %s\n", CARDNAME, __FUNCTION__); DBG(2, "%s: %s\n", CARDNAME, __FUNCTION__);
/* no more interrupts for me */ /* no more interrupts for me */
spin_lock(&lp->lock); spin_lock_irq(&lp->lock);
SMC_SELECT_BANK(2); SMC_SELECT_BANK(2);
SMC_SET_INT_MASK(0); SMC_SET_INT_MASK(0);
pending_skb = lp->pending_tx_skb; pending_skb = lp->pending_tx_skb;
lp->pending_tx_skb = NULL; lp->pending_tx_skb = NULL;
spin_unlock(&lp->lock); spin_unlock_irq(&lp->lock);
if (pending_skb) if (pending_skb)
dev_kfree_skb(pending_skb); dev_kfree_skb(pending_skb);