Re: [PATCH] Hookup group-scheduler with task container infrastructure [message #20012] |
Mon, 10 September 2007 22:45  |
Paul Menage
Messages: 642 Registered: September 2006
|
Senior Member |
|
|
On 9/10/07, Dmitry Adamushko <dmitry.adamushko@gmail.com> wrote:
> On 10/09/2007, Srivatsa Vaddagiri <vatsa@linux.vnet.ibm.com> wrote:
> > On Mon, Sep 10, 2007 at 10:22:59AM -0700, Andrew Morton wrote:
> > > objection ;) "cpuctlr" isn't memorable. Kernel code is write-rarely,
> > > read-often. "cpu_controller", please. The extra typing is worth it ;)
> >
> > Ok! Here's the modified patch (against 2.6.23-rc4-mm1).
>
> as everyone seems to be in a quest for a better name... I think, the
> obvious one would be just 'group_sched'.
>
But "sched" on its own could refer to CPU scheduling, I/O scheduling,
network scheduling, ...
And "group" is more or less implied by the fact that it's in the
containers/control groups filesystem.
So "group_sched" isn't really all that informative. The name should
definitely contain either "cpu" or "cfs".
Paul
_______________________________________________
Containers mailing list
Containers@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers
|
|
|
Re: [PATCH] Hookup group-scheduler with task container infrastructure [message #20036 is a reply to message #20012] |
Tue, 11 September 2007 07:20   |
Cedric Le Goater
Messages: 443 Registered: February 2006
|
Senior Member |
|
|
Paul Menage wrote:
> On 9/10/07, Dmitry Adamushko <dmitry.adamushko@gmail.com> wrote:
>> On 10/09/2007, Srivatsa Vaddagiri <vatsa@linux.vnet.ibm.com> wrote:
>>> On Mon, Sep 10, 2007 at 10:22:59AM -0700, Andrew Morton wrote:
>>>> objection ;) "cpuctlr" isn't memorable. Kernel code is write-rarely,
>>>> read-often. "cpu_controller", please. The extra typing is worth it ;)
>>> Ok! Here's the modified patch (against 2.6.23-rc4-mm1).
>> as everyone seems to be in a quest for a better name... I think, the
>> obvious one would be just 'group_sched'.
>>
>
> But "sched" on its own could refer to CPU scheduling, I/O scheduling,
> network scheduling, ...
>
> And "group" is more or less implied by the fact that it's in the
> containers/control groups filesystem.
"control groups" is the name of your framework. right ?
> So "group_sched" isn't really all that informative. The name should
> definitely contain either "cpu" or "cfs".
"cfs" control group subsystem.
"cfs" looks good enough to identify the subsystem.
C.
_______________________________________________
Containers mailing list
Containers@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers
|
|
|
Re: [PATCH] Hookup group-scheduler with task container infrastructure [message #20050 is a reply to message #20036] |
Tue, 11 September 2007 15:22   |
Randy Dunlap
Messages: 25 Registered: April 2007
|
Junior Member |
|
|
On Tue, 11 Sep 2007 09:20:33 +0200 Cedric Le Goater wrote:
> Paul Menage wrote:
> > On 9/10/07, Dmitry Adamushko <dmitry.adamushko@gmail.com> wrote:
> >> On 10/09/2007, Srivatsa Vaddagiri <vatsa@linux.vnet.ibm.com> wrote:
> >>> On Mon, Sep 10, 2007 at 10:22:59AM -0700, Andrew Morton wrote:
> >>>> objection ;) "cpuctlr" isn't memorable. Kernel code is write-rarely,
> >>>> read-often. "cpu_controller", please. The extra typing is worth it ;)
> >>> Ok! Here's the modified patch (against 2.6.23-rc4-mm1).
> >> as everyone seems to be in a quest for a better name... I think, the
> >> obvious one would be just 'group_sched'.
> >>
> >
> > But "sched" on its own could refer to CPU scheduling, I/O scheduling,
> > network scheduling, ...
> >
> > And "group" is more or less implied by the fact that it's in the
> > containers/control groups filesystem.
>
> "control groups" is the name of your framework. right ?
>
> > So "group_sched" isn't really all that informative. The name should
> > definitely contain either "cpu" or "cfs".
>
> "cfs" control group subsystem.
That looks odd, like it's a filesystem.
What does cfs really mean?
> "cfs" looks good enough to identify the subsystem.
---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
_______________________________________________
Containers mailing list
Containers@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers
|
|
|
|
|
|
Re: [PATCH] Hookup group-scheduler with task container infrastructure [message #20056 is a reply to message #20053] |
Tue, 11 September 2007 15:53  |
Randy Dunlap
Messages: 25 Registered: April 2007
|
Junior Member |
|
|
On Tue, 11 Sep 2007 21:21:19 +0530 Srivatsa Vaddagiri wrote:
> On Tue, Sep 11, 2007 at 08:22:43AM -0700, Randy Dunlap wrote:
> > That looks odd, like it's a filesystem.
> > What does cfs really mean?
>
> cfs = completely fair scheduler :)
>
> In this thread, we are talking of hooking the cfs cpu scheduler with the
> task-container framework in -mm tree, so that the scheduler can deal
> with groups of tasks rather than just tasks, while handling fairness of
> cpu allocation.
>
> I agree "cfs" control subsystem does look odd a bit here. "cpu" control
> subsystem seems better.
Thanks. I agree that using "cpu" is better. I.e., don't tie it
to a particular scheduler name. It would just need to change the
next time we have a new scheduler. ;)
---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
_______________________________________________
Containers mailing list
Containers@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers
|
|
|