]> git.baikalelectronics.ru Git - kernel.git/commit
amd-xgbe: Use wmb before updating current descriptor count
authorLendacky, Thomas <Thomas.Lendacky@amd.com>
Wed, 21 Oct 2015 20:37:05 +0000 (15:37 -0500)
committerDavid S. Miller <davem@davemloft.net>
Fri, 23 Oct 2015 09:59:04 +0000 (02:59 -0700)
commit1a28b2c7705702e7ecfbb701fbb5cf1b9018e4d9
tree953d8bbad9cc0bd690448e6858a3c40703cf66fb
parent74adacb23914c0c4ef6b497f3f9f1dda7e197d4b
amd-xgbe: Use wmb before updating current descriptor count

The code currently uses the lightweight dma_wmb barrier before updating
the current descriptor count. Under heavy load, the Tx cleanup routine
was seeing the updated current descriptor count before the updated
descriptor information. As a result, the Tx descriptor was being cleaned
up before it was used because it was not "owned" by the hardware yet,
resulting in a Tx queue hang.

Using the wmb barrier insures that the descriptor is updated before the
descriptor counter preventing the Tx queue hang. For extra insurance,
the Tx cleanup routine is changed to grab the current decriptor count on
entry and uses that initial value in the processing loop rather than
trying to chase the current value.

Signed-off-by: Tom Lendacky <thomas.lendacky@amd.com>
Tested-by: Christoffer Dall <christoffer.dall@linaro.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/amd/xgbe/xgbe-dev.c
drivers/net/ethernet/amd/xgbe/xgbe-drv.c