On Tue, Apr 19, 2011 at 11:12:49AM +0530, Nishant Sarmukadam wrote: > From: Pradeep Nemavat <pnemavat@xxxxxxxxxxx> > > This is in preparation to support life time expiry of packets in the > hardware to avoid head-of-line blocking where a slow client can > hog a tx queue and affect the traffic to a faster client from the same > queue. Time stamp the packets in driver to allow dropping them in the > hardware if they are queued for more than 500ms. > > Since queues are not being stopped now, we need to be prepared for > a situation where packets hit the driver after the queues are full. > Drop all such packets in the driver itself. What this patch seems to do is: don't propagate queue start/stop indications to mac80211, and if a packet is handed to mwl8k for transmission while a TX queue is full, just drop it in the driver. (This doesn't seem to correspond with the commit message.) But since the driver still has an accurate idea of TX queue fullness (i.e. it's not as if at transmit time, you go and transparently reclaim TX queue entries), why do you need to start/stop mac80211 TX queues at all? Won't it still work fine without deleting that logic? -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
