On Tue, Nov 04, 2014 at 03:22:50PM -0800, greearb@xxxxxxxxxxxxxxx wrote: > From: Ben Greear <greearb@xxxxxxxxxxxxxxx> > > The commit that went into 3.17: > > ath9k: Summarize hw state per channel context > > Group and set hw state (opmode, primary_sta, beacon conf) per > channel context instead of whole list of vifs. This would allow > each channel context to run in different mode (STA/AP). > > Signed-off-by: Felix Fietkau <nbd@xxxxxxxxxxx> > Signed-off-by: Rajkumar Manoharan <rmanohar@xxxxxxxxxxxxxxxx> > Signed-off-by: John W. Linville <linville@xxxxxxxxxxxxx> > > broke multi-vif configuration due to not properly calculating > the bssid mask. > > The test case that caught this was: > > create wlan0 and sta0-4 (6 total), not sure how much that matters. > associate all 6 (works fine) > disconnect 5 of them, leaving sta0 up > Start trying to bring up the other 5 one at a time. It will > fail, with iw events looking like this (in these logs, several > sta are trying to come up, but symptom is the same with just one) > > The patch causing the regression made quite a few changes, but > the part I think caused this particular problem was not > recalculating the bssid mask when adding and removing interfaces. > > Re-adding those calls fixes my test case. Fix bad comment > as well. > > Signed-off-by: Ben Greear <greearb@xxxxxxxxxxxxxxx> > --- > > This needs to be reviewed by ath9k driver folks that added > the original patch, at least. Any word from the ath9k posse? > And, should be considered for 3.17 stable if fix is accepted upstream. > > drivers/net/wireless/ath/ath9k/main.c | 9 ++++++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > diff --git a/drivers/net/wireless/ath/ath9k/main.c b/drivers/net/wireless/ath/ath9k/main.c > index fbf23ac..d2ad9e95 100644 > --- a/drivers/net/wireless/ath/ath9k/main.c > +++ b/drivers/net/wireless/ath/ath9k/main.c > @@ -947,9 +947,8 @@ void ath9k_calculate_iter_data(struct ath_softc *sc, > struct ath_vif *avp; > > /* > - * Pick the MAC address of the first interface as the new hardware > - * MAC address. The hardware will use it together with the BSSID mask > - * when matching addresses. > + * The hardware will use primary station addr together with the > + * BSSID mask when matching addresses. > */ > memset(iter_data, 0, sizeof(*iter_data)); > memset(&iter_data->mask, 0xff, ETH_ALEN); > @@ -1169,6 +1168,8 @@ static int ath9k_add_interface(struct ieee80211_hw *hw, > list_add_tail(&avp->list, &avp->chanctx->vifs); > } > > + ath9k_calculate_summary_state(sc, avp->chanctx); > + > ath9k_assign_hw_queues(hw, vif); > > an->sc = sc; > @@ -1238,6 +1239,8 @@ static void ath9k_remove_interface(struct ieee80211_hw *hw, > > ath_tx_node_cleanup(sc, &avp->mcast_node); > > + ath9k_calculate_summary_state(sc, avp->chanctx); > + > mutex_unlock(&sc->mutex); > } > > -- > 1.9.3 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-wireless" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- John W. Linville Someday the world will need a hero, and you linville@xxxxxxxxxxxxx might be all we have. Be ready. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
