net: plip: don't call kfree_skb/dev_kfree_skb() under spin_lock_irq()
authorYang Yingliang <yangyingliang@huawei.com>
Wed, 7 Dec 2022 01:53:10 +0000 (09:53 +0800)
committerJakub Kicinski <kuba@kernel.org>
Thu, 8 Dec 2022 04:10:47 +0000 (20:10 -0800)
It is not allowed to call kfree_skb() or consume_skb() from
hardware interrupt context or with interrupts being disabled.
So replace kfree_skb/dev_kfree_skb() with dev_kfree_skb_irq()
and dev_consume_skb_irq() under spin_lock_irq().

Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
Reviewed-by: Jiri Pirko <jiri@nvidia.com>
Link: https://lore.kernel.org/r/20221207015310.2984909-1-yangyingliang@huawei.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/plip/plip.c

index c8791e9..40ce8ab 100644 (file)
@@ -450,12 +450,12 @@ plip_bh_timeout_error(struct net_device *dev, struct net_local *nl,
        }
        rcv->state = PLIP_PK_DONE;
        if (rcv->skb) {
-               kfree_skb(rcv->skb);
+               dev_kfree_skb_irq(rcv->skb);
                rcv->skb = NULL;
        }
        snd->state = PLIP_PK_DONE;
        if (snd->skb) {
-               dev_kfree_skb(snd->skb);
+               dev_consume_skb_irq(snd->skb);
                snd->skb = NULL;
        }
        spin_unlock_irq(&nl->lock);