Hi, On Wed, 2018-06-20 at 11:06 -0700, Cong Wang wrote: > On Wed, Jun 20, 2018 at 10:34 AM, Paolo Abeni <pab...@redhat.com> wrote: > > > > +static void fl_mask_free(struct fl_flow_mask *mask) > > +{ > > + rhashtable_destroy(&mask->ht); > > I don't believe you can call rhashtable_destroy() in BH > context, it acquires a mutex...
Thank you for the review. You are right. Tomorrow (I'm on holiday today) I'll look towards leverging tcf_queue_work() to schedule fl_mask_free()/rhashtable_destroy() at a proper time. Cheers, Paolo