linux/drivers/net/can
Marc Kleine-Budde 51d3e462ea can: m_can: m_can_tx_work_queue(): fix tx_skb race condition
[ Upstream commit e04b2cfe61072c7966e1a5fb73dd1feb30c206ed ]

The m_can_start_xmit() function checks if the cdev->tx_skb is NULL and
returns with NETDEV_TX_BUSY in case tx_sbk is not NULL.

There is a race condition in the m_can_tx_work_queue(), where first
the skb is send to the driver and then the case tx_sbk is set to NULL.
A TX complete IRQ might come in between and wake the queue, which
results in tx_skb not being cleared yet.

Fixes: f524f829b7 ("can: m_can: Create a m_can platform framework")
Tested-by: Torin Cooper-Bennun <torin@maxiluxsystems.com>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
Signed-off-by: Sasha Levin <sashal@kernel.org>
2021-05-19 10:08:28 +02:00
..
c_can can: c_can: move runtime PM enable/disable to c_can_platform 2021-03-30 14:35:26 +02:00
cc770
dev net: introduce CAN specific pointer in the struct net_device 2021-04-07 14:47:41 +02:00
ifi_canfd
m_can can: m_can: m_can_tx_work_queue(): fix tx_skb race condition 2021-05-19 10:08:28 +02:00
mscan
peak_canfd
rcar
sja1000
softing
spi
usb drivers: net: fix memory leak in peak_usb_create_dev 2021-04-14 08:24:18 +02:00
Kconfig
Makefile can: dev: move driver related infrastructure into separate subdir 2021-04-07 14:47:41 +02:00
at91_can.c
flexcan.c
grcan.c
janz-ican3.c
kvaser_pciefd.c can: kvaser_pciefd: Always disable bus load reporting 2021-03-30 14:35:26 +02:00
led.c
pch_can.c
slcan.c net: introduce CAN specific pointer in the struct net_device 2021-04-07 14:47:41 +02:00
sun4i_can.c
ti_hecc.c
vcan.c net: introduce CAN specific pointer in the struct net_device 2021-04-07 14:47:41 +02:00
vxcan.c net: introduce CAN specific pointer in the struct net_device 2021-04-07 14:47:41 +02:00
xilinx_can.c