]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commit
net: ethernet: sxgbe: remove private tx queue lock
authorLino Sanfilippo <LinoSanfilippo@gmx.de>
Thu, 8 Dec 2016 23:55:42 +0000 (00:55 +0100)
committerDavid S. Miller <davem@davemloft.net>
Sun, 11 Dec 2016 04:23:35 +0000 (23:23 -0500)
commit980f140493b6d9d9e1fc67ce594170186be848a1
tree5a4cf0d4b4d96b42617e9a0699cdbc339b6cac0b
parentc280b482662018b2178f67ea017275191c30823e
net: ethernet: sxgbe: remove private tx queue lock

The driver uses a private lock for synchronization of the xmit function and
the xmit completion handler, but since the NETIF_F_LLTX flag is not set,
the xmit function is also called with the xmit_lock held.

On the other hand the completion handler uses the reverse locking order by
first taking the private lock and (in case that the tx queue had been
stopped) then the xmit_lock.

Improve the locking by removing the private lock and using only the
xmit_lock for synchronization instead.

Signed-off-by: Lino Sanfilippo <LinoSanfilippo@gmx.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/samsung/sxgbe/sxgbe_common.h
drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c