ath9k: Fix p2p address management
authorSujith Manoharan <c_manoha@qca.qualcomm.com>
Fri, 19 Sep 2014 07:30:42 +0000 (13:00 +0530)
committerJohn W. Linville <linville@tuxdriver.com>
Fri, 26 Sep 2014 21:06:51 +0000 (17:06 -0400)
When multiple channel contexts are enabled, a p2p interface
that is assigned to a context will have an address that
is different from the device mac address, which is used
by wpa_s as the p2p device ID.

Certain frames like provision requests use the device address
and these get dropped since ath9k_calculate_summary_state()
iterates over only the active interfaces in a context and the
device address is not used.

Fix this by adding the device mac address to the bssid mask.

Signed-off-by: Sujith Manoharan <c_manoha@qca.qualcomm.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ath/ath9k/ath9k.h
drivers/net/wireless/ath/ath9k/main.c

index d2a1ee1b072dbb643d6104d867d931634d2098f0..37a5ccf64dffe8a56f34895bb07a667d26159c4c 100644 (file)
@@ -354,6 +354,7 @@ struct ath_chanctx {
        bool switch_after_beacon;
 
        short nvifs;
+       short nvifs_assigned;
        unsigned int rxfilter;
 };
 
index 78fc5398c99eb91c5897046126169a4d7ccebb26..32f9db9bd53c4b80d04a4bbd97d5e32ce227f885 100644 (file)
@@ -940,6 +940,34 @@ static void ath9k_vif_iter(struct ath9k_vif_iter_data *iter_data,
        }
 }
 
+static void ath9k_update_bssid_mask(struct ath_softc *sc,
+                                   struct ath_chanctx *ctx,
+                                   struct ath9k_vif_iter_data *iter_data)
+{
+       struct ath_common *common = ath9k_hw_common(sc->sc_ah);
+       struct ath_vif *avp;
+       int i;
+
+       if (!ath9k_is_chanctx_enabled())
+               return;
+
+       list_for_each_entry(avp, &ctx->vifs, list) {
+               if (ctx->nvifs_assigned != 1)
+                       continue;
+
+               if (!avp->vif->p2p || !iter_data->has_hw_macaddr)
+                       continue;
+
+               ether_addr_copy(common->curbssid, avp->bssid);
+
+               /* perm_addr will be used as the p2p device address. */
+               for (i = 0; i < ETH_ALEN; i++)
+                       iter_data->mask[i] &=
+                               ~(iter_data->hw_macaddr[i] ^
+                                 sc->hw->wiphy->perm_addr[i]);
+       }
+}
+
 /* Called with sc->mutex held. */
 void ath9k_calculate_iter_data(struct ath_softc *sc,
                               struct ath_chanctx *ctx,
@@ -958,6 +986,8 @@ void ath9k_calculate_iter_data(struct ath_softc *sc,
 
        list_for_each_entry(avp, &ctx->vifs, list)
                ath9k_vif_iter(iter_data, avp->vif->addr, avp->vif);
+
+       ath9k_update_bssid_mask(sc, ctx, iter_data);
 }
 
 static void ath9k_set_assoc_state(struct ath_softc *sc,
@@ -1122,6 +1152,10 @@ void ath9k_calculate_summary_state(struct ath_softc *sc,
        else
                clear_bit(ATH_OP_PRIM_STA_VIF, &common->op_flags);
 
+       ath_dbg(common, CONFIG,
+               "macaddr: %pM, bssid: %pM, bssidmask: %pM\n",
+               common->macaddr, common->curbssid, common->bssidmask);
+
        ath9k_ps_restore(sc);
 }
 
@@ -2356,6 +2390,7 @@ static int ath9k_assign_vif_chanctx(struct ieee80211_hw *hw,
                conf->def.chan->center_freq);
 
        avp->chanctx = ctx;
+       ctx->nvifs_assigned++;
        list_add_tail(&avp->list, &ctx->vifs);
        ath9k_calculate_summary_state(sc, ctx);
        for (i = 0; i < IEEE80211_NUM_ACS; i++)
@@ -2384,6 +2419,7 @@ static void ath9k_unassign_vif_chanctx(struct ieee80211_hw *hw,
                conf->def.chan->center_freq);
 
        avp->chanctx = NULL;
+       ctx->nvifs_assigned--;
        list_del(&avp->list);
        ath9k_calculate_summary_state(sc, ctx);
        for (ac = 0; ac < IEEE80211_NUM_ACS; ac++)