Re: Revert for cgroups CPU accounting subsystem patch

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Srivatsa Vaddagiri
Date: Tuesday, November 13, 2007 - 12:48 am

On Mon, Nov 12, 2007 at 10:05:24PM -0800, Paul Menage wrote:

That's still introducing a deviation from the normal behavior we would
have had we allowed all tasks to be part of the same "control" group/runqueue.

For ex: using nice value to vary bandwidth between tasks makes sense if
they are all part of the same group.

Also an application with more tasks will get more cpu power (as intended)
compared to another app with less tasks, provided they are all
part of the same group.

Regarding your concern about tracking cpu usage in different ways, it
could be mitigated if we have cpuacct controller track usage as per
information present in a task's sched entity structure
(tsk->se.sum_exec_runtime) i.e call cpuacct_charge() from
__update_curr() which would accumulate the execution time of the 
group in a SMP friendly manner (i.e dump it in a per-cpu per-group counter 
first and then aggregate to a global per-group counter).

This will let account and control grouping to be independent if desired.

What do you think?


-- 
Regards,
vatsa
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Revert for cgroups CPU accounting subsystem patch, Paul Menage, (Mon Nov 12, 10:25 pm)
Re: Revert for cgroups CPU accounting subsystem patch, Srivatsa Vaddagiri, (Mon Nov 12, 11:00 pm)
Re: Revert for cgroups CPU accounting subsystem patch, Paul Menage, (Mon Nov 12, 11:05 pm)
Re: Revert for cgroups CPU accounting subsystem patch, Balbir Singh, (Tue Nov 13, 12:00 am)
Re: Revert for cgroups CPU accounting subsystem patch, Paul Menage, (Tue Nov 13, 12:10 am)
Re: Revert for cgroups CPU accounting subsystem patch, Balbir Singh, (Tue Nov 13, 12:29 am)
Re: Revert for cgroups CPU accounting subsystem patch, Paul Menage, (Tue Nov 13, 12:34 am)
Re: Revert for cgroups CPU accounting subsystem patch, Srivatsa Vaddagiri, (Tue Nov 13, 12:48 am)
Re: Revert for cgroups CPU accounting subsystem patch, Paul Menage, (Tue Nov 13, 12:57 am)
Re: Revert for cgroups CPU accounting subsystem patch, Srivatsa Vaddagiri, (Tue Nov 13, 12:59 am)
Re: Revert for cgroups CPU accounting subsystem patch, Paul Menage, (Tue Nov 13, 12:59 am)
[PATCH] sched: cpu accounting controller, Srivatsa Vaddagiri, (Thu Nov 29, 12:17 pm)
Re: [PATCH] sched: cpu accounting controller, Ingo Molnar, (Thu Nov 29, 12:20 pm)
Re: [PATCH] sched: cpu accounting controller, Andrew Morton, (Thu Nov 29, 12:30 pm)
Re: [PATCH] sched: cpu accounting controller, Srivatsa Vaddagiri, (Thu Nov 29, 12:39 pm)
Re: [PATCH] sched: cpu accounting controller, Srivatsa Vaddagiri, (Thu Nov 29, 1:18 pm)
Re: [PATCH] sched: cpu accounting controller (V2), Ingo Molnar, (Fri Nov 30, 5:35 am)
[PATCH] sched: cpu accounting controller (V2), Srivatsa Vaddagiri, (Fri Nov 30, 5:42 am)
Re: [PATCH] sched: cpu accounting controller (V2), Balbir Singh, (Fri Nov 30, 5:45 am)
Re: [PATCH] sched: cpu accounting controller (V2), Srivatsa Vaddagiri, (Fri Nov 30, 6:09 am)
Re: [PATCH] sched: cpu accounting controller (V2), Ingo Molnar, (Fri Nov 30, 6:34 am)
Re: [PATCH] sched: cpu accounting controller (V2), Balbir Singh, (Fri Nov 30, 7:00 am)
Re: [PATCH] sched: cpu accounting controller (V2), Ingo Molnar, (Fri Nov 30, 12:46 pm)
Re: [PATCH] sched: cpu accounting controller (V2), Paul Menage, (Sat Dec 1, 12:48 am)
Re: [PATCH] sched: cpu accounting controller (V2), Balbir Singh, (Sat Dec 1, 2:51 am)