2011-09-14 08:57:41

by Rajkumar Manoharan

[permalink] [raw]
Subject: [PATCH] wireless: Reset beacon_found while updating regulatory

During the association, the regulatory is updated by country IE
that reaps the previously found beacons. The impact is that
after a STA disconnects *or* when for any reason a regulatory
domain change happens the beacon hint flag is not cleared
therefore preventing future beacon hints to be learned.
This is important as a regulatory domain change or a restore
of regulatory settings would set back the passive scan and no-ibss
flags on the channel. This is the right place to do this given that
it covers any regulatory domain change.

Cc: [email protected]
Reviewed-by: Luis R. Rodriguez <[email protected]>
Signed-off-by: Rajkumar Manoharan <[email protected]>
---
net/wireless/reg.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/net/wireless/reg.c b/net/wireless/reg.c
index a2b09c2..1bbd2a9 100644
--- a/net/wireless/reg.c
+++ b/net/wireless/reg.c
@@ -851,6 +851,7 @@ static void handle_channel(struct wiphy *wiphy,
return;
}

+ chan->beacon_found = false;
chan->flags = flags | bw_flags | map_regdom_flags(reg_rule->flags);
chan->max_antenna_gain = min(chan->orig_mag,
(int) MBI_TO_DBI(power_rule->max_antenna_gain));
--
1.7.6.1



2011-09-14 09:34:03

by Luis R. Rodriguez

[permalink] [raw]
Subject: Re: [PATCH] wireless: Reset beacon_found while updating regulatory

On Wed, Sep 14, 2011 at 1:58 AM, Rajkumar Manoharan
<[email protected]> wrote:
> During the association, the regulatory is updated by country IE
> that reaps the previously found beacons. The impact is that
> after a STA disconnects *or* when for any reason a regulatory
> domain change happens the beacon hint flag is not cleared
> therefore preventing future beacon hints to be learned.
> This is important as a regulatory domain change or a restore
> of regulatory settings would set back the passive scan and no-ibss
> flags on the channel. This is the right place to do this given that
> it covers any regulatory domain change.
>
> Cc: [email protected]
> Reviewed-by: Luis R. Rodriguez <[email protected]>
> Signed-off-by: Rajkumar Manoharan <[email protected]>

Acked-by: Luis R. Rodriguez <[email protected]>

Luis