diff options
author | Casper Andersson <casper.casan@gmail.com> | 2022-12-02 09:35:44 +0100 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2022-12-05 11:33:09 +0000 |
commit | 121c6672b0191ffcebff4b88ec022c39e0a95789 (patch) | |
tree | 9a97153cbc47a322749ead815cee2a7fdc996d25 /drivers/net/wwan/iosm/iosm_ipc_mux.c | |
parent | fbf33f5ac76f2cdb47ad9763f620026d5cfa57ce (diff) |
net: microchip: sparx5: correctly free skb in xmit
consume_skb on transmitted, kfree_skb on dropped, do not free on
TX_BUSY.
Previously the xmit function could return -EBUSY without freeing, which
supposedly is interpreted as a drop. And was using kfree on successfully
transmitted packets.
sparx5_fdma_xmit and sparx5_inject returns error code, where -EBUSY
indicates TX_BUSY and any other error code indicates dropped.
Fixes: f3cad2611a77 ("net: sparx5: add hostmode with phylink support")
Signed-off-by: Casper Andersson <casper.casan@gmail.com>
Reviewed-by: Horatiu Vultur <horatiu.vultur@microchip.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/wwan/iosm/iosm_ipc_mux.c')
0 files changed, 0 insertions, 0 deletions