(2012/06/29 3:29), Tejun Heo wrote:
Hello, Michal. On Thu, Jun 28, 2012 at 02:36:11PM +0200, Michal Hocko wrote:@@ -2726,6 +2726,9 @@ static int cgroup_addrm_files(struct cgroup *cgrp, struct cgroup_subsys *subsys, int err, ret = 0; for (cft = cfts; cft->name[0] != '\0'; cft++) { + if (subsys->cftype_enabled && !subsys->cftype_enabled(cft->name)) + continue; + if (is_add) err = cgroup_add_file(cgrp, subsys, cft); elseI hope we could avoid this dynamic decision. That was one of the main reasons behind doing the cftype thing. It's better to be able to "declare" these kind of things rather than being able to implement fully flexible dynamic logic. Too much flexibility often doesn't achieve much while being a hindrance to evolution of code base (trying to improve / simplify X - ooh... there's this single wacko corner case YYY here which is really different from all other users). really_do_swap_account can't change once booted, right? Why not just separate out memsw cfts into a separate array and call cgroup_add_cftypes() from init path? Can't we do that from enable_swap_cgroup()?
Yes, that's will be good. Thanks, -Kame -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>