On Tue, Nov 20, 2018 at 1:19 PM Davide Caratti <dcara...@redhat.com> wrote: > @@ -257,25 +261,28 @@ static int tcf_police_act(struct sk_buff *skb, const > struct tc_action *a, > } > > now = ktime_get_ns(); > - toks = min_t(s64, now - p->tcfp_t_c, p->tcfp_burst); > + spin_lock_bh(&police->tcfp_lock); > + toks = min_t(s64, now - police->tcfp_t_c, p->tcfp_burst);
I don't think you need to disable BH here.
