Messages in this thread | | | Date | Thu, 11 Aug 2011 13:46:19 +0530 | From | Raghavendra K T <> | Subject | Re: [PATCH 0/2][cleanup] memcg: renaming of mem variable to memcg |
| |
On 08/11/2011 01:23 PM, Michal Hocko wrote: > On Wed 10-08-11 22:59:17, Raghavendra K T wrote: >> Hi, >> This is the memcg cleanup patch for that was talked little ago to change the "struct >> mem_cgroup *mem" variable to "struct mem_cgroup *memcg". >> >> The patch is though trivial, it is huge one. >> Testing : Compile tested with following configurations. >> 1) CONFIG_CGROUP_MEM_RES_CTLR=y CONFIG_CGROUP_MEM_RES_CTLR_SWAP=y >> 2) CONFIG_CGROUP_MEM_RES_CTLR=y CONFIG_CGROUP_MEM_RES_CTLR_SWAP=n >> 3) CONFIG_CGROUP_MEM_RES_CTLR=n CONFIG_CGROUP_MEM_RES_CTLR_SWAP=n > > How exactly have you tested? Compiled and compared before/after binaries > (it shouldn't change, right)? Yes, But idea was to ensure that both #ifdef and #else part are hit during compilation, which could expose some corrections needed. > >> >> Also tested basic mounting with memcgroup. >> Raghu. > > Thanks
| |