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, 1 Dec 2008 16:15:47 -0800 From: Paul Menage <menage@...gle.com> To: Li Zefan <lizf@...fujitsu.com> Cc: Andrew Morton <akpm@...ux-foundation.org>, LKML <linux-kernel@...r.kernel.org>, Linux Containers <containers@...ts.linux-foundation.org> Subject: Re: [PATCH 1/2] cgroups: cleanup for dummy root On Fri, Nov 28, 2008 at 1:03 AM, Li Zefan <lizf@...fujitsu.com> wrote: > > Hmmm...Sorry, this patch is wrong, when a subsystem is inactive thus in > rootnode, it may still iterate tasks in dummy top cgroup, though currently > no subsystem does so. Right - iterating across the tasks in a cgroup for an unmounted subsystem is a completely valid thing to do. I suppose it might make sense to optimize that to just scan the tasklist and thread lists in that case, to avoid the overhead of maintaining the cg links for the unmounted subsystems, but you'd have to change the cgroup iterator first, before you stop maintaining the links. Given that I suspect the main overhead of maintaining the links is taking css_set_lock, I'm not sure how much it hurts to be keeping the links for the inactive hierarchies too and keeping the code simpler. Paul -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists