> - netif_queue_stopped need not be called inside qdisc_restart as > it has been called already in qdisc_run() before the first skb > is sent, and in __qdisc_run() after each intermediate skb is > sent (note : we are the only sender, so the queue cannot get > stopped while the tx lock was got in the ~LLTX case).
I somewhat disagree here. The underlying driver can conceivably stop the device queue even if the stack holds the queue lock during an interrupt to clean Tx descriptors, and it finds it's out of them or needs to grab the device for whatever reason. Granted this is a corner case, and the net effect would be a simple requeue of the skb, but checking the status of the queue at the last possible moment before entering the driver could alleviate the requeue in the time between ->dequeue() from the qdisc, and hard_start_xmit() if an event like I mentioned happened. I'm ok with it either way, especially since this is a corner case. But it does need to be considered that it can happen. Cheers, -PJ Waskiewicz - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
