Messages in this thread | | | Date | Tue, 14 Aug 2012 21:16:16 +0200 (CEST) | From | Thomas Gleixner <> | Subject | Re: [PATCHSET] timer: clean up initializers and implement irqsafe timers |
| |
On Mon, 13 Aug 2012, Tejun Heo wrote: > On Wed, Aug 08, 2012 at 11:10:24AM -0700, Tejun Heo wrote: > > Timer internals are protected by irqsafe lock but the lock is > > naturally dropped and irq enabled while a timer is executed. This > > makes dequeueing timer for execution and the actual execution > > non-atomic against IRQs. No matter what the timer function does, IRQs > > can occur between timer dispatch and execution. This means that an > > IRQ handler could interrupt any timer in progress and it's impossible > > for an IRQ handler to cancel and drain a timer. > > If nobody objects, I'll route this through wq/for-3.7 together with > "workqueue: use irqsafe timer in delayed_work" patchset. If you > object, please scream.
Second thoughts.
Why the hell are you trying to rush stuff which affects a well maintained part of the kernel through your own tree w/o having the courtesy of contacting the maintainer politely instead of sending an ultimatum?
You posted that series less than a week ago and there is no reason why you need to push hard on inclusion into next after a few days.
If you really cared about my opinion you could have figured out that I'm on vacation.
Yours grumpy
tglx
| |