> > On Tue, Nov 4, 2014 at 2:10 PM, Emmanuel Grumbach > <[email protected]> wrote: > > From: Andrei Otcheretianski <[email protected]> > > > > Deliver up to 128 frames during SP instead of 8 if unlimited max SP is > > specified during association. > > > > Signed-off-by: Andrei Otcheretianski <[email protected]> > > Signed-off-by: Emmanuel Grumbach <[email protected]> > > --- > > net/mac80211/sta_info.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/net/mac80211/sta_info.c b/net/mac80211/sta_info.c index > > de494df..3352741 100644 > > --- a/net/mac80211/sta_info.c > > +++ b/net/mac80211/sta_info.c > > @@ -1531,7 +1531,7 @@ void ieee80211_sta_ps_deliver_uapsd(struct > sta_info *sta) > > break; > > case 0: > > /* XXX: what is a good value? */ > > - n_frames = 8; > > + n_frames = 128; > > break; > > } > What is criteria for deciding the threshold? > Any specific use case for 128?
Well - this is why this commit doesn't remove the TODO :) The reason for this change is a certification issue in WMM-PS with P2P GO (NoA cutting short a SP). I guess the proper way to handle that is to make it configurable.
