Messages in this thread | | | Date | Wed, 27 May 2015 14:41:59 +0530 | From | Viresh Kumar <> | Subject | Re: [patch 2/7] timer: Remove FIFO guarantee |
| |
On 26-05-15, 22:50, Thomas Gleixner wrote: > The FIFO guarantee has been violated by the introduction of timer > slack already. Remove it. > > This is a preparatory patch for converting the timer wheel to hlist > which reduces the memory foot print of the wheel by 50%. It's a > seperate patch so any (unlikely to happen) regression caused by this > can be identified clearly. > > Signed-off-by: Thomas Gleixner <tglx@linutronix.de> > --- > kernel/time/timer.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > Index: tip/kernel/time/timer.c > =================================================================== > --- tip.orig/kernel/time/timer.c > +++ tip/kernel/time/timer.c > @@ -403,10 +403,8 @@ __internal_add_timer(struct tvec_base *b > i = (expires >> (TVR_BITS + 3 * TVN_BITS)) & TVN_MASK; > vec = base->tv5.vec + i; > } > - /* > - * Timers are FIFO: > - */ > - list_add_tail(&timer->entry, vec); > + > + list_add(&timer->entry, vec); > }
Reviewed-by: Viresh Kumar <viresh.kumar@linaro.org>
-- viresh
| |