lkml.org 
[lkml]   [2005]   [Feb]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[11/14] Orinoco driver updates - delay Tx wake
    Delay netif_wake_queue() until the packet has actually been
    transmitted, rather than just when the firmware has copied it into its
    internal buffers. This seems to prevent problems on some Intersil
    firmware versions (I suspect the problems were caused by the
    firmware's buffers filling up).

    Signed-off-by: David Gibson <hermes@gibson.dropbear.id.au>

    Index: working-2.6/drivers/net/wireless/orinoco.c
    ===================================================================
    --- working-2.6.orig/drivers/net/wireless/orinoco.c 2005-02-18 12:48:30.523655896 +1100
    +++ working-2.6/drivers/net/wireless/orinoco.c 2005-02-18 12:58:09.407652152 +1100
    @@ -901,8 +901,6 @@
    printk(KERN_WARNING "%s: Allocate event on unexpected fid (%04X)\n",
    dev->name, fid);
    return;
    - } else {
    - netif_wake_queue(dev);
    }

    hermes_write_regn(hw, ALLOCFID, DUMMY_FID);
    @@ -915,6 +913,8 @@

    stats->tx_packets++;

    + netif_wake_queue(dev);
    +
    hermes_write_regn(hw, TXCOMPLFID, DUMMY_FID);
    }

    @@ -941,6 +941,7 @@

    stats->tx_errors++;

    + netif_wake_queue(dev);
    hermes_write_regn(hw, TXCOMPLFID, DUMMY_FID);
    }

    --
    David Gibson | I'll have my music baroque, and my code
    david AT gibson.dropbear.id.au | minimalist. NOT _the_ _other_ _way_
    | _around_!
    http://www.ozlabs.org/people/dgibson
    -
    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: 2005-03-22 14:10    [W:2.432 / U:0.048 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site