mac80211: count authorized stations per BSS
authorJohannes Berg <johannes.berg@intel.com>
Wed, 14 Dec 2011 11:20:31 +0000 (12:20 +0100)
committerJohn W. Linville <linville@tuxdriver.com>
Thu, 15 Dec 2011 19:46:34 +0000 (14:46 -0500)
Currently, each AP interface will send multicast
traffic if any interface has a station entry even
if that station entry is allocated only. With the
new station state management we can easily fix it
by adding a counter that counts each authorized
station only and send multicast traffic only when
the correct interface has at least one authorized
station.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
net/mac80211/debugfs_netdev.c
net/mac80211/ieee80211_i.h
net/mac80211/sta_info.c
net/mac80211/tx.c

index 8df28910b8ee9865d333d46773e67c250313bc20..176c08ffb13c50e5f5fa8c212175a8367f09bb97 100644 (file)
@@ -321,6 +321,7 @@ static ssize_t ieee80211_if_parse_tkip_mic_test(
 __IEEE80211_IF_FILE_W(tkip_mic_test);
 
 /* AP attributes */
+IEEE80211_IF_FILE(num_sta_authorized, u.ap.num_sta_authorized, ATOMIC);
 IEEE80211_IF_FILE(num_sta_ps, u.ap.num_sta_ps, ATOMIC);
 IEEE80211_IF_FILE(dtim_count, u.ap.dtim_count, DEC);
 
@@ -458,6 +459,7 @@ static void add_ap_files(struct ieee80211_sub_if_data *sdata)
        DEBUGFS_ADD(rc_rateidx_mask_2ghz);
        DEBUGFS_ADD(rc_rateidx_mask_5ghz);
 
+       DEBUGFS_ADD(num_sta_authorized);
        DEBUGFS_ADD(num_sta_ps);
        DEBUGFS_ADD(dtim_count);
        DEBUGFS_ADD(num_buffered_multicast);
index 96fe75410bbe09549929682d31a7756286869759..9516c3088fba75e6e65fde685fafc4edf306b759 100644 (file)
@@ -243,6 +243,7 @@ struct ieee80211_if_ap {
        u8 tim[sizeof(unsigned long) * BITS_TO_LONGS(IEEE80211_MAX_AID + 1)];
        struct sk_buff_head ps_bc_buf;
        atomic_t num_sta_ps; /* number of stations in PS mode */
+       atomic_t num_sta_authorized; /* number of authorized stations */
        int dtim_count;
        bool dtim_bc_mc;
 };
index b22775cdcdf5bb12cae957ceae6983d7f53475c4..141315ce17d8097b70de1064395158a37fe7a277 100644 (file)
@@ -1567,17 +1567,21 @@ int sta_info_move_state_checked(struct sta_info *sta,
                        return -EINVAL;
                break;
        case IEEE80211_STA_ASSOC:
-               if (sta->sta_state == IEEE80211_STA_AUTH)
+               if (sta->sta_state == IEEE80211_STA_AUTH) {
                        set_bit(WLAN_STA_ASSOC, &sta->_flags);
-               else if (sta->sta_state == IEEE80211_STA_AUTHORIZED)
+               } else if (sta->sta_state == IEEE80211_STA_AUTHORIZED) {
+                       if (sta->sdata->vif.type == NL80211_IFTYPE_AP)
+                               atomic_dec(&sta->sdata->u.ap.num_sta_authorized);
                        clear_bit(WLAN_STA_AUTHORIZED, &sta->_flags);
-               else
+               else
                        return -EINVAL;
                break;
        case IEEE80211_STA_AUTHORIZED:
-               if (sta->sta_state == IEEE80211_STA_ASSOC)
+               if (sta->sta_state == IEEE80211_STA_ASSOC) {
+                       if (sta->sdata->vif.type == NL80211_IFTYPE_AP)
+                               atomic_inc(&sta->sdata->u.ap.num_sta_authorized);
                        set_bit(WLAN_STA_AUTHORIZED, &sta->_flags);
-               else
+               else
                        return -EINVAL;
                break;
        default:
index 50c4be9a64041e479a6d9d65434c4ffab7e3e205..6bbd6cccb4a050cee2d554ea49269681a0379924 100644 (file)
@@ -304,17 +304,14 @@ ieee80211_tx_h_check_assoc(struct ieee80211_tx_data *tx)
                        I802_DEBUG_INC(tx->local->tx_handlers_drop_not_assoc);
                        return TX_DROP;
                }
-       } else {
-               if (unlikely(ieee80211_is_data(hdr->frame_control) &&
-                            tx->local->num_sta == 0 &&
-                            tx->sdata->vif.type != NL80211_IFTYPE_ADHOC)) {
-                       /*
-                        * No associated STAs - no need to send multicast
-                        * frames.
-                        */
-                       return TX_DROP;
-               }
-               return TX_CONTINUE;
+       } else if (unlikely(tx->sdata->vif.type == NL80211_IFTYPE_AP &&
+                           ieee80211_is_data(hdr->frame_control) &&
+                           !atomic_read(&tx->sdata->u.ap.num_sta_authorized))) {
+               /*
+                * No associated STAs - no need to send multicast
+                * frames.
+                */
+               return TX_DROP;
        }
 
        return TX_CONTINUE;