mac80211: Check for queued frames before entering power save.
In a highly noisy environment, the tx rate of the driver drops and the application slows down since it has not yet received ACKs for the frames already queued in the hardware. Since this ACK may take more than 100ms, stopping the dev queues for entering PS at this stage breaks applications, WMM test cases in my testing. If there are frames already pending in the tx queue, postponing the PS logic helps to avoid redundant queue stops. When power save is enabled by default and in a noisy environment, this API certainly helps in improving the average throughput. Signed-off-by: Vivek Natarajan <vnatarajan@atheros.com> Signed-off-by: John W. Linville <linville@tuxdriver.com>
This commit is contained in:

committed by
John W. Linville

parent
6fc3ba9999
commit
e8306f9894
@@ -761,15 +761,16 @@ void ieee80211_dynamic_ps_enable_work(struct work_struct *work)
|
||||
if ((local->hw.flags & IEEE80211_HW_PS_NULLFUNC_STACK) &&
|
||||
(!(ifmgd->flags & IEEE80211_STA_NULLFUNC_ACKED))) {
|
||||
netif_tx_stop_all_queues(sdata->dev);
|
||||
/*
|
||||
* Flush all the frames queued in the driver before
|
||||
* going to power save
|
||||
*/
|
||||
drv_flush(local, false);
|
||||
ieee80211_send_nullfunc(local, sdata, 1);
|
||||
|
||||
/* Flush once again to get the tx status of nullfunc frame */
|
||||
drv_flush(local, false);
|
||||
if (drv_tx_frames_pending(local))
|
||||
mod_timer(&local->dynamic_ps_timer, jiffies +
|
||||
msecs_to_jiffies(
|
||||
local->hw.conf.dynamic_ps_timeout));
|
||||
else {
|
||||
ieee80211_send_nullfunc(local, sdata, 1);
|
||||
/* Flush to get the tx status of nullfunc frame */
|
||||
drv_flush(local, false);
|
||||
}
|
||||
}
|
||||
|
||||
if (!((local->hw.flags & IEEE80211_HW_REPORTS_TX_ACK_STATUS) &&
|
||||
|
Reference in New Issue
Block a user