Commit 0f23a3a8 authored by Andrew Morton's avatar Andrew Morton Committed by Linus Torvalds

[PATCH] 3c574_cs fixes

- It was doing spin_lock_irqsave()/spin_unlock()

- Can't free the skb inside local_irq_save(): kfree_skb ends up running
  local_bh_enable(), which enables interrupts.
parent b2520649
...@@ -940,11 +940,9 @@ static int el3_start_xmit(struct sk_buff *skb, struct net_device *dev) ...@@ -940,11 +940,9 @@ static int el3_start_xmit(struct sk_buff *skb, struct net_device *dev)
outw(SetTxThreshold + (1536>>2), ioaddr + EL3_CMD); outw(SetTxThreshold + (1536>>2), ioaddr + EL3_CMD);
} }
dev_kfree_skb (skb);
pop_tx_status(dev); pop_tx_status(dev);
spin_unlock_irqrestore(&lp->window_lock, flags);
spin_unlock(&lp->window_lock); dev_kfree_skb(skb);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment