lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 05 Jun 2012 08:23:28 +0530 From: "Aneesh Kumar K.V" <aneesh.kumar@...ux.vnet.ibm.com> To: Kamezawa Hiroyuki <kamezawa.hiroyu@...fujitsu.com> Cc: linux-mm@...ck.org, dhillf@...il.com, rientjes@...gle.com, mhocko@...e.cz, akpm@...ux-foundation.org, hannes@...xchg.org, linux-kernel@...r.kernel.org, cgroups@...r.kernel.org Subject: Re: [PATCH -V7 07/14] mm/page_cgroup: Make page_cgroup point to the cgroup rather than the mem_cgroup Kamezawa Hiroyuki <kamezawa.hiroyu@...fujitsu.com> writes: > (2012/05/30 23:38), Aneesh Kumar K.V wrote: >> From: "Aneesh Kumar K.V"<aneesh.kumar@...ux.vnet.ibm.com> >> >> We will use it later to make page_cgroup track the hugetlb cgroup information. >> >> Signed-off-by: Aneesh Kumar K.V<aneesh.kumar@...ux.vnet.ibm.com> >> --- >> include/linux/mmzone.h | 2 +- >> include/linux/page_cgroup.h | 8 ++++---- >> init/Kconfig | 4 ++++ >> mm/Makefile | 3 ++- >> mm/memcontrol.c | 42 +++++++++++++++++++++++++----------------- >> 5 files changed, 36 insertions(+), 23 deletions(-) >> >> diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h >> index 2427706..2483cc5 100644 >> --- a/include/linux/mmzone.h >> +++ b/include/linux/mmzone.h >> @@ -1052,7 +1052,7 @@ struct mem_section { >> >> /* See declaration of similar field in struct zone */ >> unsigned long *pageblock_flags; >> -#ifdef CONFIG_CGROUP_MEM_RES_CTLR >> +#ifdef CONFIG_PAGE_CGROUP >> /* >> * If !SPARSEMEM, pgdat doesn't have page_cgroup pointer. We use >> * section. (see memcontrol.h/page_cgroup.h about this.) >> diff --git a/include/linux/page_cgroup.h b/include/linux/page_cgroup.h >> index a88cdba..7bbfe37 100644 >> --- a/include/linux/page_cgroup.h >> +++ b/include/linux/page_cgroup.h >> @@ -12,7 +12,7 @@ enum { >> #ifndef __GENERATING_BOUNDS_H >> #include<generated/bounds.h> >> >> -#ifdef CONFIG_CGROUP_MEM_RES_CTLR >> +#ifdef CONFIG_PAGE_CGROUP >> #include<linux/bit_spinlock.h> >> >> /* >> @@ -24,7 +24,7 @@ enum { >> */ >> struct page_cgroup { >> unsigned long flags; >> - struct mem_cgroup *mem_cgroup; >> + struct cgroup *cgroup; >> }; >> > > This patch seems very bad. I had to change that to struct page_cgroup { unsigned long flags; struct cgroup_subsys_state *css; }; to get memcg to work. We end up changing css.cgroup on cgroupfs mount/umount. > > - What is the performance impact to memcg ? Doesn't this add extra overheads > to memcg lookup ? Considering that we are stashing cgroup_subsys_state, it should be a simple addition. I haven't measured the exact numbers. Do you have any suggestion on the tests I can run ? > - Hugetlb reuquires much more smaller number of tracking information rather > than memcg requires. I guess you can record the information into page->private > if you want. So If we end up tracking page cgroup in struct page all these extra over head will go away. And in most case we would have both memcg and hugetlb enabled by default. > - This may prevent us from the work 'reducing size of page_cgroup' > by reducing you mean moving struct page_cgroup info to struct page itself ? If so this should not have any impact right ? Most of the requirement of hugetlb should be similar to memcg. > So, strong Nack to this. I guess you can use page->private or some entries in > struct page, you have many pages per accounting units. Please make an effort > to avoid using page_cgroup. > HugeTLB already use page->private of compound page head to track subpool pointer. So we won't be able to use page->private. -aneesh -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists