Re: [PATCH] memcg: make memcg's file mapped consistent with globalVM

From: KAMEZAWA Hiroyuki
Date: Wed Oct 28 2009 - 03:27:35 EST


On Wed, 28 Oct 2009 12:48:54 +0530
Balbir Singh <balbir@xxxxxxxxxxxxxxxxxx> wrote:

> * KAMEZAWA Hiroyuki <kamezawa.hiroyu@xxxxxxxxxxxxxx> [2009-10-28 12:16:19]:
>
> > Based on mmotm-Oct13 + some patches in -mm queue.
> >
> > ==
> > From: KAMEZAWA Hiroyuki <kamezawa.hiroyu@xxxxxxxxxxxxxx>
> >
> > memcg-cleanup-file-mapped-consistent-with-globarl-vm-stat.patch
> >
> > In global VM, FILE_MAPPED is used but memcg uses MAPPED_FILE.
> > This makes grep difficult. Replace memcg's MAPPED_FILE with FILE_MAPPED
> >
> > And in global VM, mapped shared memory is accounted into FILE_MAPPED.
> > But memcg doesn't. fix it.
>
> I wanted to explicitly avoid this since I wanted to do an iterative
> correct accounting of shared memory. The renaming is fine with me
> since we don't break ABI in user space.
>
To do that, FILE_MAPPED is not correct.
Because MAPPED includes shmem in global VM, no valid reason to do different
style of counting.

For shmem, we have a charge type as MEM_CGROUP_CHARGE_TYPE_SHMEM and
we can set "PCG_SHMEM" flag onto page_cgroup or some.
Then, we can count it in explicit way.

Thanks,
-Kame

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/