[Devel] Re: [PATCH -mm 0/5] swapcgroup (v3)
nishimura at mxp.nes.nec.co.jp
Sun Jul 6 23:48:30 PDT 2008
On Sat, 05 Jul 2008 12:22:25 +0530, Balbir Singh <balbir at linux.vnet.ibm.com> wrote:
> Daisuke Nishimura wrote:
> > Hi.
> > This is new version of swapcgroup.
> > Major changes from previous version
> > - Rebased on 2.6.26-rc5-mm3.
> > The new -mm has been released, but these patches
> > can be applied on 2.6.26-rc8-mm1 too with only some offset warnings.
> > I tested these patches on 2.6.26-rc5-mm3 with some fixes about memory,
> > and it seems to work fine.
> > - (NEW) Implemented force_empty.
> > Currently, it simply uncharges all the charges from the group.
> > Patches
> > - [1/5] add cgroup files
> > - [2/5] add a member to swap_info_struct
> > - [3/5] implement charge and uncharge
> > - [4/5] modify vm_swap_full()
> > - [5/5] implement force_empty
> > ToDo(in my thought. Feel free to add some others here.)
> > - need some documentation
> > Add to memory.txt? or create a new documentation file?
> I think memory.txt is good. But then, we'll need to add a Table of Contents to
> it, so that swap controller documentation can be located easily.
I think memory.txt is a self-closed documentation,
so I don't want to change it, honestlly.
I'll write a documentation for swap as a new file first for review.
> > - add option to disable only this feature
> > I'm wondering if this option is needed.
> > memcg has already the boot option to disable it.
> > Is there any case where memory should be accounted but swap should not?
> That depends on what use case you are trying to provide. Let's say I needed
> backward compatibility with 2.6.25, then I would account for memory and leave
> out swap (even though we have swap controller).
O.K. I'll add option.
Containers mailing list
Containers at lists.linux-foundation.org
More information about the Devel