82xx: FCC: Fixing a bug causing to FCC port lock-up (second try)
authorMartin Roth <martin.roth@motorolasolutions.com>
Sat, 23 Jan 2016 22:56:19 +0000 (00:56 +0200)
committerDavid S. Miller <davem@davemloft.net>
Mon, 25 Jan 2016 18:47:01 +0000 (10:47 -0800)
commit426f04684b30ced3632b29b41cb5c97dca500dca
tree3a77b6295ac9e177445e939045631e81ab033e8a
parent32b6170ca59ccf07d0e394561e54b2cd9726038c
82xx: FCC: Fixing a bug causing to FCC port lock-up (second try)

This is an additional patch to the one already submitted recently.
The previous patch was not complete, and the FCC port lock-up scenario
has been reproduced in lab.
I had an opportunity to check the current patch in lab and the FCC
port lock no longer freezes, while the previous patch still locks-up the
FCC port.
The current patch fixes a pointer arithmetic bug (second bug in the same
line), which leads FCC port lock-up during underrun/collision handling.
Within the tx_startup() function in mac-fcc.c, the address of last BD is
not calculated correctly. As a result of wrong calculation of the last BD
address, the next transmitted BD may be set to an area out of the transmit
BD ring. This actually causes to port lock-up and it is not recoverable.

Signed-off-by: Martin Roth <martin.roth@motorolasolutions.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/freescale/fs_enet/mac-fcc.c