niu: Fix readq implementation when architecture does not provide one.
authorDavid S. Miller <davem@davemloft.net>
Wed, 12 Nov 2008 22:32:54 +0000 (14:32 -0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 12 Nov 2008 22:32:54 +0000 (14:32 -0800)
commite23a59e1ca6d177a57a7791b3629db93ff1d9813
treefa96090d8b055468cc249c133e0980a3ba462187
parentd35aac10eb7bcb3b80bef16b60844af0313f47f7
niu: Fix readq implementation when architecture does not provide one.

This fixes a TX hang reported by Jesper Dangaard Brouer.

When an architecutre cannot provide a fully functional
64-bit atomic readq/writeq, the driver must implement
it's own.  This is because only the driver can say whether
doing something like using two 32-bit reads to implement
the full 64-bit read will actually work properly.

In particular one of the issues is whether the top 32-bits
or the bottom 32-bits of the 64-bit register should be read
first.  There could be side effects, and in fact that is
exactly the problem here.

The TX_CS register has counters in the upper 32-bits and
state bits in the lower 32-bits.  A read clears the state
bits.

We would read the counter half before the state bit half.
That first read would clear the state bits, and then the
driver thinks that no interrupts are pending because the
interrupt indication state bits are seen clear every time.

Fix this by reading the bottom half before the upper half.

Tested-by: Jesper Dangaard Brouer <jdb@comx.dk>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/niu.c