OpenVZ Forum: Devel » [patch00/05]: Containers(V2)- Introduction

Home » Mailing lists » Devel » [patch00/05]: Containers(V2)- Introduction

Show: Today's Messages :: Show Polls :: Message Navigator
E-mail to friend

Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction [message #6941 is a reply to message #6929]

Thu, 28 September 2006 08:01

Balbir Singh
Messages: 491
Registered: August 2006

Senior Member

Chandra Seetharaman wrote:
> On Wed, 2006-09-27 at 14:28 -0700, Rohit Seth wrote:
>
> Rohit,
>
> For 1-4, I understand the rationale. But, your implementation deviates
> from the current behavior of the VM subsystem which could affect the
> ability of these patches getting into mainline.
>
> IMO, the current behavior in terms of reclamation, LRU, vm_swappiness,
> and writeback logic should be maintained.
>

<snip>

Hi, Rohit,

I have been playing around with the containers patch. I finally got
around to reading the code.

1. Comments on reclaiming

You could try the following options to overcome some of the disadvantages of the
current scheme.

(a) You could consider a reclaim path based on Dave Hansen's Challenged memory
controller (see http://marc.theaimsgroup.com/?l=linux-mm&m=1155669825323 45&w=2).

(b) The other option is to do what the resource group memory controller does -
build a per group LRU list of pages (active, inactive) and reclaim
them using the existing code (by passing the correct container pointer,
instead of the zone pointer). One disadvantage of this approach is that
the global reclaim is impacted as the global LRU list is broken. At the
expense of another list, we could maintain two lists, global LRU and
container LRU lists. Depending on the context of the reclaim - (container
over limit, memory pressure) we could update/manipulate both lists.
This approach is definitely very expensive.

2. Comments on task migration support

(a) One of the issues I found while using the container code is that, one could
add a task to a container say "a". "a" gets charged for the tasks usage,
when the same task moves to a different container say "b", when the task
exits, the credit goes to "b" and "a" remains indefinitely charged.

(b) For tasks addition and removal, I think it's probably better to move
the entire process (thread group) rather than allow each individual thread
to move across containers. Having threads belonging to the same process
reside in different containers can be complex to handle, since they
share the same VM. Do you have a scenario where the above condition
would be useful?

--

Warm Regards,
Balbir Singh,
Linux Technology Center,
IBM Software Labs

PS: Chandra, I hope the details of the resource group memory controller
are correct.

Report message to a moderator

[Message index]

		[patch00/05]: Containers(V2)- Introduction By: Rohit Seth on Wed, 20 September 2006 02:16
		Re: [patch00/05]: Containers(V2)- Introduction By: Nick Piggin on Wed, 20 September 2006 05:39
		Re: [patch00/05]: Containers(V2)- Introduction By: Rohit Seth on Wed, 20 September 2006 16:27
		Re: [patch00/05]: Containers(V2)- Introduction By: Christoph Lameter on Wed, 20 September 2006 16:26
		Re: [patch00/05]: Containers(V2)- Introduction By: Cedric Le Goater on Wed, 20 September 2006 13:06
		Re: [patch00/05]: Containers(V2)- Introduction By: Rohit Seth on Wed, 20 September 2006 16:45
		Re: [patch00/05]: Containers(V2)- Introduction By: Christoph Lameter on Wed, 20 September 2006 16:48
		Re: [patch00/05]: Containers(V2)- Introduction By: Nick Piggin on Wed, 20 September 2006 17:07
		Re: [patch00/05]: Containers(V2)- Introduction By: Nick Piggin on Wed, 20 September 2006 16:56
		Re: [patch00/05]: Containers(V2)- Introduction By: Christoph Lameter on Wed, 20 September 2006 17:08
		Re: [patch00/05]: Containers(V2)- Introduction By: Nick Piggin on Wed, 20 September 2006 17:19
		Re: [patch00/05]: Containers(V2)- Introduction By: Alan Cox on Wed, 20 September 2006 17:16
		Re: [patch00/05]: Containers(V2)- Introduction By: Nick Piggin on Wed, 20 September 2006 17:00
		Re: [patch00/05]: Containers(V2)- Introduction By: Alan Cox on Wed, 20 September 2006 17:12
		Re: [patch00/05]: Containers(V2)- Introduction By: Nick Piggin on Wed, 20 September 2006 17:30
		Re: [patch00/05]: Containers(V2)- Introduction By: Rohit Seth on Wed, 20 September 2006 17:50
		Re: [patch00/05]: Containers(V2)- Introduction By: Christoph Lameter on Wed, 20 September 2006 17:52
		Re: [patch00/05]: Containers(V2)- Introduction By: Peter Zijlstra on Wed, 20 September 2006 18:06
		Re: [patch00/05]: Containers(V2)- Introduction By: Peter Zijlstra on Wed, 20 September 2006 18:37
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Paul Menage on Wed, 20 September 2006 17:23
		Re: [patch00/05]: Containers(V2)- Introduction By: Christoph Lameter on Wed, 20 September 2006 16:25
		Re: [patch00/05]: Containers(V2)- Introduction By: Alan Cox on Wed, 20 September 2006 17:10
		Re: [patch00/05]: Containers(V2)- Introduction By: Christoph Lameter on Wed, 20 September 2006 17:15
		Re: [patch00/05]: Containers(V2)- Introduction By: Alan Cox on Wed, 20 September 2006 17:24
		Re: [patch00/05]: Containers(V2)- Introduction By: Christoph Lameter on Wed, 20 September 2006 17:35
		Re: [patch00/05]: Containers(V2)- Introduction By: Paul Jackson on Wed, 20 September 2006 23:29
		Re: [patch00/05]: Containers(V2)- Introduction By: Paul Jackson on Wed, 20 September 2006 23:18
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Paul Menage on Wed, 20 September 2006 17:30
		Re: [patch00/05]: Containers(V2)- Introduction By: Rohit Seth on Wed, 20 September 2006 17:26
		Re: [patch00/05]: Containers(V2)- Introduction By: Christoph Lameter on Wed, 20 September 2006 17:38
		Re: [patch00/05]: Containers(V2)- Introduction By: Rohit Seth on Wed, 20 September 2006 18:07
		Re: [patch00/05]: Containers(V2)- Introduction By: Christoph Lameter on Wed, 20 September 2006 19:51
		Re: [patch00/05]: Containers(V2)- Introduction By: Paul Jackson on Wed, 20 September 2006 20:06
		Re: [patch00/05]: Containers(V2)- Introduction By: Paul Jackson on Wed, 20 September 2006 22:58
		Re: [patch00/05]: Containers(V2)- Introduction By: Christoph Lameter on Wed, 20 September 2006 23:02
		Re: [patch00/05]: Containers(V2)- Introduction By: Rohit Seth on Wed, 20 September 2006 23:33
		Re: [patch00/05]: Containers(V2)- Introduction By: Christoph Lameter on Wed, 20 September 2006 23:36
		Re: [patch00/05]: Containers(V2)- Introduction By: Rohit Seth on Wed, 20 September 2006 23:39
		Re: [patch00/05]: Containers(V2)- Introduction By: Christoph Lameter on Wed, 20 September 2006 23:51
		Re: [patch00/05]: Containers(V2)- Introduction By: Paul Jackson on Thu, 21 September 2006 00:05
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Paul Menage on Thu, 21 September 2006 00:09
		Re: [patch00/05]: Containers(V2)- Introduction By: Rohit Seth on Wed, 20 September 2006 23:26
		Re: [patch00/05]: Containers(V2)- Introduction By: Christoph Lameter on Wed, 20 September 2006 23:31
		Re: [Lhms-devel] [patch00/05]: Containers(V2)- Introduction By: KAMEZAWA Hiroyuki on Thu, 21 September 2006 00:47
		Re: [Lhms-devel] [patch00/05]: Containers(V2)- Introduction By: KAMEZAWA Hiroyuki on Thu, 21 September 2006 01:29
		Re: [ckrm-tech] [Lhms-devel] [patch00/05]: Containers(V2)- Introduction By: Paul Menage on Thu, 21 September 2006 01:36
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Paul Menage on Wed, 20 September 2006 17:42
		Re: [patch00/05]: Containers(V2)- Introduction By: Paul Jackson on Wed, 20 September 2006 22:51
		Re: [patch00/05]: Containers(V2)- Introduction By: Christoph Lameter on Wed, 20 September 2006 23:01
		Re: [patch00/05]: Containers(V2)- Introduction By: Rohit Seth on Wed, 20 September 2006 23:22
		Re: [patch00/05]: Containers(V2)- Introduction By: Paul Jackson on Wed, 20 September 2006 23:45
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Paul Menage on Wed, 20 September 2006 17:37
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Chandra Seetharaman on Wed, 20 September 2006 18:34
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Christoph Lameter on Wed, 20 September 2006 19:52
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Chandra Seetharaman on Thu, 21 September 2006 00:31
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Paul Jackson on Thu, 21 September 2006 00:36
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Paul Menage on Thu, 21 September 2006 00:42
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Paul Menage on Wed, 20 September 2006 18:43
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Chandra Seetharaman on Wed, 20 September 2006 19:09
		Re: [patch00/05]: Containers(V2)- Introduction By: Nick Piggin on Wed, 20 September 2006 16:44
		Re: [patch00/05]: Containers(V2)- Introduction By: Christoph Lameter on Wed, 20 September 2006 17:12
		Re: [patch00/05]: Containers(V2)- Introduction By: Paul Jackson on Wed, 20 September 2006 22:27
		Re: [patch00/05]: Containers(V2)- Introduction By: Christoph Lameter on Wed, 20 September 2006 22:59
		Re: [patch00/05]: Containers(V2)- Introduction By: Christoph Lameter on Wed, 20 September 2006 17:30
		Re: [patch00/05]: Containers(V2)- Introduction By: Nick Piggin on Wed, 20 September 2006 18:03
		Re: [patch00/05]: Containers(V2)- Introduction By: Rohit Seth on Wed, 20 September 2006 18:14
		Re: [patch00/05]: Containers(V2)- Introduction By: Peter Zijlstra on Wed, 20 September 2006 18:27
		Re: [patch00/05]: Containers(V2)- Introduction By: Rohit Seth on Wed, 20 September 2006 18:38
		Re: [patch00/05]: Containers(V2)- Introduction By: Rohit Seth on Wed, 20 September 2006 18:57
		Re: [patch00/05]: Containers(V2)- Introduction By: Paul Jackson on Wed, 20 September 2006 19:48
		Re: [patch00/05]: Containers(V2)- Introduction By: Christoph Lameter on Wed, 20 September 2006 19:48
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Paul Menage on Wed, 20 September 2006 19:51
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Chandra Seetharaman on Wed, 27 September 2006 19:50
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Rohit Seth on Wed, 27 September 2006 21:28
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Chandra Seetharaman on Wed, 27 September 2006 22:24
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Balbir Singh on Thu, 28 September 2006 08:01
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Rohit Seth on Thu, 28 September 2006 18:31
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Balbir Singh on Thu, 28 September 2006 21:53
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Rohit Seth on Fri, 29 September 2006 00:22
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Rohit Seth on Thu, 28 September 2006 18:12
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Chandra Seetharaman on Thu, 28 September 2006 20:23
		Re: [ckrm-tech] [patch00/05]: Containers(V2)- Introduction By: Rohit Seth on Thu, 28 September 2006 21:38

Previous Topic:	Re: Re: namespace and nsproxy syscalls
Next Topic:	[Lxc-devel] [patch 093/203] pidspace: is_init()

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

]

Current Time: Sun Nov 02 16:42:54 GMT 2025

Total time taken to generate the page: 0.10620 seconds