ucc_geth: Fix hangs after switching from full to half duplex
authorAnton Vorontsov <avorontsov@ru.mvista.com>
Thu, 10 Sep 2009 11:48:12 +0000 (11:48 +0000)
committerDavid S. Miller <davem@davemloft.net>
Fri, 11 Sep 2009 19:54:45 +0000 (12:54 -0700)
commit864fdf884e82bacbe8ca5e93bd43393a61d2e2b4
tree53a45c507d77eb8cc33cd9226ebeac947119ede3
parent7de8ee787e8e10adaf5635bffab4ee19a7558afb
ucc_geth: Fix hangs after switching from full to half duplex

MPC8360 QE UCC ethernet controllers hang when changing link duplex
under a load (a bit of NFS activity is enough).

  PHY: mdio@e0102120:00 - Link is Up - 1000/Full
  sh-3.00# ethtool -s eth0 speed 100 duplex half autoneg off
  PHY: mdio@e0102120:00 - Link is Down
  PHY: mdio@e0102120:00 - Link is Up - 100/Half
  NETDEV WATCHDOG: eth0 (ucc_geth): transmit queue 0 timed out
  ------------[ cut here ]------------
  Badness at c01fcbd0 [verbose debug info unavailable]
  NIP: c01fcbd0 LR: c01fcbd0 CTR: c0194e44
  ...

The cure is to disable the controller before changing speed/duplex
and enable it afterwards.

Though, disabling the controller might take quite a while, so we
better not grab any spinlocks in adjust_link(). Instead, we quiesce
the driver's activity, and only then disable the controller.

Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ucc_geth.c