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 <[email protected]> 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 <[email protected]>
>
>
> Acked-by: Jamal Hadi Salim <[email protected]>
>
> cheers,
> jamal
>
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
