isdn:hisax: Replace dev_kfree_skb_any by dev_consume_skb_any
authorHuang Zijiang <huang.zijiang@zte.com.cn>
Thu, 14 Feb 2019 06:39:59 +0000 (14:39 +0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 14 Feb 2019 17:22:03 +0000 (12:22 -0500)
The skb should be freed by dev_consume_skb_any() in hfcpci_fill_fifo()
when bcs->tx_skb is still used. The bcs->tx_skb is be replaced by
skb_dequeue(&bcs->squeue), so the original bcs->tx_skb should
be consumed(not drop).

Signed-off-by: Huang Zijiang <huang.zijiang@zte.com.cn>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/isdn/hisax/hfc_pci.c

index 81dd465afcf411d1bea0422f27daed2b1c2c5594..71a8312592d6fd9f0858b04834161d1ac8f3ddf7 100644 (file)
@@ -656,7 +656,7 @@ hfcpci_fill_fifo(struct BCState *bcs)
                                schedule_event(bcs, B_ACKPENDING);
                        }
 
-                       dev_kfree_skb_any(bcs->tx_skb);
+                       dev_consume_skb_any(bcs->tx_skb);
                        bcs->tx_skb = skb_dequeue(&bcs->squeue);        /* fetch next data */
                }
                test_and_clear_bit(BC_FLG_BUSY, &bcs->Flag);