cpuacct.stat in no-root cgroups shows user time without guest time
included int it. This doesn't match with user time shown in root
cpuacct.stat and /proc/<pid>/stat.

Make account_guest_time() to add user time to cgroup's cpustat to
fix this.

Fixes: ef12fefabf94 ("cpuacct: add per-cgroup utime/stime statistics")
Signed-off-by: Andrey Ryabinin <a...@yandex-team.com>
Cc: <sta...@vger.kernel.org>
---
 kernel/sched/cputime.c | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/kernel/sched/cputime.c b/kernel/sched/cputime.c
index 5f611658eeab..95a9c5603d29 100644
--- a/kernel/sched/cputime.c
+++ b/kernel/sched/cputime.c
@@ -139,8 +139,6 @@ void account_user_time(struct task_struct *p, u64 cputime)
  */
 void account_guest_time(struct task_struct *p, u64 cputime)
 {
-       u64 *cpustat = kcpustat_this_cpu->cpustat;
-
        /* Add guest time to process. */
        p->utime += cputime;
        account_group_user_time(p, cputime);
@@ -148,11 +146,11 @@ void account_guest_time(struct task_struct *p, u64 
cputime)
 
        /* Add guest time to cpustat. */
        if (task_nice(p) > 0) {
-               cpustat[CPUTIME_NICE] += cputime;
-               cpustat[CPUTIME_GUEST_NICE] += cputime;
+               task_group_account_field(p, CPUTIME_NICE, cputime);
+               task_group_account_field(p, CPUTIME_GUEST_NICE, cputime);
        } else {
-               cpustat[CPUTIME_USER] += cputime;
-               cpustat[CPUTIME_GUEST] += cputime;
+               task_group_account_field(p, CPUTIME_USER, cputime);
+               task_group_account_field(p, CPUTIME_GUEST, cputime);
        }
 }
 
-- 
2.26.2

Reply via email to