[PATCH] prism54: Free skb after disabling interrupts

The dev_kfree_skb in islpci_eth_transmit happens while irqs are still
disabled, so either dev_kfree_skb_irq needs to be used or the skb
needs to be freed after irqs have been enabled again. This patch
should fix it.

Signed-off-by: Patrick McHardy <[email protected]>
Signed-off-by: Daniel Drake <[email protected]>
Signed-off-by: Jeff Garzik <[email protected]>

diff --git a/drivers/net/wireless/prism54/islpci_eth.c b/drivers/net/wireless/prism54/islpci_eth.c
index 0975dd4ed77d1008d99487fa4f14265d0c446f68..3b49efa37ee56050f3294304ad0ce0c8dd539d59 100644 (file)
--- a/drivers/net/wireless/prism54/islpci_eth.c
+++ b/drivers/net/wireless/prism54/islpci_eth.c
@@ -241,12 +241,10 @@ islpci_eth_transmit(struct sk_buff *skb, struct net_device *ndev)
        return 0;
 
       drop_free:
-       /* free the skbuf structure before aborting */
-       dev_kfree_skb(skb);
-       skb = NULL;
-
        priv->statistics.tx_dropped++;
        spin_unlock_irqrestore(&priv->slock, flags);
+       dev_kfree_skb(skb);
+       skb = NULL;
        return err;
 }