OpenVZ Forum: Devel » [PATCH 0/9] Containers (V9): Generic Process Containers

Home » Mailing lists » Devel » [PATCH 0/9] Containers (V9): Generic Process Containers

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

Re: [ckrm-tech] [PATCH 0/9] Containers (V9): Generic Process Containers [message #12474 is a reply to message #12473]

Mon, 30 April 2007 18:09

Srivatsa Vaddagiri
Messages: 241
Registered: August 2006

Senior Member

On Mon, Apr 30, 2007 at 06:23:52PM +0100, Christoph Hellwig wrote:
> Umm, no - please naje two versions with and without the lock.

Ok this patch may not be necessary (given Paul's observation that
read_lock will suffice in update_nodemask).

> Also Please fix up the codingstyle, the { belongs onto a line of it's own.

Paul, I suspect that there are a number of Codingstyle related changes you
may need to address in your next version (including the one pointed by
Christoph)

--
Regards,
vatsa

Report message to a moderator

[Message index]

		[PATCH 0/9] Containers (V9): Generic Process Containers By: Paul Menage on Fri, 27 April 2007 10:46
		[PATCH 9/9] Containers (V9): Simple debug info subsystem By: Paul Menage on Fri, 27 April 2007 10:46
		[PATCH 3/9] Containers (V9): Add tasks file interface By: Paul Menage on Fri, 27 April 2007 10:46
		Re: [PATCH 3/9] Containers (V9): Add tasks file interface By: Balbir Singh on Tue, 01 May 2007 18:12
		Re: [ckrm-tech] [PATCH 3/9] Containers (V9): Add tasks file interface By: Paul Menage on Tue, 01 May 2007 20:37
		Re: [ckrm-tech] [PATCH 3/9] Containers (V9): Add tasks file interface By: Srivatsa Vaddagiri on Wed, 02 May 2007 03:17
		Re: [ckrm-tech] [PATCH 3/9] Containers (V9): Add tasks file interface By: Paul Menage on Wed, 02 May 2007 03:25
		Re: [ckrm-tech] [PATCH 3/9] Containers (V9): Add tasks file interface By: Srivatsa Vaddagiri on Wed, 02 May 2007 03:38
		Re: [ckrm-tech] [PATCH 3/9] Containers (V9): Add tasks file interface By: Balbir Singh on Tue, 08 May 2007 14:51
		Re: [ckrm-tech] [PATCH 3/9] Containers (V9): Add tasks file interface By: Paul Menage on Thu, 10 May 2007 21:21
		Re: [ckrm-tech] [PATCH 3/9] Containers (V9): Add tasks file interface By: Balbir Singh on Fri, 11 May 2007 02:31
		Re: [ckrm-tech] [PATCH 3/9] Containers (V9): Add tasks file interface By: Balbir Singh on Wed, 02 May 2007 03:58
		[PATCH 1/9] Containers (V9): Basic container framework By: Paul Menage on Fri, 27 April 2007 10:46
		Re: [ckrm-tech] [PATCH 1/9] Containers (V9): Basic container framework By: Paul Jackson on Sun, 29 April 2007 03:12
		Re: [PATCH 1/9] Containers (V9): Basic container framework By: Paul Menage on Tue, 01 May 2007 17:46
		Re: [PATCH 1/9] Containers (V9): Basic container framework By: Balbir Singh on Tue, 01 May 2007 17:40
		Re: [ckrm-tech] [PATCH 1/9] Containers (V9): Basic container framework By: Paul Jackson on Tue, 01 May 2007 18:40
		Re: [ckrm-tech] [PATCH 1/9] Containers (V9): Basic container framework By: Balbir Singh on Wed, 02 May 2007 03:44
		Re: [ckrm-tech] [PATCH 1/9] Containers (V9): Basic container framework By: Paul Jackson on Wed, 02 May 2007 06:12
		Re: [ckrm-tech] [PATCH 1/9] Containers (V9): Basic container framework By: Balbir Singh on Thu, 10 May 2007 04:09
		Re: [ckrm-tech] [PATCH 1/9] Containers (V9): Basic container framework By: Paul Jackson on Thu, 10 May 2007 04:47
		Re: [ckrm-tech] [PATCH 1/9] Containers (V9): Basic container framework By: Balbir Singh on Thu, 10 May 2007 04:49
		[PATCH 7/9] Containers (V9): Make cpusets a client of containers By: Paul Menage on Fri, 27 April 2007 10:46
		[PATCH 4/9] Containers (V9): Add fork/exit hooks By: Paul Menage on Fri, 27 April 2007 10:46
		[PATCH 8/9] Containers (V9): Share css_group arrays between tasks with same container memberships By: Paul Menage on Fri, 27 April 2007 10:46
		[PATCH 6/9] Containers (V9): Add procfs interface By: Paul Menage on Fri, 27 April 2007 10:46
		[PATCH 2/9] Containers (V9): Example CPU accounting subsystem By: Paul Menage on Fri, 27 April 2007 10:46
		Re: [PATCH 2/9] Containers (V9): Example CPU accounting subsystem By: Balbir Singh on Tue, 01 May 2007 17:52
		Re: [PATCH 0/9] Containers (V9): Generic Process Containers By: Paul Jackson on Sun, 29 April 2007 01:47
		Re: [PATCH 0/9] Containers (V9): Generic Process Containers By: Paul Jackson on Sun, 29 April 2007 09:37
		Re: [PATCH 0/9] Containers (V9): Generic Process Containers By: Srivatsa Vaddagiri on Mon, 30 April 2007 17:04
		Re: [PATCH 0/9] Containers (V9): Generic Process Containers By: Paul Menage on Mon, 30 April 2007 17:09
		Re: [PATCH 0/9] Containers (V9): Generic Process Containers By: Srivatsa Vaddagiri on Mon, 30 April 2007 17:59
		Re: [PATCH 0/9] Containers (V9): Generic Process Containers By: Christoph Hellwig on Mon, 30 April 2007 17:23
		Re: [ckrm-tech] [PATCH 0/9] Containers (V9): Generic Process Containers By: Srivatsa Vaddagiri on Mon, 30 April 2007 18:09

Previous Topic:	[PATCH -utrace] Move utrace into task_struct
Next Topic:	[patch 39/68] attach_pid() with struct pid parameter

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

]

Current Time: Tue Aug 05 06:10:38 GMT 2025

Total time taken to generate the page: 1.04305 seconds