Re: [PATCHSET] timer: clean up initializers and implement irqsafetimers

From: Thomas Gleixner
Date: Tue Aug 14 2012 - 19:12:02 EST


Tejun,

On Tue, 14 Aug 2012, Tejun Heo wrote:
> On Tue, Aug 14, 2012 at 10:43:30PM +0200, Thomas Gleixner wrote:
> > > It makes the workqueue users messy. It's difficult to get completely
> > > correct and subtle errors are difficult to detect / verify.
> >
> > Ah, the function which does not exist makes the users
> > messy. Interesting observation.
>
> Can we get a little less snarky please? It's tiring.

Can you please try to answer my questions instead of throwing random
blurb into my direction?

Just for the record. The thread evolved from here:

<tj> * mod_delayed_work() can't be used from IRQ handlers.

My answer was:

<tglx> This function does not exist. So what?

Which was completely appropriate as this function does not exist
though you used it as a primary argument for your patches.

Now your answer to my reply was:

<tj> It makes the workqueue users messy. It's difficult to get
completely correct and subtle errors are difficult to
detect / verify.

Can you please point out any relevance to my question which would have
me prevented from writing the following?

<tglx> Ah, the function which does not exist makes the users
messy. Interesting observation.

So instead of saying, that you wrote an utter nonsense reply you
accuse me of being obnoxious:

<tj> Can we get a little less snarky please? It's tiring.

Can you please sit down for a little while and think about your own
snarkiness and your own tiring behaviour against other kernel
maintainers?

Thanks,

tglx
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/