[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