>
> On Tue, Nov 4, 2014 at 2:10 PM, Emmanuel Grumbach
> <emmanuel.grumbach@xxxxxxxxx> wrote:
> > From: Andrei Otcheretianski <andrei.otcheretianski@xxxxxxxxx>
> >
> > Deliver up to 128 frames during SP instead of 8 if unlimited max SP is
> > specified during association.
> >
> > Signed-off-by: Andrei Otcheretianski <andrei.otcheretianski@xxxxxxxxx>
> > Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@xxxxxxxxx>
> > ---
> > 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.
��.n��������+%������w��{.n�����{���zW����ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f
