Commit 516a0cd1 authored by Marc Kleine-Budde's avatar Marc Kleine-Budde
Browse files

can: gs_usb: gs_usb_xmit_callback(): fix handling of failed transmitted URBs 

The driver lacks the cleanup of failed transfers of URBs. This reduces the
number of available URBs per error by 1. This leads to reduced performance
and ultimately to a complete stop of the transmission.

If the sending of a bulk URB fails do proper cleanup:
- increase netdev stats
- mark the echo_sbk as free
- free the driver's context and do accounting
- wake the send queue

Closes: https://github.com/candle-usb/candleLight_fw/issues/187
Fixes: d08e973a ("can: gs_usb: Added support for the GS_USB CAN devices")
Link: https://patch.msgid.link/20251114-gs_usb-fix-usb-callbacks-v1-1-a29b42eacada@pengutronix.de


Signed-off-by: default avatarMarc Kleine-Budde <mkl@pengutronix.de>
parent 30db4451

drivers/net/can/usb/gs_usb.c

+15 −2
Original line number Diff line number Diff line
@@ -750,8 +750,21 @@ static void gs_usb_xmit_callback(struct urb *urb) 
	struct gs_can *dev = txc->dev;

	struct net_device *netdev = dev->netdev;



	if (urb->status)

		netdev_info(netdev, "usb xmit fail %u\n", txc->echo_id);

	if (!urb->status)

		return;



	if (urb->status != -ESHUTDOWN && net_ratelimit())

		netdev_info(netdev, "failed to xmit URB %u: %pe\n",

			    txc->echo_id, ERR_PTR(urb->status));



	netdev->stats.tx_dropped++;

	netdev->stats.tx_errors++;



	can_free_echo_skb(netdev, txc->echo_id, NULL);

	gs_free_tx_context(txc);

	atomic_dec(&dev->active_tx_urbs);



	netif_wake_queue(netdev);

}



static netdev_tx_t gs_can_start_xmit(struct sk_buff *skb,