]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
net: stmmac: Re-work the queue selection for TSO packets
authorJose Abreu <Jose.Abreu@synopsys.com>
Mon, 8 Jul 2019 12:26:28 +0000 (14:26 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 28 Jul 2019 06:28:30 +0000 (08:28 +0200)
commit7a78bda3cad948939f006cb923893184a30a8d75
tree3088f8012f413fc91373277ccbdea7c4197785f9
parente6da0a5873732f9ea06345bb01a8117fd4529a43
net: stmmac: Re-work the queue selection for TSO packets

[ Upstream commit 4993e5b37e8bcb55ac90f76eb6d2432647273747 ]

Ben Hutchings says:
"This is the wrong place to change the queue mapping.
stmmac_xmit() is called with a specific TX queue locked,
and accessing a different TX queue results in a data race
for all of that queue's state.

I think this commit should be reverted upstream and in all
stable branches.  Instead, the driver should implement the
ndo_select_queue operation and override the queue mapping there."

Fixes: c5acdbee22a1 ("net: stmmac: Send TSO packets always from Queue 0")
Suggested-by: Ben Hutchings <ben@decadent.org.uk>
Signed-off-by: Jose Abreu <joabreu@synopsys.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/net/ethernet/stmicro/stmmac/stmmac_main.c