Chromium Code Reviews

Issue 3427005: CHROMIUMOS: ath9k: fix enabling ANI / tx monitor after bg scan (Closed)

Created:
10 years, 3 months ago by Paul Stewart
Modified:
9 years, 7 months ago
Reviewers:
Sam Leffler
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, sleffler+cc_chromium.org, Olof Johansson, vb
Base URL:
ssh://gitrw.chromium.org/kernel.git
Visibility:
Public.

Description

CHROMIUMOS: ath9k: fix enabling ANI / tx monitor after bg scan ath9k's entire logic with SC_OP_SCANNING is incorrect due to the way mac80211 currently implements the scan complete callback and we handle it in ath9k. This patch removes the flag completely in preference for the SC_OP_OFFCHANNEL which is really what we wanted. The scanning flag was used to ensure we reset ANI to the old values when we go back to the home channel, but if we are offchannel we use some defaults. The flag was also used to re-enable the TX monitor. Without this patch we simply never re-enabled ANI and the TX monitor after going offchannel. This means that after one background scan we are prone to noise issues and if we had a TX hang we would not recover. To get this to work properly we must enable ANI after we have configured the beacon timers, otherwise hardware acts really oddly. This patch has stable fixes which apply down to [2.6.36+], there *may* be a to fix this on older kernels but requires a bit of work since this patch relies on the new mac80211 flag IEEE80211_CONF_OFFCHANNEL which was introduced as of 2.6.36. Cc: stable@kernel.org Cc: Paul Stewart <pstew@google.com>; Cc: Amod Bodas <amod.bodas@atheros.com>; Signed-off-by: Luis R. Rodriguez <lrodriguez@atheros.com>; BUG=chromium-os:5715 TEST=Walkaround tests + SecMat and MatFunc testbed runs Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=30e945c

Patch Set 1 #

Unified diffs Side-by-side diffs Stats (+5 lines, -10 lines)
M chromeos/compat-wireless/drivers/net/wireless/ath/ath9k/ath9k.h View 1 chunk +0 lines, -1 line 0 comments
M chromeos/compat-wireless/drivers/net/wireless/ath/ath9k/main.c View 3 chunks +3 lines, -7 lines 0 comments
M chromeos/compat-wireless/drivers/net/wireless/ath/ath9k/recv.c View 2 chunks +2 lines, -2 lines 0 comments

Messages

Total messages: 2 (0 generated)
Paul Stewart
10 years, 3 months ago (2010-09-15 18:03:48 UTC) #1
Sam Leffler
10 years, 3 months ago (2010-09-17 00:46:22 UTC) #2
LGTM

Powered by Google App Engine