Jakub Kicinski <jakub.kicin...@netronome.com> writes: > On Mon, 04 Mar 2019 13:47:47 +0100, Toke Høiland-Jørgensen wrote: >> In an earlier version I had a namespace enter/exit notifier in devmap.c >> as well, to react to new namespaces. And that notifier has a comment >> about avoiding calls to synchronize_rcu(). But since this version >> doesn't actually need that, maybe I can just keep using direct calls and >> synchronize_rcu() and avoid the callback? I'm a bit worried about adding >> both synchronize_rcu() and cond_resched() as a possible side effect of >> every call to bpf_prog_put(), though; so maybe it's better to move the >> cleanup somewhere it's actually safe to call cond_resched(); what would >> that be, a workqueue? > > Workqueue would be my go to.
Gotcha! Thanks :) -Toke
