On 16-08-30 08:35 AM, Eric Dumazet wrote:

synchronize_rcu() might bee to expensive if you plan to change actions
hundred of times per second.

You could instead add a 'struct rcu_head rcu;'  field in struct
tcf_skbmod_params  (but make sure this is not exported to user space)

Then :

        if (ovr)
                 spin_unlock_bh(&d->tcf_lock);
         kfree_rcu(p_old, rcu);


Ok, working on this variant. Will post today or tommorow.

cheers,
jamal


Reply via email to