Commit 4000b375 authored by Jeff Garzik's avatar Jeff Garzik

[netdrvr 8139too] respond to "isn't this racy?" comment

parent b6a7a9b6
...@@ -2143,9 +2143,7 @@ static int rtl8139_close (struct net_device *dev) ...@@ -2143,9 +2143,7 @@ static int rtl8139_close (struct net_device *dev)
spin_unlock_irqrestore (&tp->lock, flags); spin_unlock_irqrestore (&tp->lock, flags);
/* TODO: isn't this code racy? we synchronize the IRQ and then free it, */ synchronize_irq (dev->irq); /* racy, but that's ok here */
/* but another IRQ could've happened in between the sync and free */
synchronize_irq (dev->irq);
free_irq (dev->irq, dev); free_irq (dev->irq, dev);
rtl8139_tx_clear (tp); rtl8139_tx_clear (tp);
......
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