2011-06-10 15:02:56

by Paul Stewart

Subject: [PATCH/Resend] mac80211: Start monitor work on restart

Trigger connection monitor on resume from suspend. Since we
have been sleeping, there is reason to suspect that we might
not still be associated. The speed of detecting loss of
{connection,authentication} is worth the cost of the small
additional traffic at resume.

Signed-off-by: Paul Stewart <[email protected]>
---
net/mac80211/mlme.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/net/mac80211/mlme.c b/net/mac80211/mlme.c
index d595265..c188238 100644
--- a/net/mac80211/mlme.c
+++ b/net/mac80211/mlme.c
@@ -2206,6 +2206,7 @@ void ieee80211_sta_restart(struct ieee80211_sub_if_data *sdata)
add_timer(&ifmgd->chswitch_timer);
ieee80211_sta_reset_beacon_monitor(sdata);
ieee80211_restart_sta_timer(sdata);
+ ieee80211_queue_work(&sdata->local->hw, &sdata->u.mgd.monitor_work);
}
#endif

--
1.7.3.1

2011-06-19 08:12:06

by Johannes Berg

Subject: Re: [PATCH/Resend] mac80211: Start monitor work on restart

On Fri, 2011-06-10 at 07:00 -0800, Paul Stewart wrote:
> Trigger connection monitor on resume from suspend. Since we
> have been sleeping, there is reason to suspect that we might
> not still be associated. The speed of detecting loss of
> {connection,authentication} is worth the cost of the small
> additional traffic at resume.

Acked-by: Johannes Berg <[email protected]>

> Signed-off-by: Paul Stewart <[email protected]>
> ---
> net/mac80211/mlme.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/net/mac80211/mlme.c b/net/mac80211/mlme.c
> index d595265..c188238 100644
> --- a/net/mac80211/mlme.c
> +++ b/net/mac80211/mlme.c
> @@ -2206,6 +2206,7 @@ void ieee80211_sta_restart(struct ieee80211_sub_if_data *sdata)
> add_timer(&ifmgd->chswitch_timer);
> ieee80211_sta_reset_beacon_monitor(sdata);
> ieee80211_restart_sta_timer(sdata);
> + ieee80211_queue_work(&sdata->local->hw, &sdata->u.mgd.monitor_work);
> }
> #endif
>