On Mon, Feb 1, 2010 at 12:12 PM, Eric Dumazet <eric.dumazet@xxxxxxxxx> wrote: > Le lundi 01 février 2010 à 11:36 +0200, Alexey Dobriyan a écrit : >> On Mon, Feb 1, 2010 at 11:32 AM, Jon Masters <jonathan@xxxxxxxxxxxxxx> wrote: >> > I hacked up a per-namespace version of hashtables (this needs doing >> > anyway, since the global stuff is just waiting to break) >> >> Which ones? Conntrack hashtables are per-netns. > > It seems they are, but this is not a complete work : They are per-netns. It's not "complete", because right now there is no point in doing more. nf_conntrack_max was rejected given the absense of per-netns kernel memory consumption limiting. > 1) Global settings (shared by all netns) Only hashtable size which is module parameter and there is no generic way to limit kernel memory (like beancounters). > 2) nf_conntrack_cachep is shared, it should be not shared. There is no need for it to be shared, unless you measured something. > 3) nf_conntrack_untracked shared by all netns, it should be local. > > nf_conntrack_cleanup_net() can block forever because of this. > > while (atomic_read(&nf_conntrack_untracked.ct_general.use) > 1) > schedule(); This is known, yes, thinking on it, as naive way was agreed to suck. -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
