Re: [PATCH 0/3] Per cgroup swap file support

From: Johannes Weiner
Date: Sat Mar 22 2014 - 11:44:17 EST


Hello Yu!

On Fri, Mar 21, 2014 at 03:50:31PM -0700, Yu Zhao wrote:
> This series of patches adds support to configure a cgroup to swap to a
> particular file by using control file memory.swapfile.
>
> A value of "default" in memory.swapfile indicates that this cgroup should
> use the default, system-wide, swap files. A value of "none" indicates that
> this cgroup should never swap. Other values are interpreted as the path
> to a private swap file that can only be used by the owner (and its children).
>
> The swap file has to be created and swapon() has to be done on it with
> SWAP_FLAG_PRIVATE, before it can be used. This flag ensures that the swap
> file is private and does not get used by others.
>
> Jamie Liu (1):
> swap: do not store private swap files on swap_list
>
> Suleiman Souhlal (2):
> mm/swap: support per memory cgroup swapfiles
> swap: Increase the maximum number of swap files to 8192.
>
> Documentation/cgroups/memory.txt | 15 ++
> arch/x86/include/asm/pgtable_64.h | 63 ++++++--
> include/linux/memcontrol.h | 2 +
> include/linux/swap.h | 45 +++---
> mm/memcontrol.c | 76 ++++++++++
> mm/memory.c | 3 +-
> mm/shmem.c | 2 +-
> mm/swap_state.c | 2 +-
> mm/swapfile.c | 307 +++++++++++++++++++++++++++++++-------
> mm/vmscan.c | 2 +-
> 10 files changed, 423 insertions(+), 94 deletions(-)

For feature patches like this, please include a rationale. What is
this functionality good for, and who is going to use this?
--
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/