Cong Wang <xiyou.wangc...@gmail.com> wrote:
> > - hlist_for_each_entry(tmp, &v->hhead, bydst)
> > - tmp->bydst_reinsert = true;
> > - hlist_for_each_entry(tmp, &n->hhead, bydst)
> > + hlist_for_each_entry(tmp, &v->hhead, bydst) {
>
>
> hlist_for_each_entry_safe()?Good question. Its not necessary from a technical point of view because tmp isn't free'd and hlist_del_rcu leaves tmp->next alone. But perhaps its still better to use _safe variant. I'll let Steffen decide.
