sched: Remove task_{u,s,g}time()

Now all task_{u,s}time() pairs are replaced by task_times().
And task_gtime() is too simple to be an inline function.

Cleanup them all.

Signed-off-by: Hidetoshi Seto <seto.hidetoshi@jp.fujitsu.com>
Acked-by: Peter Zijlstra <peterz@infradead.org>
Cc: Stanislaw Gruszka <sgruszka@redhat.com>
Cc: Spencer Candland <spencer@bluehost.com>
Cc: Oleg Nesterov <oleg@redhat.com>
Cc: Balbir Singh <balbir@in.ibm.com>
Cc: Americo Wang <xiyou.wangcong@gmail.com>
LKML-Reference: <4B0E16D1.70902@jp.fujitsu.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>

diff --git a/fs/proc/array.c b/fs/proc/array.c
index 330deda..ca61a88 100644 (file)
--- a/fs/proc/array.c
+++ b/fs/proc/array.c
@@ -511,7 +511,7 @@ static int do_task_stat(struct seq_file *m, struct pid_namespace *ns,
                        do {
                                min_flt += t->min_flt;
                                maj_flt += t->maj_flt;
-                               gtime = cputime_add(gtime, task_gtime(t));
+                               gtime = cputime_add(gtime, t->gtime);
                                t = next_thread(t);
                        } while (t != task);
 
@@ -536,7 +536,7 @@ static int do_task_stat(struct seq_file *m, struct pid_namespace *ns,
                min_flt = task->min_flt;
                maj_flt = task->maj_flt;
                task_times(task, &utime, &stime);
-               gtime = task_gtime(task);
+               gtime = task->gtime;
        }
 
        /* scale priority and nice values from timeslices to -20..20 */

diff --git a/include/linux/sched.h b/include/linux/sched.h
index fe6ae15..0395b0f 100644 (file)
--- a/include/linux/sched.h
+++ b/include/linux/sched.h
@@ -1720,9 +1720,6 @@ static inline void put_task_struct(struct task_struct *t)
                __put_task_struct(t);
 }
 
-extern cputime_t task_utime(struct task_struct *p);
-extern cputime_t task_stime(struct task_struct *p);
-extern cputime_t task_gtime(struct task_struct *p);
 extern void task_times(struct task_struct *p, cputime_t *ut, cputime_t *st);
 
 /*

diff --git a/kernel/exit.c b/kernel/exit.c
index 29068ab..2eaf68b 100644 (file)
--- a/kernel/exit.c
+++ b/kernel/exit.c
@@ -115,7 +115,7 @@ static void __exit_signal(struct task_struct *tsk)
                task_times(tsk, &utime, &stime);
                sig->utime = cputime_add(sig->utime, utime);
                sig->stime = cputime_add(sig->stime, stime);
-               sig->gtime = cputime_add(sig->gtime, task_gtime(tsk));
+               sig->gtime = cputime_add(sig->gtime, tsk->gtime);
                sig->min_flt += tsk->min_flt;
                sig->maj_flt += tsk->maj_flt;
                sig->nvcsw += tsk->nvcsw;

diff --git a/kernel/sched.c b/kernel/sched.c
index 475a6f2..82251c2 100644 (file)
--- a/kernel/sched.c
+++ b/kernel/sched.c
@@ -5182,22 +5182,12 @@ void account_idle_ticks(unsigned long ticks)
  * Use precise platform statistics if available:
  */
 #ifdef CONFIG_VIRT_CPU_ACCOUNTING
-cputime_t task_utime(struct task_struct *p)
-{
-       return p->utime;
-}
-
-cputime_t task_stime(struct task_struct *p)
-{
-       return p->stime;
-}
-
 void task_times(struct task_struct *p, cputime_t *ut, cputime_t *st)
 {
        if (ut)
-               *ut = task_utime(p);
+               *ut = p->utime;
        if (st)
-               *st = task_stime(p);
+               *st = p->stime;
 }
 #else
 
@@ -5235,27 +5225,8 @@ void task_times(struct task_struct *p, cputime_t *ut, cputime_t *st)
        if (st)
                *st = p->prev_stime;
 }
-
-cputime_t task_utime(struct task_struct *p)
-{
-       cputime_t utime;
-       task_times(p, &utime, NULL);
-       return utime;
-}
-
-cputime_t task_stime(struct task_struct *p)
-{
-       cputime_t stime;
-       task_times(p, NULL, &stime);
-       return stime;
-}
 #endif
 
-inline cputime_t task_gtime(struct task_struct *p)
-{
-       return p->gtime;
-}
-
 /*
  * This function gets called by the timer code, with HZ frequency.
  * We call it with interrupts disabled.