lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 4 Mar 2019 09:08:59 -0800 From: Jakub Kicinski <jakub.kicinski@...ronome.com> To: Toke Høiland-Jørgensen <toke@...hat.com> Cc: David Miller <davem@...emloft.net>, netdev@...r.kernel.org, Jesper Dangaard Brouer <brouer@...hat.com>, Daniel Borkmann <daniel@...earbox.net>, Alexei Starovoitov <ast@...nel.org> Subject: Re: [PATCH net-next v3 1/3] xdp: Refactor devmap code in preparation for subsequent additions 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.
Powered by blists - more mailing lists