Messages in this thread | | | Date | Wed, 27 Jun 2012 12:43:50 +0400 | From | Glauber Costa <> | Subject | Re: [PATCH v2] memcg: unify mem_control_xxx to memcg_xxx |
| |
On 06/27/2012 04:16 AM, Wanpeng Li wrote: > From: Wanpeng Li <liwp@linux.vnet.ibm.com> > > Changelog: > > V2 -> V1: > > * replace memory_cgroup_xxx by memcg_xxx > > Unify memcg functions to memcg_xxx.
This patch achieves nothing. You're changing a couple of function names. Andrew stated goal was to change *all* function names from mem_cgroup_xxx to memcg_xxx, not just one or two.
This can be done mechanically by replacing occurrences of mem_cgroup_xxx with memcg_xxx, and verifying that your final result is correct by compiling the kernel with and without the combination of config options that matters here, plus a set of randconfigs - or anything like that.
Make sure you have Andrew's latest patch in your tree, that converts the config option names.
| |