Re: [PATCH] Remove spurious cputimer restart and eliminate its drift

2013-04-19 Thread Frederic Weisbecker
2013/4/19 Frederic Weisbecker : >> if (!task_cputime_zero(&tsk->cputime_expires)) { >> struct task_cputime task_sample = { >> - .utime = utime, >> - .stime = stime, >> .sum_exec_runtime = tsk->se.sum_exec_ru

Re: [PATCH] Remove spurious cputimer restart and eliminate its drift

2013-04-19 Thread Frederic Weisbecker
2013/4/17 Olivier Langlois : > Move the call to stop_process_timers() in order to: > > 1. It catches the exceptionnal case where it would be >started without arming any timers in posix_cpu_timer_set() Oh I see now. fastpath_timer_check() sees the sig->cputimer.running but return 0 because the

[PATCH] Remove spurious cputimer restart and eliminate its drift

2013-04-17 Thread Olivier Langlois
Move the call to stop_process_timers() in order to: 1. It catches the exceptionnal case where it would be started without arming any timers in posix_cpu_timer_set() 2. You could end up with no process timers for a short period of time when they find themselves in the firing list. Let 1