OpenVZ Forum: Devel » [PATCH 0/8] RSS controller based on process containers (v2)

Home » Mailing lists » Devel » [PATCH 0/8] RSS controller based on process containers (v2)

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

Re: [PATCH 2/8] Add container pointer on struct page [message #11981 is a reply to message #11837]

Fri, 13 April 2007 14:48

xemul
Messages: 248
Registered: November 2005

Senior Member

Jean-Pierre Dion wrote:
> Hi Pavel,
>
> I have been implied in the work for the
> memory controller of res groups a few months ago.
>
> I see that you propose to modify the struct
> page to point to rss container struct.
> This has made some debate because of the struct
> page size increase, but this allows a quicker
> scan to reclaim pages (I mean having per-container
> lists of active/inactive pages).
> We (here at Bull and others) proposed this implementation
> for res groups and I am interested in knowing
> if this has a chance of being accepted today (hope so).

So do I :) I'm not the one who makes the final decision ;)

> I know this uses memory for internal management
> and increases a lot the memory size used for
> a large memory configuration, but in that case
> we have lot of memory, so where is the issue ?
> We tested this on a 28 GB server and it worked.

Thank you for additional testing on enterprise servers!
Hope this will be a good argument in favour of the patches.

> Also we can use larger page size to reduce
> the overhead, and I believe this makes sense
> on large servers with big memory.
>
> So we balance between using more memory internally
> and so getting faster access to pages for reclaim,
> or do nothing. ;-)

That's right. I made some small testing which showed
that moving this pointer in a mirrored array saves less
than 0.1% of performance on 4CPU i386 node. I don't know
how this will be on enterprise hardware, but I do believe
that the results will be the same (or even better).

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

Report message to a moderator

[Message index]

		[PATCH 0/8] RSS controller based on process containers (v2) By: xemul on Mon, 09 April 2007 12:18
		[PATCH 1/8] Resource counters By: xemul on Mon, 09 April 2007 12:31
		[PATCH 2/8] Add container pointer on struct page By: xemul on Mon, 09 April 2007 12:37
		Re: [PATCH 2/8] Add container pointer on struct page By: xemul on Fri, 13 April 2007 14:48
		Re: [PATCH 2/8] Add container pointer on struct page By: Jean-Pierre Dion on Fri, 13 April 2007 13:56
		[PATCH 3/8] Add container pointer on mm_struct By: xemul on Mon, 09 April 2007 12:43
		[PATCH 4/8] RSS container core By: xemul on Mon, 09 April 2007 12:45
		[PATCH 5/8] RSS accounting hooks over the code By: xemul on Mon, 09 April 2007 12:50
		[PATCH 6/8] Per container OOM killer By: xemul on Mon, 09 April 2007 12:52
		[PATCH 7/8] Page scanner changes needed to implement per-container scanner By: xemul on Mon, 09 April 2007 12:57
		[PATCH 8/8] Per-container pages reclamation By: xemul on Mon, 09 April 2007 12:59
		Re: [PATCH 8/8] Per-container pages reclamation By: xemul on Tue, 24 April 2007 10:29
		Re: [PATCH 8/8] Per-container pages reclamation By: Balbir Singh on Tue, 24 April 2007 11:01
		Re: [PATCH 8/8] Per-container pages reclamation By: xemul on Tue, 24 April 2007 11:37
		Re: [PATCH 8/8] Per-container pages reclamation By: Balbir Singh on Tue, 24 April 2007 09:47
		Re: [PATCH 8/8] Per-container pages reclamation By: Balbir Singh on Wed, 02 May 2007 09:51
		Re: [PATCH 8/8] Per-container pages reclamation By: Balbir Singh on Thu, 17 May 2007 11:31
		Re: [PATCH 8/8] Per-container pages reclamation By: xemul on Mon, 21 May 2007 15:15
		Re: [PATCH 8/8] Per-container pages reclamation By: Balbir Singh on Thu, 24 May 2007 07:59
		Re: [PATCH 0/8] RSS controller based on process containers (v2) By: Peter Zijlstra on Mon, 09 April 2007 15:54
		Re: [PATCH 0/8] RSS controller based on process containers (v2) By: xemul on Tue, 10 April 2007 08:27
		Re: [PATCH 0/8] RSS controller based on process containers (v2) By: Vaidyanathan Srinivas on Thu, 19 April 2007 05:37

Previous Topic:	Re: [RFC] [PATCH 0/3] Add group fairness to CFS
Next Topic:	RSS controller v2 Test results (lmbench )

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

]

Current Time: Fri Dec 05 05:03:23 GMT 2025

Total time taken to generate the page: 0.17352 seconds