linux-kernel - Re: [PATCH][RESEND] Export per-tid and per-tgid cputime in nanoseconds.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Sat, 27 Feb 2010 12:43:45 +0100
From:	Peter Zijlstra <peterz@...radead.org>
To:	Divyesh Shah <dpshah@...gle.com>
Cc:	linux-kernel@...r.kernel.org, mingo@...e.hu, kenchen@...gle.com
Subject: Re: [PATCH][RESEND] Export per-tid and per-tgid cputime in
 nanoseconds.

On Fri, 2010-02-26 at 18:03 -0800, Divyesh Shah wrote:
> This can be used by applications to get finer granularity cputime usage on
> platforms that use timestamp counters or HPET.

I guess the patch looks good, I'm just not sure what HPET got to do with
anything.. the scheduler certainly doesn't use HPET for timekeeping, its
terribly slow to read.

Also, it would be good to get some more justification than 'some
applications can use this', which is basically a truism for any patch
that adds a user interface.

> Signed-off-by: Divyesh Shah<dpshah@...gle.com>
> ---
> 
>  fs/proc/array.c    |   40 ++++++++++++++++++++++++++++++++++++++++
>  fs/proc/base.c     |    2 ++
>  fs/proc/internal.h |    2 ++
>  3 files changed, 44 insertions(+), 0 deletions(-)
> 
> diff --git a/fs/proc/array.c b/fs/proc/array.c
> index 13b5d07..54604b8 100644
> --- a/fs/proc/array.c
> +++ b/fs/proc/array.c
> @@ -547,3 +547,43 @@ int proc_pid_statm(struct seq_file *m, struct pid_namespace *ns,
>  
>  	return 0;
>  }
> +
> +static int do_task_cputime(struct task_struct *task, char * buffer, int whole)
> +{
> +	int res;
> +	unsigned long flags;
> +	unsigned long long sum_exec_runtime = 0;
> +	struct task_struct *t;
> +
> +	if (lock_task_sighand(task, &flags)) {
> +		if (whole) {
> +			t = task;
> +			/* 
> +			 * Add up live thread sum_exec_runtime at the group
> +			 * level.
> +			 */
> +			do {
> +				sum_exec_runtime += t->se.sum_exec_runtime;
> +				t = next_thread(t);
> +			} while (t != task);
> +			sum_exec_runtime += task->signal->sum_sched_runtime;
> +		}
> +		unlock_task_sighand(task, &flags);
> +	}
> +
> +	if (!whole)
> +		sum_exec_runtime = task->se.sum_exec_runtime;
> +
> +	res = sprintf(buffer,"%llu\n", sum_exec_runtime);
> +	return res;
> +}
> +
> +int proc_tid_cputime(struct task_struct *task, char * buffer)
> +{
> +	return do_task_cputime(task, buffer, 0);
> +}
> +
> +int proc_tgid_cputime(struct task_struct *task, char * buffer)
> +{
> +	return do_task_cputime(task, buffer, 1);
> +}
> diff --git a/fs/proc/base.c b/fs/proc/base.c
> index 58324c2..8fbc785 100644
> --- a/fs/proc/base.c
> +++ b/fs/proc/base.c
> @@ -2595,6 +2595,7 @@ static const struct pid_entry tgid_base_stuff[] = {
>  	REG("numa_maps",  S_IRUGO, proc_numa_maps_operations),
>  #endif
>  	REG("mem",        S_IRUSR|S_IWUSR, proc_mem_operations),
> +	INF("cputime_ns", S_IRUGO, proc_tgid_cputime),
>  	LNK("cwd",        proc_cwd_link),
>  	LNK("root",       proc_root_link),
>  	LNK("exe",        proc_exe_link),
> @@ -2930,6 +2931,7 @@ static const struct pid_entry tid_base_stuff[] = {
>  	REG("numa_maps", S_IRUGO, proc_numa_maps_operations),
>  #endif
>  	REG("mem",       S_IRUSR|S_IWUSR, proc_mem_operations),
> +	INF("cputime_ns", S_IRUGO, proc_tid_cputime),
>  	LNK("cwd",       proc_cwd_link),
>  	LNK("root",      proc_root_link),
>  	LNK("exe",       proc_exe_link),
> diff --git a/fs/proc/internal.h b/fs/proc/internal.h
> index 1f24a3e..f9e9799 100644
> --- a/fs/proc/internal.h
> +++ b/fs/proc/internal.h
> @@ -51,6 +51,8 @@ extern int proc_pid_status(struct seq_file *m, struct pid_namespace *ns,
>  				struct pid *pid, struct task_struct *task);
>  extern int proc_pid_statm(struct seq_file *m, struct pid_namespace *ns,
>  				struct pid *pid, struct task_struct *task);
> +extern int proc_tid_cputime(struct task_struct *, char *);
> +extern int proc_tgid_cputime(struct task_struct *, char *);
>  extern loff_t mem_lseek(struct file *file, loff_t offset, int orig);
>  
>  extern const struct file_operations proc_maps_operations;
> 


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/