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@xxxxxxxxx> > > > > 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.