Re: mac80211 breaks suspend to disk

[Maxim Levitsky <maximlevitsky@gmail.com>]

On Wed, 2009-11-18 at 22:03 +0100, Johannes Berg wrote: 
> On Wed, 2009-11-18 at 22:39 +0200, Maxim Levitsky wrote:
> 
> > I forgot to pay attention to few lines before the oops message:
> > 
> > 
> > [  331.307139] No probe response from AP 00:1b:9e:d8:77:02 after 500ms, try 1
> > [  331.320121] ------------[ cut here ]------------
> > [  331.334935] WARNING: at /home/maxim/software/kernel/linux-2.6/kernel/timer.c:791 add_timer+0x36/0x40()
> > [  331.347374] Hardware name: Aspire 5720     
> > .....
> 
> Thanks.
> 
> > This reveals that somehow the ieee80211_sta_work manages to run before
> > ieee80211_sta_restart and it sets the timer....
> 
> Indeed.
> 
> > looking at ieee80211_reconfig it seems suspicious that
> > 'local->suspended = false;' is set so early.
> > In fact it is set again to false just prior to 
> > 
> > list_for_each_entry(sdata, &local->interfaces, list) {....
> > 
> > So, I would suspect some lines in this function trigger the work
> > queue.
> 
> No, I doubt it, it's most likely just receiving a frame to trigger the
> function -- ieee80211_sta_rx_mgmt queues the work. If we were still
> suspended then we'd run into an error situation, like the comment in
> ieee80211_reconfig explains ...
> 
> Catch-22.
> 
> Then again, we drop RX frames while local->suspended is true, and I
> guess we don't really care about missing a few more frames when we're
> just waking up, so it should be OK to keep suspended == true while doing
> the startup.
> 
> EXCEPT drivers are going to start using ieee80211_queue_work() already
> then from the callbacks, which we must allow.
> 
> So ... maybe this?


This will likely fix this problem.
I will run my suspend/resume loop for a while and let you know if it
works.

Best regards,
Maxim Levitsky