[PATCH] USBATM: use dev_kfree_skb_any rather than dev_kfree_skb
authorDuncan Sands <baldrick@free.fr>
Fri, 13 Jan 2006 09:13:19 +0000 (10:13 +0100)
committerGreg Kroah-Hartman <gregkh@suse.de>
Wed, 1 Feb 2006 01:23:40 +0000 (17:23 -0800)
In one spot (usbatm_cancel_send) we were calling dev_kfree_skb with irqs
disabled.  This mistake is just too easy to make, so systematically use
dev_kfree_skb_any rather than dev_kfree_skb.

Signed-off-by: Duncan Sands <baldrick@free.fr>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/usb/atm/usbatm.c

index 103764d..98b74b9 100644 (file)
@@ -72,6 +72,7 @@
 #include <linux/kernel.h>
 #include <linux/module.h>
 #include <linux/moduleparam.h>
+#include <linux/netdevice.h>
 #include <linux/proc_fs.h>
 #include <linux/sched.h>
 #include <linux/signal.h>
@@ -199,7 +200,7 @@ static inline void usbatm_pop(struct atm_vcc *vcc, struct sk_buff *skb)
        if (vcc->pop)
                vcc->pop(vcc, skb);
        else
-               dev_kfree_skb(skb);
+               dev_kfree_skb_any(skb);
 }
 
 
@@ -397,7 +398,7 @@ static void usbatm_extract_cells(struct usbatm_data *instance,
                        if (!atm_charge(vcc, skb->truesize)) {
                                atm_rldbg(instance, "%s: failed atm_charge (skb->truesize: %u)!\n",
                                                __func__, skb->truesize);
-                               dev_kfree_skb(skb);
+                               dev_kfree_skb_any(skb);
                                goto out;       /* atm_charge increments rx_drop */
                        }