aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFelix Fietkau <nbd@openwrt.org>2009-11-13 21:29:38 -0500
committerJohn W. Linville <linville@tuxdriver.com>2009-11-18 17:09:10 -0500
commit3e5b1101f59fb3e17a8eb32cca100ae07fd7100e (patch)
tree99f38c9d8e1ec97c0713325e3628d3836839eb8c
parent7d57b73a73040525dfe22c56d823f146711ff971 (diff)
mac80211: reduce the amount of unnecessary traffic on cooked monitor interfaces
In order to handle association and authentication in AP mode, hostapd needs access to the tx status info of its own frames through a cooked monitor interface. Without this patch the cooked monitor interfaces also passed on tx status information for packets from other virtual interfaces. This creates a significant performance issue on embedded system. Hostapd tries to work around this by installing a Linux Socket Filter that only captures the frames it's interested in, however data duplication and socket filter matching still uses up enough CPU cycles to be very noticeable on small systems. This patch ensures that tx status information of non-injected frames does not make it to cooked monitor interfaces. Signed-off-by: Felix Fietkau <nbd@openwrt.org> Signed-off-by: John W. Linville <linville@tuxdriver.com>
-rw-r--r--net/mac80211/main.c5
1 files changed, 5 insertions, 0 deletions
diff --git a/net/mac80211/main.c b/net/mac80211/main.c
index beb8718d905e..c74a6a1935b3 100644
--- a/net/mac80211/main.c
+++ b/net/mac80211/main.c
@@ -587,6 +587,11 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb)
587 if (!netif_running(sdata->dev)) 587 if (!netif_running(sdata->dev))
588 continue; 588 continue;
589 589
590 if ((sdata->u.mntr_flags & MONITOR_FLAG_COOK_FRAMES) &&
591 !(info->flags & IEEE80211_TX_CTL_INJECTED) &&
592 (type == IEEE80211_FTYPE_DATA))
593 continue;
594
590 if (prev_dev) { 595 if (prev_dev) {
591 skb2 = skb_clone(skb, GFP_ATOMIC); 596 skb2 = skb_clone(skb, GFP_ATOMIC);
592 if (skb2) { 597 if (skb2) {