diff options
author | Casey Leedom <leedom@chelsio.com> | 2010-06-29 08:53:39 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-06-30 16:57:11 -0400 |
commit | 64bb336c8f4de8b281d0d44f2ec2c900b9b28466 (patch) | |
tree | dbc0ed4027c3c3cafa4876f7bf10b5db26a4e7cb | |
parent | d6bebca92c663fb216c072193945946f3807ca7f (diff) |
cxgb4vf: Remove obsolete comment about the lack of a TX Timer Callback
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>
-rw-r--r-- | drivers/net/cxgb4vf/sge.c | 13 |
1 files changed, 1 insertions, 12 deletions
diff --git a/drivers/net/cxgb4vf/sge.c b/drivers/net/cxgb4vf/sge.c index f857d20e1d30..5c4a81dae19a 100644 --- a/drivers/net/cxgb4vf/sge.c +++ b/drivers/net/cxgb4vf/sge.c | |||
@@ -1301,18 +1301,7 @@ int t4vf_eth_xmit(struct sk_buff *skb, struct net_device *dev) | |||
1301 | * wait for acks to really free up the data the extra memory | 1301 | * wait for acks to really free up the data the extra memory |
1302 | * is even less. On the positive side we run the destructors | 1302 | * is even less. On the positive side we run the destructors |
1303 | * on the sending CPU rather than on a potentially different | 1303 | * on the sending CPU rather than on a potentially different |
1304 | * completing CPU, usually a good thing. We also run them | 1304 | * completing CPU, usually a good thing. |
1305 | * without holding our TX queue lock, unlike what | ||
1306 | * reclaim_completed_tx() would otherwise do. | ||
1307 | * | ||
1308 | * XXX Actually the above is somewhat incorrect since we don't | ||
1309 | * XXX yet have a periodic timer which reclaims TX Descriptors. | ||
1310 | * XXX What's our plan for this? | ||
1311 | * XXX | ||
1312 | * XXX Also, we don't currently have a TX Queue lock but | ||
1313 | * XXX that may be the result of not having any current | ||
1314 | * XXX asynchronous path for reclaiming completed TX | ||
1315 | * XXX Descriptors ... | ||
1316 | * | 1305 | * |
1317 | * Run the destructor before telling the DMA engine about the | 1306 | * Run the destructor before telling the DMA engine about the |
1318 | * packet to make sure it doesn't complete and get freed | 1307 | * packet to make sure it doesn't complete and get freed |