net: axienet: Be more careful about updating tx_bd_tail
authorRobert Hancock <robert.hancock@calian.com>
Thu, 12 May 2022 17:18:52 +0000 (11:18 -0600)
committerDavid S. Miller <davem@davemloft.net>
Fri, 13 May 2022 11:22:11 +0000 (12:22 +0100)
commitf0cf4000f5867ec4325d19d32bd83cf583065667
treec2c09a341e6e456612642c389edc734114d81eda
parent4915d50e300e96929d2462041d6f6c6f061167fd
net: axienet: Be more careful about updating tx_bd_tail

The axienet_start_xmit function was updating the tx_bd_tail variable
multiple times, with potential rollbacks on error or invalid
intermediate positions, even though this variable is also used in the
TX completion path. Use READ_ONCE where this variable is read and
WRITE_ONCE where it is written to make this update more atomic, and
move the write before the MMIO write to start the transfer, so it is
protected by that implicit write barrier.

Signed-off-by: Robert Hancock <robert.hancock@calian.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/xilinx/xilinx_axienet_main.c