Thanks guys for the review. For completeness, I'll add smp_wmb() here :
gact->tcfg_pval = max_t(u16, 1, p_parm->pval); smp_wmb(); gact->tcfg_ptype = p_parm->ptype; And corresponding smp_rmb() On Fri, Jul 3, 2015 at 12:49 PM, Jamal Hadi Salim <j...@mojatatu.com> wrote: > On 07/02/15 09:07, Eric Dumazet wrote: >> >> First step for gact RCU operation : >> >> Instead of testing if tcfg_pval is zero or not, just make it 1. >> >> No change in behavior, but slightly faster code. >> >> Signed-off-by: Eric Dumazet <eduma...@google.com> > > > Acked-by: Jamal Hadi Salim <j...@mojatatu.com> > > cheers, > jamal > -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html