On Wed, 2012-06-13 at 11:00 +0300, Eliad Peller wrote: > > @@ -1329,6 +1329,8 @@ static void ieee80211_set_disassoc(struct ieee80211_sub_if_data *sdata, > > u32 changed = 0; > > u8 bssid[ETH_ALEN]; > > > > + ieee80211_stop_poll(sdata); > > + > > ASSERT_MGD_MTX(ifmgd); > > it doesn't matter much, but maybe better add the call after the > function "prologue" (lockdep, WARN_ONs, ...)? I think we shouldn't do it after the warnings since those affect the control flow. OTOH, they should never happen ... 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