[linux] 01/01: rt2x00: fix monitor mode regression (regression in 4.2)

debian-kernel at lists.debian.org debian-kernel at lists.debian.org
Mon Feb 1 17:16:24 UTC 2016


This is an automated email from the git hooks/post-receive script.

benh pushed a commit to branch sid
in repository linux.

commit 3180443250b377ceb6e6eb2997d07f014dca270c
Author: Ben Hutchings <ben at decadent.org.uk>
Date:   Mon Feb 1 17:16:15 2016 +0000

    rt2x00: fix monitor mode regression (regression in 4.2)
---
 debian/changelog                                   |   1 +
 .../all/rt2x00-fix-monitor-mode-regression.patch   | 149 +++++++++++++++++++++
 debian/patches/series                              |   1 +
 3 files changed, 151 insertions(+)

diff --git a/debian/changelog b/debian/changelog
index 3d92a19..f7ec1b5 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -211,6 +211,7 @@ linux (4.3.5-1) UNRELEASED; urgency=medium
   [ Ben Hutchings ]
   * fuse: break infinite loop in fuse_fill_write_pages() (CVE-2015-8785)
   * SCSI: fix crashes in sd and sr runtime PM (Closes: #801925)
+  * rt2x00: fix monitor mode regression (regression in 4.2)
 
   [ Salvatore Bonaccorso ]
   * netfilter: nf_nat_redirect: add missing NULL pointer check (CVE-2015-8787)
diff --git a/debian/patches/bugfix/all/rt2x00-fix-monitor-mode-regression.patch b/debian/patches/bugfix/all/rt2x00-fix-monitor-mode-regression.patch
new file mode 100644
index 0000000..c96ef41
--- /dev/null
+++ b/debian/patches/bugfix/all/rt2x00-fix-monitor-mode-regression.patch
@@ -0,0 +1,149 @@
+From: Eli Cooper <elicooper at gmx.com>
+Date: Mon, 18 Jan 2016 19:30:19 +0800
+Subject: rt2x00: fix monitor mode regression
+Origin: https://git.kernel.org/cgit/linux/kernel/git/wireless/wireless-testing.git/commit?id=262c741e0825b29447a9e53b6582afd6b14c3706
+
+Since commit df1404650ccb ("mac80211: remove support for IFF_PROMISC")
+monitor mode for rt2x00 has been made effectively useless because the
+hardware filter is configured to drop packets whose intended recipient is
+not the device, regardless of the presence of monitor mode interfaces.
+
+This patch fixes this regression by adding explicit monitor mode support,
+and by configuring the hardware filter accordingly.
+
+Signed-off-by: Eli Cooper <elicooper at gmx.com>
+Acked-by: Stanislaw Gruszka <sgruszka at redhat.com>
+Signed-off-by: Kalle Valo <kvalo at codeaurora.org>
+[bwh: Backported to 4.3: adjust filenames]
+---
+ drivers/net/wireless/rt2x00/rt2400pci.c    | 4 +++-
+ drivers/net/wireless/rt2x00/rt2500pci.c    | 4 +++-
+ drivers/net/wireless/rt2x00/rt2500usb.c    | 4 +++-
+ drivers/net/wireless/rt2x00/rt2800lib.c    | 3 ++-
+ drivers/net/wireless/rt2x00/rt2x00.h       | 1 +
+ drivers/net/wireless/rt2x00/rt2x00config.c | 5 +++++
+ drivers/net/wireless/rt2x00/rt2x00mac.c    | 5 -----
+ drivers/net/wireless/rt2x00/rt61pci.c      | 4 +++-
+ drivers/net/wireless/rt2x00/rt73usb.c      | 4 +++-
+ 9 files changed, 23 insertions(+), 11 deletions(-)
+
+--- a/drivers/net/wireless/rt2x00/rt2400pci.c
++++ b/drivers/net/wireless/rt2x00/rt2400pci.c
+@@ -273,8 +273,10 @@ static void rt2400pci_config_filter(stru
+ 			   !(filter_flags & FIF_PLCPFAIL));
+ 	rt2x00_set_field32(&reg, RXCSR0_DROP_CONTROL,
+ 			   !(filter_flags & FIF_CONTROL));
+-	rt2x00_set_field32(&reg, RXCSR0_DROP_NOT_TO_ME, 1);
++	rt2x00_set_field32(&reg, RXCSR0_DROP_NOT_TO_ME,
++			   !test_bit(CONFIG_MONITORING, &rt2x00dev->flags));
+ 	rt2x00_set_field32(&reg, RXCSR0_DROP_TODS,
++			   !test_bit(CONFIG_MONITORING, &rt2x00dev->flags) &&
+ 			   !rt2x00dev->intf_ap_count);
+ 	rt2x00_set_field32(&reg, RXCSR0_DROP_VERSION_ERROR, 1);
+ 	rt2x00mmio_register_write(rt2x00dev, RXCSR0, reg);
+--- a/drivers/net/wireless/rt2x00/rt2500pci.c
++++ b/drivers/net/wireless/rt2x00/rt2500pci.c
+@@ -274,8 +274,10 @@ static void rt2500pci_config_filter(stru
+ 			   !(filter_flags & FIF_PLCPFAIL));
+ 	rt2x00_set_field32(&reg, RXCSR0_DROP_CONTROL,
+ 			   !(filter_flags & FIF_CONTROL));
+-	rt2x00_set_field32(&reg, RXCSR0_DROP_NOT_TO_ME, 1);
++	rt2x00_set_field32(&reg, RXCSR0_DROP_NOT_TO_ME,
++			   !test_bit(CONFIG_MONITORING, &rt2x00dev->flags));
+ 	rt2x00_set_field32(&reg, RXCSR0_DROP_TODS,
++			   !test_bit(CONFIG_MONITORING, &rt2x00dev->flags) &&
+ 			   !rt2x00dev->intf_ap_count);
+ 	rt2x00_set_field32(&reg, RXCSR0_DROP_VERSION_ERROR, 1);
+ 	rt2x00_set_field32(&reg, RXCSR0_DROP_MCAST,
+--- a/drivers/net/wireless/rt2x00/rt2500usb.c
++++ b/drivers/net/wireless/rt2x00/rt2500usb.c
+@@ -434,8 +434,10 @@ static void rt2500usb_config_filter(stru
+ 			   !(filter_flags & FIF_PLCPFAIL));
+ 	rt2x00_set_field16(&reg, TXRX_CSR2_DROP_CONTROL,
+ 			   !(filter_flags & FIF_CONTROL));
+-	rt2x00_set_field16(&reg, TXRX_CSR2_DROP_NOT_TO_ME, 1);
++	rt2x00_set_field16(&reg, TXRX_CSR2_DROP_NOT_TO_ME,
++			   !test_bit(CONFIG_MONITORING, &rt2x00dev->flags));
+ 	rt2x00_set_field16(&reg, TXRX_CSR2_DROP_TODS,
++			   !test_bit(CONFIG_MONITORING, &rt2x00dev->flags) &&
+ 			   !rt2x00dev->intf_ap_count);
+ 	rt2x00_set_field16(&reg, TXRX_CSR2_DROP_VERSION_ERROR, 1);
+ 	rt2x00_set_field16(&reg, TXRX_CSR2_DROP_MULTICAST,
+--- a/drivers/net/wireless/rt2x00/rt2800lib.c
++++ b/drivers/net/wireless/rt2x00/rt2800lib.c
+@@ -1490,7 +1490,8 @@ void rt2800_config_filter(struct rt2x00_
+ 			   !(filter_flags & FIF_FCSFAIL));
+ 	rt2x00_set_field32(&reg, RX_FILTER_CFG_DROP_PHY_ERROR,
+ 			   !(filter_flags & FIF_PLCPFAIL));
+-	rt2x00_set_field32(&reg, RX_FILTER_CFG_DROP_NOT_TO_ME, 1);
++	rt2x00_set_field32(&reg, RX_FILTER_CFG_DROP_NOT_TO_ME,
++			   !test_bit(CONFIG_MONITORING, &rt2x00dev->flags));
+ 	rt2x00_set_field32(&reg, RX_FILTER_CFG_DROP_NOT_MY_BSSD, 0);
+ 	rt2x00_set_field32(&reg, RX_FILTER_CFG_DROP_VER_ERROR, 1);
+ 	rt2x00_set_field32(&reg, RX_FILTER_CFG_DROP_MULTICAST,
+--- a/drivers/net/wireless/rt2x00/rt2x00.h
++++ b/drivers/net/wireless/rt2x00/rt2x00.h
+@@ -669,6 +669,7 @@ enum rt2x00_state_flags {
+ 	CONFIG_POWERSAVING,
+ 	CONFIG_HT_DISABLED,
+ 	CONFIG_QOS_DISABLED,
++	CONFIG_MONITORING,
+ 
+ 	/*
+ 	 * Mark we currently are sequentially reading TX_STA_FIFO register
+--- a/drivers/net/wireless/rt2x00/rt2x00config.c
++++ b/drivers/net/wireless/rt2x00/rt2x00config.c
+@@ -277,6 +277,11 @@ void rt2x00lib_config(struct rt2x00_dev
+ 	else
+ 		clear_bit(CONFIG_POWERSAVING, &rt2x00dev->flags);
+ 
++	if (conf->flags & IEEE80211_CONF_MONITOR)
++		set_bit(CONFIG_MONITORING, &rt2x00dev->flags);
++	else
++		clear_bit(CONFIG_MONITORING, &rt2x00dev->flags);
++
+ 	rt2x00dev->curr_band = conf->chandef.chan->band;
+ 	rt2x00dev->curr_freq = conf->chandef.chan->center_freq;
+ 	rt2x00dev->tx_power = conf->power_level;
+--- a/drivers/net/wireless/rt2x00/rt2x00mac.c
++++ b/drivers/net/wireless/rt2x00/rt2x00mac.c
+@@ -385,11 +385,6 @@ void rt2x00mac_configure_filter(struct i
+ 			*total_flags |= FIF_PSPOLL;
+ 	}
+ 
+-	/*
+-	 * Check if there is any work left for us.
+-	 */
+-	if (rt2x00dev->packet_filter == *total_flags)
+-		return;
+ 	rt2x00dev->packet_filter = *total_flags;
+ 
+ 	rt2x00dev->ops->lib->config_filter(rt2x00dev, *total_flags);
+--- a/drivers/net/wireless/rt2x00/rt61pci.c
++++ b/drivers/net/wireless/rt2x00/rt61pci.c
+@@ -530,8 +530,10 @@ static void rt61pci_config_filter(struct
+ 			   !(filter_flags & FIF_PLCPFAIL));
+ 	rt2x00_set_field32(&reg, TXRX_CSR0_DROP_CONTROL,
+ 			   !(filter_flags & (FIF_CONTROL | FIF_PSPOLL)));
+-	rt2x00_set_field32(&reg, TXRX_CSR0_DROP_NOT_TO_ME, 1);
++	rt2x00_set_field32(&reg, TXRX_CSR0_DROP_NOT_TO_ME,
++			   !test_bit(CONFIG_MONITORING, &rt2x00dev->flags));
+ 	rt2x00_set_field32(&reg, TXRX_CSR0_DROP_TO_DS,
++			   !test_bit(CONFIG_MONITORING, &rt2x00dev->flags) &&
+ 			   !rt2x00dev->intf_ap_count);
+ 	rt2x00_set_field32(&reg, TXRX_CSR0_DROP_VERSION_ERROR, 1);
+ 	rt2x00_set_field32(&reg, TXRX_CSR0_DROP_MULTICAST,
+--- a/drivers/net/wireless/rt2x00/rt73usb.c
++++ b/drivers/net/wireless/rt2x00/rt73usb.c
+@@ -480,8 +480,10 @@ static void rt73usb_config_filter(struct
+ 			   !(filter_flags & FIF_PLCPFAIL));
+ 	rt2x00_set_field32(&reg, TXRX_CSR0_DROP_CONTROL,
+ 			   !(filter_flags & (FIF_CONTROL | FIF_PSPOLL)));
+-	rt2x00_set_field32(&reg, TXRX_CSR0_DROP_NOT_TO_ME, 1);
++	rt2x00_set_field32(&reg, TXRX_CSR0_DROP_NOT_TO_ME,
++			   !test_bit(CONFIG_MONITORING, &rt2x00dev->flags));
+ 	rt2x00_set_field32(&reg, TXRX_CSR0_DROP_TO_DS,
++			   !test_bit(CONFIG_MONITORING, &rt2x00dev->flags) &&
+ 			   !rt2x00dev->intf_ap_count);
+ 	rt2x00_set_field32(&reg, TXRX_CSR0_DROP_VERSION_ERROR, 1);
+ 	rt2x00_set_field32(&reg, TXRX_CSR0_DROP_MULTICAST,
diff --git a/debian/patches/series b/debian/patches/series
index 49a34c5..c62983f 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -139,3 +139,4 @@ bugfix/all/fuse-break-infinite-loop-in-fuse_fill_write_pages.patch
 bugfix/all/scsi-fix-crashes-in-sd-and-sr-runtime-pm.patch
 bugfix/all/netfilter-nf_nat_redirect-add-missing-NULL-pointer-c.patch
 debian/usb-fix-abi-change-in-4.3.5.patch
+bugfix/all/rt2x00-fix-monitor-mode-regression.patch

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/kernel/linux.git



More information about the Kernel-svn-changes mailing list