On Thu, 23 Jul 2015 09:07:37 -0700 Nikolay Aleksandrov <nikolay@xxxxxxxxxxxxxxxxxxx> wrote: > + /* Stop hello and hold timer */ > + spin_lock_bh(&br->lock); > + del_timer(&br->hello_timer); > + list_for_each_entry(p, &br->port_list, list) > + del_timer(&p->hold_timer); > + spin_unlock_bh(&br->lock); Wouldn't it be easier to use del_timer_sync here?