lkml.org 
[lkml]   [2010]   [Mar]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [RFC,PATCH 1/2] cputimers/proc: do_task_stat()->task_times() can race with getrusage()
* Oleg Nesterov <oleg@redhat.com> [2010-03-24 21:45:50]:

> do_task_stat()->task_times() can race with getrusage(), they both can
> try to update task->prev_Xtime at the same time.
>
> Remove this bit of d180c5bc "sched: Introduce task_times() to replace
> task_{u,s}time()".

One of the reasons for adding this accuracy was to avoid sampling
based noise and errors that occur with utime and stime.

As long as there is no preemption during the assignment, I think we
should be OK.

I see two options

1. Disable preemption around assignment
2. Remove task_times() from getrusage()


--
Three Cheers,
Balbir


\
 
 \ /
  Last update: 2010-03-26 04:57    [W:0.102 / U:0.344 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site