lkml.org 
[lkml]   [2001]   [Sep]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: struct_task->start_time, jiffies or hz_to_std(jiffies)
On Thu, Sep 13, 2001 at 03:48:55PM +1000, Matt wrote:

> 2.4.9-ac10 linux/kernel/fork.c:658
> p->start_time = jiffies;
>
> is this
>
> 2.4.9-ac10 linux/fs/proc/array.c
> task->start_time,
>
> correct? or should it be
> hz_to_std(task->start_time),
>
> ??
>
> i know this will affect libproc, however libproc appears to be broken
> anyway; i changed HZ and CLOCKS_PER_SEC to 1024 in include/asm/param.h
> on x86 and top / ps etc are giving me very whacked out numbers.. the
> machine itself is stable however and appears to be working just fine.

Antirely correct; I'll make a patch and send it to Alan in case he doesn't
have this one yet.

A quick grep also shows that kernel accounting also seems to have it's
issues with time granularity. Bad, accounting date should be portable
between architectures.

Alan, real patch below in case this isn't yet in -ac please apply.

Ralf

Index: fs/proc/array.c
===================================================================
RCS file: /home/pub/cvs/linux/fs/proc/array.c,v
retrieving revision 1.48
diff -u -r1.48 array.c
--- fs/proc/array.c 2001/08/24 03:38:51 1.48
+++ fs/proc/array.c 2001/09/16 22:48:28
@@ -365,7 +365,7 @@
nice,
0UL /* removed */,
task->it_real_value,
- task->start_time,
+ hz_to_std(task->start_time),
vsize,
mm ? mm->rss : 0, /* you might want to shift this left 3 */
task->rlim[RLIMIT_RSS].rlim_cur,
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
\
 
 \ /
  Last update: 2005-03-22 13:03    [W:0.037 / U:2.996 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site