[kernel/f16] Add two patches to fix mac80211 issues (rhbz 731365)
Josh Boyer
jwboyer at fedoraproject.org
Mon Nov 7 17:31:11 UTC 2011
commit 12413987823b6dfbef7ff61c61d4d6d60969bd2b
Author: Josh Boyer <jwboyer at redhat.com>
Date: Mon Nov 7 12:30:59 2011 -0500
Add two patches to fix mac80211 issues (rhbz 731365)
kernel.spec | 13 +++++-
...0211-config-hw-when-going-back-on-channel.patch | 40 ++++++++++++++++++
mac80211-fix-remain_off_channel-regression.patch | 44 ++++++++++++++++++++
3 files changed, 96 insertions(+), 1 deletions(-)
---
diff --git a/kernel.spec b/kernel.spec
index 42279c0..a1138f3 100644
--- a/kernel.spec
+++ b/kernel.spec
@@ -51,7 +51,7 @@ Summary: The Linux kernel
# For non-released -rc kernels, this will be prepended with "0.", so
# for example a 3 here will become 0.3
#
-%global baserelease 9
+%global baserelease 10
%global fedora_build %{baserelease}
# base_sublevel is the kernel version we're starting with and patching
@@ -774,6 +774,10 @@ Patch21060: elantech.patch
#rhbz 748210
Patch21061: ideapad-Check-if-acpi-already-handle-backlight.patch
+#rhbz 731365
+Patch21062: mac80211-fix-remain_off_channel-regression.patch
+Patch21063: mac80211-config-hw-when-going-back-on-channel.patch
+
%endif
BuildRoot: %{_tmppath}/kernel-%{KVERREL}-root
@@ -1428,6 +1432,10 @@ ApplyPatch elantech.patch
#rhbz 748210
ApplyPatch ideapad-Check-if-acpi-already-handle-backlight.patch
+#rhbz 731365
+ApplyPatch mac80211-fix-remain_off_channel-regression.patch
+ApplyPatch mac80211-config-hw-when-going-back-on-channel.patch
+
# END OF PATCH APPLICATIONS
%endif
@@ -2129,6 +2137,9 @@ fi
# and build.
%changelog
+* Mon Nov 07 2011 Josh Boyer <jwboyer at redhat.com>
+- Add two patches to fix mac80211 issues (rhbz 731365)
+
* Thu Nov 03 2011 Josh Boyer <jwboyer at redhat.com>
- Add commits queued for 3.2 for elantech driver (rhbz 728607)
- Fix crash when setting brightness via Fn keys on ideapads (rhbz 748210)
diff --git a/mac80211-config-hw-when-going-back-on-channel.patch b/mac80211-config-hw-when-going-back-on-channel.patch
new file mode 100644
index 0000000..3510827
--- /dev/null
+++ b/mac80211-config-hw-when-going-back-on-channel.patch
@@ -0,0 +1,40 @@
+From 776308dd5fa8a41e8bbb79818a66bd68a7db757e Mon Sep 17 00:00:00 2001
+From: Eliad Peller <eliad at wizery.com>
+Date: Thu, 20 Oct 2011 19:05:50 +0200
+Subject: [PATCH] mac80211: config hw when going back on-channel
+
+When going back on-channel, we should reconfigure
+the hw iff the hardware is not already configured
+to the operational channel.
+
+Signed-off-by: Eliad Peller <eliad at wizery.com>
+Cc: stable at kernel.org # 2.6.39+
+Signed-off-by: John W. Linville <linville at tuxdriver.com>
+---
+ net/mac80211/work.c | 3 +--
+ 1 files changed, 1 insertions(+), 2 deletions(-)
+
+diff --git a/net/mac80211/work.c b/net/mac80211/work.c
+index bf5be22..6c53b6d 100644
+--- a/net/mac80211/work.c
++++ b/net/mac80211/work.c
+@@ -1091,7 +1091,6 @@ static void ieee80211_work_work(struct work_struct *work)
+ }
+
+ if (!remain_off_channel && local->tmp_channel) {
+- bool on_oper_chan = ieee80211_cfg_on_oper_channel(local);
+ local->tmp_channel = NULL;
+ /* If tmp_channel wasn't operating channel, then
+ * we need to go back on-channel.
+@@ -1101,7 +1100,7 @@ static void ieee80211_work_work(struct work_struct *work)
+ * we still need to do a hardware config. Currently,
+ * we cannot be here while scanning, however.
+ */
+- if (ieee80211_cfg_on_oper_channel(local) && !on_oper_chan)
++ if (!ieee80211_cfg_on_oper_channel(local))
+ ieee80211_hw_config(local, 0);
+
+ /* At the least, we need to disable offchannel_ps,
+--
+1.7.6.2
+
diff --git a/mac80211-fix-remain_off_channel-regression.patch b/mac80211-fix-remain_off_channel-regression.patch
new file mode 100644
index 0000000..9f5d8f6
--- /dev/null
+++ b/mac80211-fix-remain_off_channel-regression.patch
@@ -0,0 +1,44 @@
+From 682ba5a44516529b29fc780c055e06104d36e31e Mon Sep 17 00:00:00 2001
+From: Eliad Peller <eliad at wizery.com>
+Date: Thu, 20 Oct 2011 19:05:49 +0200
+Subject: [PATCH] mac80211: fix remain_off_channel regression
+
+The offchannel code is currently broken - we should
+remain_off_channel if the work was started, and
+the work's channel and channel_type are the same
+as local->tmp_channel and local->tmp_channel_type.
+
+However, if wk->chan_type and local->tmp_channel_type
+coexist (e.g. have the same channel type), we won't
+remain_off_channel.
+
+This behavior was introduced by commit da2fd1f
+("mac80211: Allow work items to use existing
+channel type.")
+
+Tested-by: Ben Greear <greearb at candelatech.com>
+Signed-off-by: Eliad Peller <eliad at wizery.com>
+Cc: stable at kernel.org # 2.6.39+
+Signed-off-by: John W. Linville <linville at tuxdriver.com>
+---
+ net/mac80211/work.c | 4 ++--
+ 1 files changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/net/mac80211/work.c b/net/mac80211/work.c
+index 94472eb..bf5be22 100644
+--- a/net/mac80211/work.c
++++ b/net/mac80211/work.c
+@@ -1084,8 +1084,8 @@ static void ieee80211_work_work(struct work_struct *work)
+ continue;
+ if (wk->chan != local->tmp_channel)
+ continue;
+- if (ieee80211_work_ct_coexists(wk->chan_type,
+- local->tmp_channel_type))
++ if (!ieee80211_work_ct_coexists(wk->chan_type,
++ local->tmp_channel_type))
+ continue;
+ remain_off_channel = true;
+ }
+--
+1.7.6.2
+
More information about the scm-commits
mailing list