Commit 3c3070d7 authored by Maciej W. Rozycki's avatar Maciej W. Rozycki Committed by Jeff Garzik

[PATCH] 2.6.18: sb1250-mac: Phylib IRQ handling fixes

 This patch fixes a couple of problems discovered with interrupt handling
in the phylib core, namely:

1. The driver uses timer and workqueue calls, but does not include
   <linux/timer.h> nor <linux/workqueue.h>.

2. The driver uses schedule_work() for handling interrupts, but does not
   make sure any pending work scheduled thus has been completed before
   driver's structures get freed from memory.  This is especially
   important as interrupts may keep arriving if the line is shared with
   another PHY.

   The solution is to ignore phy_interrupt() calls if the reported device
   has already been halted and calling flush_scheduled_work() from
   phy_stop_interrupts() (but guarded with current_is_keventd() in case
   the function has been called through keventd from the MAC device's
   close call to avoid a deadlock on the netlink lock).
Signed-off-by: default avatarMaciej W. Rozycki <macro@linux-mips.org>

patch-mips-2.6.18-20060920-phy-irq-16
Signed-off-by: default avatarJeff Garzik <jeff@garzik.org>
parent 13df29f6
...@@ -7,6 +7,7 @@ ...@@ -7,6 +7,7 @@
* Author: Andy Fleming * Author: Andy Fleming
* *
* Copyright (c) 2004 Freescale Semiconductor, Inc. * Copyright (c) 2004 Freescale Semiconductor, Inc.
* Copyright (c) 2006 Maciej W. Rozycki
* *
* This program is free software; you can redistribute it and/or modify it * This program is free software; you can redistribute it and/or modify it
* under the terms of the GNU General Public License as published by the * under the terms of the GNU General Public License as published by the
...@@ -32,6 +33,8 @@ ...@@ -32,6 +33,8 @@
#include <linux/mii.h> #include <linux/mii.h>
#include <linux/ethtool.h> #include <linux/ethtool.h>
#include <linux/phy.h> #include <linux/phy.h>
#include <linux/timer.h>
#include <linux/workqueue.h>
#include <asm/io.h> #include <asm/io.h>
#include <asm/irq.h> #include <asm/irq.h>
...@@ -484,6 +487,9 @@ static irqreturn_t phy_interrupt(int irq, void *phy_dat) ...@@ -484,6 +487,9 @@ static irqreturn_t phy_interrupt(int irq, void *phy_dat)
{ {
struct phy_device *phydev = phy_dat; struct phy_device *phydev = phy_dat;
if (PHY_HALTED == phydev->state)
return IRQ_NONE; /* It can't be ours. */
/* The MDIO bus is not allowed to be written in interrupt /* The MDIO bus is not allowed to be written in interrupt
* context, so we need to disable the irq here. A work * context, so we need to disable the irq here. A work
* queue will write the PHY to disable and clear the * queue will write the PHY to disable and clear the
...@@ -577,6 +583,13 @@ int phy_stop_interrupts(struct phy_device *phydev) ...@@ -577,6 +583,13 @@ int phy_stop_interrupts(struct phy_device *phydev)
if (err) if (err)
phy_error(phydev); phy_error(phydev);
/*
* Finish any pending work; we might have been scheduled
* to be called from keventd ourselves, though.
*/
if (!current_is_keventd())
flush_scheduled_work();
free_irq(phydev->irq, phydev); free_irq(phydev->irq, phydev);
return err; return err;
...@@ -603,7 +616,8 @@ static void phy_change(void *data) ...@@ -603,7 +616,8 @@ static void phy_change(void *data)
enable_irq(phydev->irq); enable_irq(phydev->irq);
/* Reenable interrupts */ /* Reenable interrupts */
err = phy_config_interrupt(phydev, PHY_INTERRUPT_ENABLED); if (PHY_HALTED != phydev->state)
err = phy_config_interrupt(phydev, PHY_INTERRUPT_ENABLED);
if (err) if (err)
goto irq_enable_err; goto irq_enable_err;
...@@ -624,18 +638,24 @@ void phy_stop(struct phy_device *phydev) ...@@ -624,18 +638,24 @@ void phy_stop(struct phy_device *phydev)
if (PHY_HALTED == phydev->state) if (PHY_HALTED == phydev->state)
goto out_unlock; goto out_unlock;
if (phydev->irq != PHY_POLL) { phydev->state = PHY_HALTED;
/* Clear any pending interrupts */
phy_clear_interrupt(phydev);
if (phydev->irq != PHY_POLL) {
/* Disable PHY Interrupts */ /* Disable PHY Interrupts */
phy_config_interrupt(phydev, PHY_INTERRUPT_DISABLED); phy_config_interrupt(phydev, PHY_INTERRUPT_DISABLED);
}
phydev->state = PHY_HALTED; /* Clear any pending interrupts */
phy_clear_interrupt(phydev);
}
out_unlock: out_unlock:
spin_unlock(&phydev->lock); spin_unlock(&phydev->lock);
/*
* Cannot call flush_scheduled_work() here as desired because
* of rtnl_lock(), but PHY_HALTED shall guarantee phy_change()
* will not reenable interrupts.
*/
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment