cxgb4vf: Remove obsolete comment about the lack of a TX Timer Callback
authorCasey Leedom <leedom@chelsio.com>
Tue, 29 Jun 2010 12:53:39 +0000 (12:53 +0000)
committerDavid S. Miller <davem@davemloft.net>
Wed, 30 Jun 2010 20:57:11 +0000 (13:57 -0700)
Remove obsolete comment about the lack of a TX Timer Callback -- which
we now _do_ have ...

Signed-off-by: Casey Leedom <leedom@chelsio.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/cxgb4vf/sge.c

index f857d20e1d3023ab8ae2b856692315b76efeabd7..5c4a81dae19a11b7d307c3c42694a03318058eac 100644 (file)
@@ -1301,18 +1301,7 @@ int t4vf_eth_xmit(struct sk_buff *skb, struct net_device *dev)
                 * wait for acks to really free up the data the extra memory
                 * is even less.  On the positive side we run the destructors
                 * on the sending CPU rather than on a potentially different
-                * completing CPU, usually a good thing.  We also run them
-                * without holding our TX queue lock, unlike what
-                * reclaim_completed_tx() would otherwise do.
-                *
-                * XXX Actually the above is somewhat incorrect since we don't
-                * XXX yet have a periodic timer which reclaims TX Descriptors.
-                * XXX What's our plan for this?
-                * XXX
-                * XXX Also, we don't currently have a TX Queue lock but
-                * XXX that may be the result of not having any current
-                * XXX asynchronous path for reclaiming completed TX
-                * XXX Descriptors ...
+                * completing CPU, usually a good thing.
                 *
                 * Run the destructor before telling the DMA engine about the
                 * packet to make sure it doesn't complete and get freed