Hi, Ingo, > +static clock_t task_utime(struct task_struct *p) > +{ > + /* > + * Use CFS's precise accounting, if available: > + */ > + if (!has_rt_policy(p) && !(sysctl_sched_load_smoothing & 128)) > + return nsec_to_clock_t(p->sum_exec_runtime);
I wonder if this leads to data truncation, p->sum_exec_runtime is unsigned long long and clock_t is long (on all architectures from what my cscope shows me). I have my other patch ready on top of this. I'll post it out soon. > + > + return cputime_to_clock_t(p->utime); > +} > + -- Warm Regards, Balbir Singh Linux Technology Center IBM, ISTL - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/