On Wed 14-11-12 09:55:08, Li Zefan wrote: > On 2012/11/13 23:30, Michal Hocko wrote: > > Hi all, > > this patch set tries to make mem_cgroup_iter saner in the way how it > > walks hierarchies. css->id based traversal is far from being ideal as it > > is not deterministic because it depends on the creation ordering. > > > > Diffstat looks promising but it is fair the say that the biggest cleanup is > > just css_get_next removal. The memcg code has grown a bit but I think it is > > worth the resulting outcome (the sanity ;)). > > > > So memcg won't use css id at all, right? Unfortunately we still use it for the swap accounting but that one could be replaced by something else, probably. Have to think about it. > Then we can remove the whole css_id stuff, and that's quite a bunch of > code. Is memcg the only user of css_id? Quick grep shows that yes but I haven't checked all the callers of the exported functions. I would be happy if more code goes away. -- Michal Hocko SUSE Labs -- 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>