Commit 8ea02606 authored by Ingo Molnar's avatar Ingo Molnar

[PATCH] sched: fix up fs/proc/array.c whitespace problems

while changing task_stime() i noticed a whitespace style problem in
array.c - fix it. While at it, fix all the other style problems too,
most of them in the scheduler-stats related portions of array.c.

There is no change in functionality:

   text    data     bss     dec     hex filename
   4356      28       0    4384    1120 array.o-before
   4356      28       0    4384    1120 array.o-after
Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
parent e4af30be
...@@ -62,6 +62,8 @@ ...@@ -62,6 +62,8 @@
#include <linux/mman.h> #include <linux/mman.h>
#include <linux/proc_fs.h> #include <linux/proc_fs.h>
#include <linux/ioport.h> #include <linux/ioport.h>
#include <linux/uaccess.h>
#include <linux/io.h>
#include <linux/mm.h> #include <linux/mm.h>
#include <linux/hugetlb.h> #include <linux/hugetlb.h>
#include <linux/pagemap.h> #include <linux/pagemap.h>
...@@ -76,9 +78,7 @@ ...@@ -76,9 +78,7 @@
#include <linux/rcupdate.h> #include <linux/rcupdate.h>
#include <linux/delayacct.h> #include <linux/delayacct.h>
#include <asm/uaccess.h>
#include <asm/pgtable.h> #include <asm/pgtable.h>
#include <asm/io.h>
#include <asm/processor.h> #include <asm/processor.h>
#include "internal.h" #include "internal.h"
...@@ -87,10 +87,10 @@ ...@@ -87,10 +87,10 @@
do { memcpy(buffer, string, strlen(string)); \ do { memcpy(buffer, string, strlen(string)); \
buffer += strlen(string); } while (0) buffer += strlen(string); } while (0)
static inline char * task_name(struct task_struct *p, char * buf) static inline char *task_name(struct task_struct *p, char *buf)
{ {
int i; int i;
char * name; char *name;
char tcomm[sizeof(p->comm)]; char tcomm[sizeof(p->comm)];
get_task_comm(tcomm, p); get_task_comm(tcomm, p);
...@@ -138,7 +138,7 @@ static const char *task_state_array[] = { ...@@ -138,7 +138,7 @@ static const char *task_state_array[] = {
"X (dead)" /* 32 */ "X (dead)" /* 32 */
}; };
static inline const char * get_task_state(struct task_struct *tsk) static inline const char *get_task_state(struct task_struct *tsk)
{ {
unsigned int state = (tsk->state & (TASK_RUNNING | unsigned int state = (tsk->state & (TASK_RUNNING |
TASK_INTERRUPTIBLE | TASK_INTERRUPTIBLE |
...@@ -156,7 +156,7 @@ static inline const char * get_task_state(struct task_struct *tsk) ...@@ -156,7 +156,7 @@ static inline const char * get_task_state(struct task_struct *tsk)
return *p; return *p;
} }
static inline char * task_state(struct task_struct *p, char *buffer) static inline char *task_state(struct task_struct *p, char *buffer)
{ {
struct group_info *group_info; struct group_info *group_info;
int g; int g;
...@@ -172,8 +172,8 @@ static inline char * task_state(struct task_struct *p, char *buffer) ...@@ -172,8 +172,8 @@ static inline char * task_state(struct task_struct *p, char *buffer)
"Uid:\t%d\t%d\t%d\t%d\n" "Uid:\t%d\t%d\t%d\t%d\n"
"Gid:\t%d\t%d\t%d\t%d\n", "Gid:\t%d\t%d\t%d\t%d\n",
get_task_state(p), get_task_state(p),
p->tgid, p->pid, p->tgid, p->pid,
pid_alive(p) ? rcu_dereference(p->real_parent)->tgid : 0, pid_alive(p) ? rcu_dereference(p->real_parent)->tgid : 0,
pid_alive(p) && p->ptrace ? rcu_dereference(p->parent)->pid : 0, pid_alive(p) && p->ptrace ? rcu_dereference(p->parent)->pid : 0,
p->uid, p->euid, p->suid, p->fsuid, p->uid, p->euid, p->suid, p->fsuid,
p->gid, p->egid, p->sgid, p->fsgid); p->gid, p->egid, p->sgid, p->fsgid);
...@@ -191,15 +191,15 @@ static inline char * task_state(struct task_struct *p, char *buffer) ...@@ -191,15 +191,15 @@ static inline char * task_state(struct task_struct *p, char *buffer)
get_group_info(group_info); get_group_info(group_info);
task_unlock(p); task_unlock(p);
for (g = 0; g < min(group_info->ngroups,NGROUPS_SMALL); g++) for (g = 0; g < min(group_info->ngroups, NGROUPS_SMALL); g++)
buffer += sprintf(buffer, "%d ", GROUP_AT(group_info,g)); buffer += sprintf(buffer, "%d ", GROUP_AT(group_info, g));
put_group_info(group_info); put_group_info(group_info);
buffer += sprintf(buffer, "\n"); buffer += sprintf(buffer, "\n");
return buffer; return buffer;
} }
static char * render_sigset_t(const char *header, sigset_t *set, char *buffer) static char *render_sigset_t(const char *header, sigset_t *set, char *buffer)
{ {
int i, len; int i, len;
...@@ -239,7 +239,7 @@ static void collect_sigign_sigcatch(struct task_struct *p, sigset_t *ign, ...@@ -239,7 +239,7 @@ static void collect_sigign_sigcatch(struct task_struct *p, sigset_t *ign,
} }
} }
static inline char * task_sig(struct task_struct *p, char *buffer) static inline char *task_sig(struct task_struct *p, char *buffer)
{ {
unsigned long flags; unsigned long flags;
sigset_t pending, shpending, blocked, ignored, caught; sigset_t pending, shpending, blocked, ignored, caught;
...@@ -289,14 +289,14 @@ static inline char *task_cap(struct task_struct *p, char *buffer) ...@@ -289,14 +289,14 @@ static inline char *task_cap(struct task_struct *p, char *buffer)
cap_t(p->cap_effective)); cap_t(p->cap_effective));
} }
int proc_pid_status(struct task_struct *task, char * buffer) int proc_pid_status(struct task_struct *task, char *buffer)
{ {
char * orig = buffer; char *orig = buffer;
struct mm_struct *mm = get_task_mm(task); struct mm_struct *mm = get_task_mm(task);
buffer = task_name(task, buffer); buffer = task_name(task, buffer);
buffer = task_state(task, buffer); buffer = task_state(task, buffer);
if (mm) { if (mm) {
buffer = task_mem(mm, buffer); buffer = task_mem(mm, buffer);
mmput(mm); mmput(mm);
...@@ -344,8 +344,7 @@ static clock_t task_stime(struct task_struct *p) ...@@ -344,8 +344,7 @@ static clock_t task_stime(struct task_struct *p)
return stime; return stime;
} }
static int do_task_stat(struct task_struct *task, char *buffer, int whole)
static int do_task_stat(struct task_struct *task, char * buffer, int whole)
{ {
unsigned long vsize, eip, esp, wchan = ~0UL; unsigned long vsize, eip, esp, wchan = ~0UL;
long priority, nice; long priority, nice;
...@@ -353,7 +352,7 @@ static int do_task_stat(struct task_struct *task, char * buffer, int whole) ...@@ -353,7 +352,7 @@ static int do_task_stat(struct task_struct *task, char * buffer, int whole)
sigset_t sigign, sigcatch; sigset_t sigign, sigcatch;
char state; char state;
int res; int res;
pid_t ppid = 0, pgid = -1, sid = -1; pid_t ppid = 0, pgid = -1, sid = -1;
int num_threads = 0; int num_threads = 0;
struct mm_struct *mm; struct mm_struct *mm;
unsigned long long start_time; unsigned long long start_time;
...@@ -424,7 +423,7 @@ static int do_task_stat(struct task_struct *task, char * buffer, int whole) ...@@ -424,7 +423,7 @@ static int do_task_stat(struct task_struct *task, char * buffer, int whole)
} }
rcu_read_unlock(); rcu_read_unlock();
if (!whole || num_threads<2) if (!whole || num_threads < 2)
wchan = get_wchan(task); wchan = get_wchan(task);
if (!whole) { if (!whole) {
min_flt = task->min_flt; min_flt = task->min_flt;
...@@ -445,7 +444,7 @@ static int do_task_stat(struct task_struct *task, char * buffer, int whole) ...@@ -445,7 +444,7 @@ static int do_task_stat(struct task_struct *task, char * buffer, int whole)
/* convert nsec -> ticks */ /* convert nsec -> ticks */
start_time = nsec_to_clock_t(start_time); start_time = nsec_to_clock_t(start_time);
res = sprintf(buffer,"%d (%s) %c %d %d %d %d %d %u %lu \ res = sprintf(buffer, "%d (%s) %c %d %d %d %d %d %u %lu \
%lu %lu %lu %lu %lu %ld %ld %ld %ld %d 0 %llu %lu %ld %lu %lu %lu %lu %lu \ %lu %lu %lu %lu %lu %ld %ld %ld %ld %d 0 %llu %lu %ld %lu %lu %lu %lu %lu \
%lu %lu %lu %lu %lu %lu %lu %lu %d %d %u %u %llu\n", %lu %lu %lu %lu %lu %lu %lu %lu %d %d %u %u %llu\n",
task->pid, task->pid,
...@@ -471,7 +470,7 @@ static int do_task_stat(struct task_struct *task, char * buffer, int whole) ...@@ -471,7 +470,7 @@ static int do_task_stat(struct task_struct *task, char * buffer, int whole)
start_time, start_time,
vsize, vsize,
mm ? get_mm_rss(mm) : 0, mm ? get_mm_rss(mm) : 0,
rsslim, rsslim,
mm ? mm->start_code : 0, mm ? mm->start_code : 0,
mm ? mm->end_code : 0, mm ? mm->end_code : 0,
mm ? mm->start_stack : 0, mm ? mm->start_stack : 0,
...@@ -493,17 +492,17 @@ static int do_task_stat(struct task_struct *task, char * buffer, int whole) ...@@ -493,17 +492,17 @@ static int do_task_stat(struct task_struct *task, char * buffer, int whole)
task->rt_priority, task->rt_priority,
task->policy, task->policy,
(unsigned long long)delayacct_blkio_ticks(task)); (unsigned long long)delayacct_blkio_ticks(task));
if(mm) if (mm)
mmput(mm); mmput(mm);
return res; return res;
} }
int proc_tid_stat(struct task_struct *task, char * buffer) int proc_tid_stat(struct task_struct *task, char *buffer)
{ {
return do_task_stat(task, buffer, 0); return do_task_stat(task, buffer, 0);
} }
int proc_tgid_stat(struct task_struct *task, char * buffer) int proc_tgid_stat(struct task_struct *task, char *buffer)
{ {
return do_task_stat(task, buffer, 1); return do_task_stat(task, buffer, 1);
} }
...@@ -512,12 +511,12 @@ int proc_pid_statm(struct task_struct *task, char *buffer) ...@@ -512,12 +511,12 @@ int proc_pid_statm(struct task_struct *task, char *buffer)
{ {
int size = 0, resident = 0, shared = 0, text = 0, lib = 0, data = 0; int size = 0, resident = 0, shared = 0, text = 0, lib = 0, data = 0;
struct mm_struct *mm = get_task_mm(task); struct mm_struct *mm = get_task_mm(task);
if (mm) { if (mm) {
size = task_statm(mm, &shared, &text, &data, &resident); size = task_statm(mm, &shared, &text, &data, &resident);
mmput(mm); mmput(mm);
} }
return sprintf(buffer,"%d %d %d %d %d %d %d\n", return sprintf(buffer, "%d %d %d %d %d %d %d\n",
size, resident, shared, text, lib, data, 0); size, resident, shared, text, lib, data, 0);
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment