net: smsc911x: Fix crash if loopback test fails
authorPavel Fedin <p.fedin@samsung.com>
Thu, 29 Oct 2015 06:45:22 +0000 (09:45 +0300)
committerDavid S. Miller <davem@davemloft.net>
Sun, 1 Nov 2015 17:05:24 +0000 (12:05 -0500)
commitb43c142f22236b37aa68faf23ac1bbd75f94c5b5
treeaa946f48003031fd07602ad6e5a54bccfe2271af
parent5cbb28a4bf65c7e4daa6c25b651fed8eb888c620
net: smsc911x: Fix crash if loopback test fails

On certain hardware in certain situations loopback test fails and the
driver gets removed. During mdiobus_unregister() instance of PHY driver
gets disposed. But by this time it has already been started using
phy_connect_direct().

PHY driver uses DELAYED_WORK in order to maintain its state. Attempting
to dispose the driver without calling phy_disconnect() causes deallocation
of DELAYED_WORK being active. This shortly causes a bad crash in timer
code.

The problem can be discovered by enabling CONFIG_DEBUG_OBJECTS_TIMERS and
CONFIG_DEBUG_OBJECTS_FREE

Signed-off-by: Pavel Fedin <p.fedin@samsung.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/smsc/smsc911x.c