On Tue 23-09-14 10:05:26, Johannes Weiner wrote: [...] > That's one way to put it. But the way I see it is that I remove a > generic resource counter and replace it with a pure memory counter > which I put where we account and limit memory - with one exception > that is hardly worth creating a dedicated library file for. So you completely seem to ignore people doing CONFIG_CGROUP_HUGETLB && !CONFIG_MEMCG without any justification and hiding it behind performance improvement which those users even didn't ask for yet. All that just to not have one additional header and c file hidden by CONFIG_PAGE_COUNTER selected by both controllers. No special configuration option is really needed for CONFIG_PAGE_COUNTER. > I only explained my plans of merging all memory controllers because I > assumed we could ever be on the same page when it comes to this code. I doubt this is a good plan but I might be wrong here. The main thing stays there is no good reason to make hugetlb depend on memcg right now and such a change _shouldn't_ be hidden behind an unrelated change. From hugetlb container point of view this is just a different counter which doesn't depend on memcg. I am really surprised you are pushing for this so hard right now because it only distracts from the main motivation of your patch. > But regardless of that, my approach immediately simplifies Kconfig, > Makefiles, #includes, and you haven't made a good point why the > hugetlb controller depending on memcg would harm anybody in real life. Usecases for hugetlb and memcg controllers have basically zero overlap they simply manage different resources and so it is natural to use one without the other. -- Michal Hocko SUSE Labs -- To unsubscribe from this list: send the line "unsubscribe cgroups" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
