Re: [PATCH 1/5] mac80211: always update the PM state of a peer on MGMT / DATA frames

[Luciano Coelho <luciano.coelho@intel.com>]

On Mon, 2017-10-30 at 09:48 +0100, Johannes Berg wrote:
> On Sun, 2017-10-29 at 11:51 +0200, Luca Coelho wrote:
> > From: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
> > 
> > The 2016 version of the spec is more generic about when the
> > AP should update the power management state of the peer:
> > the AP shall update the state based on any management or
> > data frames. This means that even non-bufferable management
> > frames should be looked at to update to maintain the power
> > management state of the peer.
> > 
> > This can avoid problematic cases for example if a station
> > disappears while being asleep and then re-appears. The AP
> > would remember it as in power save, but the Authentication
> > frame couldn't be used to set the peer as awake again.
> > Note that this issues wasn't really critical since at some
> > point (after the association) we would have removed the
> > station and created another one with all the states cleared.
> 
> IIRC this caused a hwsim test failure - is that addressed yet?

FTR, I'm sending a separate patch with the hwsim fix, as we agreed, as
part of the patchset I'm sending today.

--
Luca.