Re: [PATCH] sched: Sanitize irq accounting madness

2014-07-09 Thread Frederic Weisbecker
On Fri, May 02, 2014 at 11:26:24PM +0200, Thomas Gleixner wrote: > Russell reported, that irqtime_account_idle_ticks() takes ages due to: > >for (i = 0; i < ticks; i++) >irqtime_account_process_tick(current, 0, rq); > > It's sad, that this code was written way _AFTER_ the

Re: [PATCH] sched: Sanitize irq accounting madness

2014-07-09 Thread Frederic Weisbecker
On Fri, May 02, 2014 at 11:26:24PM +0200, Thomas Gleixner wrote: Russell reported, that irqtime_account_idle_ticks() takes ages due to: for (i = 0; i ticks; i++) irqtime_account_process_tick(current, 0, rq); It's sad, that this code was written way _AFTER_ the NOHZ

Re: [PATCH] sched: Sanitize irq accounting madness

2014-05-02 Thread Paul E. McKenney
On Fri, May 02, 2014 at 11:26:24PM +0200, Thomas Gleixner wrote: > Russell reported, that irqtime_account_idle_ticks() takes ages due to: > >for (i = 0; i < ticks; i++) >irqtime_account_process_tick(current, 0, rq); > > It's sad, that this code was written way _AFTER_ the

[PATCH] sched: Sanitize irq accounting madness

2014-05-02 Thread Thomas Gleixner
Russell reported, that irqtime_account_idle_ticks() takes ages due to: for (i = 0; i < ticks; i++) irqtime_account_process_tick(current, 0, rq); It's sad, that this code was written way _AFTER_ the NOHZ idle functionality was available. I charge myself guitly for not paying

[PATCH] sched: Sanitize irq accounting madness

2014-05-02 Thread Thomas Gleixner
Russell reported, that irqtime_account_idle_ticks() takes ages due to: for (i = 0; i ticks; i++) irqtime_account_process_tick(current, 0, rq); It's sad, that this code was written way _AFTER_ the NOHZ idle functionality was available. I charge myself guitly for not paying

Re: [PATCH] sched: Sanitize irq accounting madness

2014-05-02 Thread Paul E. McKenney
On Fri, May 02, 2014 at 11:26:24PM +0200, Thomas Gleixner wrote: Russell reported, that irqtime_account_idle_ticks() takes ages due to: for (i = 0; i ticks; i++) irqtime_account_process_tick(current, 0, rq); It's sad, that this code was written way _AFTER_ the NOHZ