On 21 May 2015 at 10:30, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Thu, 2015-05-21 at 10:16 +0200, Michal Kazior wrote: > >> >> rtnl_lock(); >> >> + list_for_each_entry(sdata, &local->interfaces, list) >> >> + cancel_delayed_work_sync(&sdata->dec_tailroom_needed_wk); >> > >> > Would it make sense to just flush the work here? That way we don't have >> > to do all the other things. >> >> Hmm.. dec_tailroom_needed_wk is queued on system workqueue now so >> there's no feasible way of flushing it (restart_work is on a system >> workqueue as well). It'd need to be moved to local->workqueue. I guess >> that would work too. > > flush_work()? Oh. I wasn't aware of this call.. Thanks for pointing out :-) Michał -- 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
