diff options
author | Huang Zijiang <huang.zijiang@zte.com.cn> | 2019-02-14 01:39:59 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2019-02-14 12:22:03 -0500 |
commit | 189a10f0108a8ebaf1e2f3d8bf68aeb5ebd30109 (patch) | |
tree | caa24796a1017eefb4fcb0596a1441576df4d1e1 | |
parent | a37d45b6fa1113f96200abdfdb0d8c454c97a30e (diff) |
isdn:hisax: Replace dev_kfree_skb_any by dev_consume_skb_any
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>
-rw-r--r-- | drivers/isdn/hisax/hfc_pci.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/isdn/hisax/hfc_pci.c b/drivers/isdn/hisax/hfc_pci.c index 81dd465afcf4..71a8312592d6 100644 --- a/drivers/isdn/hisax/hfc_pci.c +++ b/drivers/isdn/hisax/hfc_pci.c | |||
@@ -656,7 +656,7 @@ hfcpci_fill_fifo(struct BCState *bcs) | |||
656 | schedule_event(bcs, B_ACKPENDING); | 656 | schedule_event(bcs, B_ACKPENDING); |
657 | } | 657 | } |
658 | 658 | ||
659 | dev_kfree_skb_any(bcs->tx_skb); | 659 | dev_consume_skb_any(bcs->tx_skb); |
660 | bcs->tx_skb = skb_dequeue(&bcs->squeue); /* fetch next data */ | 660 | bcs->tx_skb = skb_dequeue(&bcs->squeue); /* fetch next data */ |
661 | } | 661 | } |
662 | test_and_clear_bit(BC_FLG_BUSY, &bcs->Flag); | 662 | test_and_clear_bit(BC_FLG_BUSY, &bcs->Flag); |