[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <m13awhb5mc.fsf@ebiederm.dsl.xmission.com>
Date: Thu, 11 Oct 2007 17:20:43 -0600
From: ebiederm@...ssion.com (Eric W. Biederman)
To: Paul Jackson <pj@....com>
Cc: "Paul Menage" <menage@...gle.com>, rientjes@...gle.com,
akpm@...ux-foundation.org, nickpiggin@...oo.com.au,
a.p.zijlstra@...llo.nl, serue@...ibm.com, clg@...ibm.com,
linux-kernel@...r.kernel.org, svaidy@...ux.vnet.ibm.com,
xemul@...nvz.org, containers@...ts.osdl.org,
balbir@...ux.vnet.ibm.com
Subject: Re: [PATCH] task containersv11 add tasks file interface fix for cpusets
Stupid question.
Would it help at all if we structured this as:
- Take the control group off of the cpus and runqueues.
- Modify the tasks in the control group.
- Place the control group back on the runqueues.
Essentially this is what ptrace does (except for one task at a time).
Since we know that the tasks are not running, and that we have
exclusive access to the tasks in the control group we can take action
as if we were the actual tasks themselves. Which should simplify
locking.
So I think with just a touch of care we can significantly simplify the
locking by being much more coarse grained, at the expense of this kind
of control group operation being more expensive. Plus we would have
an operation that was reusable.
I think the stop everything. modify everything, start everything
could also be used instead of an array of task structures.
Eric
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists