Re: [PATCH 2/2] mac80211: Don't go off-channel in work_work unless needed.

[Johannes Berg <johannes@sipsolutions.net>]

On Wed, 2010-09-29 at 11:03 -0700, greearb@candelatech.com wrote:
> From: Ben Greear <greearb@candelatech.com>
> 
> If the work to be done is on the current active channel, then
> do not call the offchannel and return-to-channel logic.
> 
> Signed-off-by: Ben Greear <greearb@candelatech.com>
> ---
> :100644 100644 ae344d1... 1f2673d... M	net/mac80211/work.c
>  net/mac80211/work.c |   28 ++++++++++++++++++++--------
>  1 files changed, 20 insertions(+), 8 deletions(-)
> 
> diff --git a/net/mac80211/work.c b/net/mac80211/work.c
> index ae344d1..1f2673d 100644
> --- a/net/mac80211/work.c
> +++ b/net/mac80211/work.c
> @@ -873,6 +873,7 @@ static void ieee80211_work_work(struct work_struct *work)
>  	LIST_HEAD(free_work);
>  	enum work_action rma;
>  	bool remain_off_channel = false;
> +	bool went_off_channel = false;

Heh, this is not going to work ... the return may be done in a separate
invocation of the function. Would have been nice to test this ;-)

johannes