[PATCH] ppc: fix sungem NAPI

From: Benjamin Herrenschmidt
Date: Thu Sep 09 2004 - 22:55:32 EST


Hi !

The recent sungem NAPI change introduced a bug: dev_kfree_skb() is called
within the spinlock, thus triggers all sort of WARN_ON's later on down the
stack.

This patch changes it to dev_kfree_skb_any(), I hope that is fine
as we aren't really in interrupt, are we ? (I don't know in what
context NAPI polling occurs, is it a timer IRQ ?)

Ben.

===== drivers/net/sungem.c 1.59 vs edited =====
--- 1.59/drivers/net/sungem.c 2004-09-08 06:17:59 +10:00
+++ edited/drivers/net/sungem.c 2004-09-10 13:44:18 +10:00
@@ -651,7 +651,7 @@
}

gp->net_stats.tx_packets++;
- dev_kfree_skb(skb);
+ dev_kfree_skb_any(skb);
}
gp->tx_old = entry;



-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/