UbiquitousOS
/
OpenCloudOS-Kernel
mirror of https://gitee.com/OpenCloudOS/OpenCloudOS-Kernel.git
11
0
Fork 0
Code Issues Projects Releases Wiki Activity

net: stmmac: Fix crash observed if PHY does not support EEE 

If the PHY does not support EEE mode, then a crash is observed when the
ethernet interface is enabled. The crash occurs, because if the PHY does
not support EEE, then although the EEE timer is never configured, it is
still marked as enabled and so the stmmac ethernet driver is still
trying to update the timer by calling mod_timer(). This triggers a BUG()
in the mod_timer() because we are trying to update a timer when there is
no callback function set because timer_setup() was never called for this
timer.

The problem is caused because we return true from the function
stmmac_eee_init(), marking the EEE timer as enabled, even when we have
not configured the EEE timer. Fix this by ensuring that we return false
if the PHY does not support EEE and hence, 'eee_active' is not set.

Fixes: 74371272f9 ("net: stmmac: Convert to phylink and remove phylib logic")
Signed-off-by: Jon Hunter <jonathanh@nvidia.com>
Tested-by: Thierry Reding <treding@nvidia.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
Jon Hunter 2019-06-26 11:23:22 +01:00 committed by David S. Miller
parent 0867bb9768
commit 177d935a13
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
View File

@ -398,10 +398,12 @@ bool stmmac_eee_init(struct stmmac_priv *priv)
mutex_lock(&priv->lock); mutex_lock(&priv->lock);
/* Check if it needs to be deactivated */ /* Check if it needs to be deactivated */
if (!priv->eee_active && priv->eee_enabled) { if (!priv->eee_active) {
netdev_dbg(priv->dev, "disable EEE\n"); if (priv->eee_enabled) {
del_timer_sync(&priv->eee_ctrl_timer); netdev_dbg(priv->dev, "disable EEE\n");
stmmac_set_eee_timer(priv, priv->hw, 0, tx_lpi_timer); del_timer_sync(&priv->eee_ctrl_timer);
stmmac_set_eee_timer(priv, priv->hw, 0, tx_lpi_timer);
}
mutex_unlock(&priv->lock); mutex_unlock(&priv->lock);
return false; return false;
} }