Re: [PATCH net] net: stmmac: Re-work the queue selection for TSO packets

From: David Miller
Date: Mon Jul 08 2019 - 15:41:29 EST


From: Jose Abreu <Jose.Abreu@xxxxxxxxxxxx>
Date: Mon, 8 Jul 2019 14:26:28 +0200

> 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@xxxxxxxxxxxxxxx>
> Signed-off-by: Jose Abreu <joabreu@xxxxxxxxxxxx>

Applied and queued up for -stable.