Pablo Neira Ayuso <pa...@netfilter.org> wrote:
> > diff --git a/net/bridge/netfilter/ebt_redirect.c 
> > b/net/bridge/netfilter/ebt_redirect.c
> > index 20396499..2e7c4f9 100644
> > --- a/net/bridge/netfilter/ebt_redirect.c
> > +++ b/net/bridge/netfilter/ebt_redirect.c
> > @@ -24,7 +24,7 @@ ebt_redirect_tg(struct sk_buff *skb, const struct 
> > xt_action_param *par)
> >             return EBT_DROP;
> >  
> >     if (par->hooknum != NF_BR_BROUTING)
> > -           /* rcu_read_lock()ed by nf_hook_slow */
> > +           /* rcu_read_lock()ed by nf_hook_thresh */
> 
> Why are all these comments being renamed in this patch? This patch
> description doesn't say anything about this.

Aaron, I'm sorry about this.

I suggest to do this renaming in a different patch, i.e.
keep the rcu_read_lock in nf_hook_slow in this patch.

--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to