On Sun, 2008-12-14 at 22:50 -0500, Bob Copeland wrote: > This patch introduces suspend and resume callbacks to mac80211, > allowing mac80211 to quiesce its state (bringing down interfaces, > removing keys, etc) in preparation for suspend. cfg80211 will call > the suspend hook before the device suspend, and resume hook after > the device resume. Looks good to me. > Signed-off-by: Bob Copeland <me@xxxxxxxxxxxxxxx> Acked-by: Johannes Berg <johannes@xxxxxxxxxxxxxxxx> > +int __ieee80211_suspend(struct ieee80211_hw *hw) > +{ > + struct ieee80211_local *local = hw_to_local(hw); > + struct ieee80211_sub_if_data *sdata; > + struct ieee80211_if_init_conf conf; > + struct sta_info *sta; > + > + flush_workqueue(local->hw.workqueue); > + > + /* disable keys */ > + list_for_each_entry(sdata, &local->interfaces, list) > + ieee80211_disable_keys(sdata); > + > + /* remove STAs */ > + list_for_each_entry(sta, &local->sta_list, list) { > + > + if (local->ops->sta_notify) { One could reorder this to: if (local->ops->sta_notify) { list_for_each_entry(sta, &local->sta_list, list) { and the remaining code stays, but it really doesn't matter in this code path. Does it work now? Did you ever figure out why it failed occasionally before? johannes
Attachment:
signature.asc
Description: This is a digitally signed message part