On Tue, 2010-09-21 at 15:27 -0400, Luis R. Rodriguez wrote: > + mutex_lock(&local->sta_mtx); > + list_for_each_entry(sta, &local->sta_list, list) > + if (local->hw.flags & IEEE80211_HW_AMPDU_AGGREGATION) > + set_sta_flags(sta, WLAN_STA_BLOCK_BA); > + mutex_unlock(&local->sta_mtx); > + This is really expensive, I don't think we should do it this way. Wouldn't it be possible to put a check into place wherever the flag is checked too? johannes -- 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