lkml.org 
[lkml]   [2007]   [Aug]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Date
    Subject[2.6.20.16 review 28/28] sky2: workaround for lost IRQ
    This patch restores a couple of workarounds from 2.6.16:
    * restart transmit moderation timer in case it expires during IRQ routine
    * default to having 10 HZ watchdog timer.
    At this point it more important not to hang than to worry about the
    power cost.

    Signed-off-by: Stephen Hemminger <shemminger@linux-foundation.org>
    Cc: Jeff Garzik <jeff@garzik.org>
    Signed-off-by: Chris Wright <chrisw@sous-sol.org>
    Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
    ---
    drivers/net/sky2.c | 9 ++++++++-
    1 files changed, 8 insertions(+), 1 deletions(-)

    diff --git a/drivers/net/sky2.c b/drivers/net/sky2.c
    index 38e75cf..aec8c59 100644
    --- a/drivers/net/sky2.c
    +++ b/drivers/net/sky2.c
    @@ -95,7 +95,7 @@ static int disable_msi = 0;
    module_param(disable_msi, int, 0);
    MODULE_PARM_DESC(disable_msi, "Disable Message Signaled Interrupt (MSI)");

    -static int idle_timeout = 0;
    +static int idle_timeout = 100;
    module_param(idle_timeout, int, 0);
    MODULE_PARM_DESC(idle_timeout, "Watchdog timer for lost interrupts (ms)");

    @@ -2341,6 +2341,13 @@ static int sky2_poll(struct net_device *dev0, int *budget)

    work_done = sky2_status_intr(hw, work_limit);
    if (work_done < work_limit) {
    + /* Bug/Errata workaround?
    + * Need to kick the TX irq moderation timer.
    + */
    + if (sky2_read8(hw, STAT_TX_TIMER_CTRL) == TIM_START) {
    + sky2_write8(hw, STAT_TX_TIMER_CTRL, TIM_STOP);
    + sky2_write8(hw, STAT_TX_TIMER_CTRL, TIM_START);
    + }
    netif_rx_complete(dev0);

    sky2_read32(hw, B0_Y2_SP_LISR);
    --
    1.5.2.4
    --
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2007-08-11 21:07    [W:4.392 / U:0.488 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site