Search Linux Wireless

Re: [PATCH] mac80211: add ieee80211_set_dyn_ps_timeout()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Eliad Peller <eliad@xxxxxxxxxx> writes:

> AFAIU from the coex guys, the scenario is something like this: upon
> coex activity, the fw might delay its rx and tx paths. this means
> that the fw might get a frame within the 100ms of the dyn ps, but
> delay its processing and pass it up to the driver only later. this
> will cause redundant psm enter (after 100ms) and psm exit (after the
> fw passed the packet). i'm not sure about the exact effect during
> coex operation, but eventually these psm enter/exit affect the
> throughput.
>
> another point here, is that during a specific period (during auto_mode
> on), there might or might not be coex activity. thus, we can't just
> disable dyn_ps, as it will hurt throughput (when there is no coex
> activity).

IIRC Juuso added ieee80211_enable/disable_dyn_ps() to make it possible
use BT COEX with wl12xx. Now you are saying that you actually want the
opposite? I'm confused now. 

But nevertheless I'm not still convinced that doing all this in
mac80211 is the right thing. Especially that this seems to be very
wl12xx specific, right?

> bottom line - i'm not sure about all the details, but according to
> their tests - it does improve the throughput.

Firmware engineers are notarious in making quick hacks to improve one
special case and not thinking about the big picture ;)

-- 
Kalle Valo
--
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


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux