[PATCH] mac80211: revamp interface and filter configuration

Drivers are currently supposed to keep track of monitor
interfaces if they allow so-called "hard" monitor, and
they are also supposed to keep track of multicast etc.

This patch changes that, replaces the set_multicast_list()
callback with a new configure_filter() callback that takes
filter flags (FIF_*) instead of interface flags (IFF_*).
For a driver, this means it should open the filter as much
as necessary to get all frames requested by the filter flags.
Accordingly, the filter flags are named "positively", e.g.
FIF_ALLMULTI.

Multicast filtering is a bit special in that drivers that
have no multicast address filters need to allow multicast
frames through when either the FIF_ALLMULTI flag is set or
when the mc_count value is positive.

At the same time, drivers are no longer notified about
monitor interfaces at all, this means they now need to
implement the start() and stop() callbacks and the new
change_filter_flags() callback. Also, the start()/stop()
ordering changed, start() is now called *before* any
add_interface() as it really should be, and stop() after
any remove_interface().

The patch also changes the behaviour of setting the bssid
to multicast for scanning when IEEE80211_HW_NO_PROBE_FILTERING
is set; the IEEE80211_HW_NO_PROBE_FILTERING flag is removed
and the filter flag FIF_BCN_PRBRESP_PROMISC introduced.
This is a lot more efficient for hardware like b43 that
supports it and other hardware can still set the BSSID
to all-ones.

Driver modifications by Johannes Berg (b43 & iwlwifi), Michael Wu
(rtl8187, adm8211, and p54), Larry Finger (b43legacy), and
Ivo van Doorn (rt2x00).

Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Signed-off-by: Michael Wu <flamingice@sourmilk.net>
Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
Signed-off-by: Ivo van Doorn <IvDoorn@gmail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>

1 files changed, 14 insertions, 90 deletions

diff --git a/drivers/net/wireless/rt2x00/rt2x00mac.c b/drivers/net/wireless/rt2x00/rt2x00mac.c
index 778ed41e21ef..17802f6d3d6d 100644
--- a/drivers/net/wireless/rt2x00/rt2x00mac.c
+++ b/drivers/net/wireless/rt2x00/rt2x00mac.c
@@ -176,46 +176,26 @@ int rt2x00mac_add_interface(struct ieee80211_hw *hw,
 {
         struct rt2x00_dev *rt2x00dev = hw->priv;
         struct interface *intf = &rt2x00dev->interface;
-        int retval;
 
         /*
          * We only support 1 non-monitor interface.
          */
-        if (conf->type != IEEE80211_IF_TYPE_MNTR && is_interface_present(intf))
+        if (is_interface_present(intf))
                 return -ENOBUFS;
 
-        /*
-         * HACK: Placeholder until start/stop handler has been
-         * added to the mac80211 callback functions structure.
-         */
-        retval = rt2x00mac_start(hw);
-        if (retval)
-                return retval;
-
-        /*
-         * We support muliple monitor mode interfaces.
-         * All we need to do is increase the monitor_count.
-         */
-        if (conf->type == IEEE80211_IF_TYPE_MNTR) {
-                intf->monitor_count++;
-        } else {
-                intf->id = conf->if_id;
-                intf->type = conf->type;
-                if (conf->type == IEEE80211_IF_TYPE_AP)
-                        memcpy(&intf->bssid, conf->mac_addr, ETH_ALEN);
-                memcpy(&intf->mac, conf->mac_addr, ETH_ALEN);
-                intf->filter = 0;
-        }
+        intf->id = conf->if_id;
+        intf->type = conf->type;
+        if (conf->type == IEEE80211_IF_TYPE_AP)
+                memcpy(&intf->bssid, conf->mac_addr, ETH_ALEN);
+        memcpy(&intf->mac, conf->mac_addr, ETH_ALEN);
 
         /*
-         * Configure interface.
          * The MAC adddress must be configured after the device
-         * has been initialized. Else the device can reset the
-         * MAC registers.
+         * has been initialized. Otherwise the device can reset
+         * the MAC registers.
          */
         rt2x00lib_config_mac_addr(rt2x00dev, intf->mac);
         rt2x00lib_config_type(rt2x00dev, conf->type);
-        rt2x00lib_config_packet_filter(rt2x00dev, intf->filter);
 
         return 0;
 }
@@ -230,22 +210,13 @@ void rt2x00mac_remove_interface(struct ieee80211_hw *hw,
         /*
          * We only support 1 non-monitor interface.
          */
-        if (conf->type != IEEE80211_IF_TYPE_MNTR && !is_interface_present(intf))
+        if (!is_interface_present(intf))
                 return;
 
-        /*
-         * When removing an monitor interface, decrease monitor_count.
-         * For non-monitor interfaces, all interface data needs to be reset.
-         */
-        if (conf->type == IEEE80211_IF_TYPE_MNTR) {
-                intf->monitor_count--;
-        } else if (intf->type == conf->type) {
-                intf->id = 0;
-                intf->type = INVALID_INTERFACE;
-                memset(&intf->bssid, 0x00, ETH_ALEN);
-                memset(&intf->mac, 0x00, ETH_ALEN);
-                intf->filter = 0;
-        }
+        intf->id = 0;
+        intf->type = INVALID_INTERFACE;
+        memset(&intf->bssid, 0x00, ETH_ALEN);
+        memset(&intf->mac, 0x00, ETH_ALEN);
 
         /*
          * Make sure the bssid and mac address registers
@@ -254,12 +225,6 @@ void rt2x00mac_remove_interface(struct ieee80211_hw *hw,
         rt2x00lib_config_mac_addr(rt2x00dev, intf->mac);
         rt2x00lib_config_bssid(rt2x00dev, intf->bssid);
         rt2x00lib_config_type(rt2x00dev, intf->type);
-
-        /*
-         * HACK: Placeholder untill start/stop handler has been
-         * added to the mac80211 callback functions structure.
-         */
-        rt2x00mac_stop(hw);
 }
 EXPORT_SYMBOL_GPL(rt2x00mac_remove_interface);
 
@@ -290,14 +255,6 @@ int rt2x00mac_config(struct ieee80211_hw *hw, struct ieee80211_conf *conf)
         rt2x00lib_config(rt2x00dev, conf);
 
         /*
-         * If promisc mode cannot be configured in irq context,
-         * then it is now the time to configure it.
-         */
-        if (test_bit(PACKET_FILTER_SCHEDULED, &rt2x00dev->flags))
-                rt2x00lib_config_packet_filter(rt2x00dev,
-                                               rt2x00dev->interface.filter);
-
-        /*
          * Reenable RX only if the radio should be on.
          */
         if (test_bit(DEVICE_ENABLED_RADIO, &rt2x00dev->flags))
@@ -326,13 +283,10 @@ int rt2x00mac_config_interface(struct ieee80211_hw *hw, int if_id,
                 return 0;
 
         /*
-         * Monitor mode does not need configuring.
          * If the given type does not match the configured type,
          * there has been a problem.
          */
-        if (conf->type == IEEE80211_IF_TYPE_MNTR)
-                return 0;
-        else if (conf->type != intf->type)
+        if (conf->type != intf->type)
                 return -EINVAL;
 
         /*
@@ -360,36 +314,6 @@ int rt2x00mac_config_interface(struct ieee80211_hw *hw, int if_id,
 }
 EXPORT_SYMBOL_GPL(rt2x00mac_config_interface);
 
-void rt2x00mac_set_multicast_list(struct ieee80211_hw *hw,
-                                  unsigned short flags, int mc_count)
-{
-        struct rt2x00_dev *rt2x00dev = hw->priv;
-
-        /*
-         * Check if the new state is different then the old state.
-         */
-        if (rt2x00dev->interface.filter == flags)
-                return;
-
-        rt2x00dev->interface.filter = flags;
-
-        /*
-         * Raise the pending bit to indicate the
-         * packet filter should be updated.
-         */
-        __set_bit(PACKET_FILTER_PENDING, &rt2x00dev->flags);
-
-        /*
-         * Check if Packet filter actions are allowed in
-         * atomic context. If not, raise the pending flag and
-         * let it be.
-         */
-        if (!test_bit(PACKET_FILTER_SCHEDULED, &rt2x00dev->flags) ||
-            !in_atomic())
-                rt2x00lib_config_packet_filter(rt2x00dev, flags);
-}
-EXPORT_SYMBOL_GPL(rt2x00mac_set_multicast_list);
-
 int rt2x00mac_get_stats(struct ieee80211_hw *hw,
                         struct ieee80211_low_level_stats *stats)
 {