Re: [patch 0/7] Add IRQF_IRQPOLL_IRQ flag to allow irqpoll
On Tue, 27 Mar 2007 07:01:44 +0200 (CEST) Guennadi Liakhovetski <[EMAIL PROTECTED]> wrote: > On Sun, 25 Mar 2007, Bernhard Walle wrote: > > > * Guennadi Liakhovetski <[EMAIL PROTECTED]> [2007-03-24 00:21]: > > > On Fri, 23 Mar 2007, Bernhard Walle wrote: > > > > > > > * Guennadi Liakhovetski <[EMAIL PROTECTED]> [2007-03-23 23:15]: > > > > > On Fri, 23 Mar 2007, Bernhard Walle wrote: > > > > > > > > > > > irqpoll is broken on some architectures that don't use the IRQ 0 > > > > > > for the timer > > > > > > interrupt like IA64. This patch adds a IRQF_IRQPOLL_IRQ flag. > > > > > > > > > > emn, please, cannot we come up with a better name - one that doesn't > > > > > have > > > > > "IRQ" 3 (!) times in it?... > > > > > > > > Well, it was suggested by Andrew. What's your suggestion? > > > > > > Purely esthetically (ehm, am I saying that?...) your original suggestion > > > to use IRQF_IRQPOLL looks better to me. Or even IRQF_POLL, but, maybe, > > > it's a bit too much of a simplification. > > > > I personally don't care very much about the naming. So I think it's > > Andrews final decision ... > > Sure. And for him to have a better chance to do this, I'll fix his email > address too:-) > I don't think I went as daringly far as to suggest the full name, but I must admit that IRQF_IRQPOLL_IRQ is actually logical. - 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/
Re: [patch 0/7] Add IRQF_IRQPOLL_IRQ flag to allow irqpoll
On Sun, 25 Mar 2007, Bernhard Walle wrote: > * Guennadi Liakhovetski <[EMAIL PROTECTED]> [2007-03-24 00:21]: > > On Fri, 23 Mar 2007, Bernhard Walle wrote: > > > > > * Guennadi Liakhovetski <[EMAIL PROTECTED]> [2007-03-23 23:15]: > > > > On Fri, 23 Mar 2007, Bernhard Walle wrote: > > > > > > > > > irqpoll is broken on some architectures that don't use the IRQ 0 for > > > > > the timer > > > > > interrupt like IA64. This patch adds a IRQF_IRQPOLL_IRQ flag. > > > > > > > > emn, please, cannot we come up with a better name - one that doesn't > > > > have > > > > "IRQ" 3 (!) times in it?... > > > > > > Well, it was suggested by Andrew. What's your suggestion? > > > > Purely esthetically (ehm, am I saying that?...) your original suggestion > > to use IRQF_IRQPOLL looks better to me. Or even IRQF_POLL, but, maybe, > > it's a bit too much of a simplification. > > I personally don't care very much about the naming. So I think it's > Andrews final decision ... Sure. And for him to have a better chance to do this, I'll fix his email address too:-) Thanks Guennadi --- Guennadi Liakhovetski - 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/
Re: [patch 0/7] Add IRQF_IRQPOLL_IRQ flag to allow irqpoll
* Guennadi Liakhovetski <[EMAIL PROTECTED]> [2007-03-24 00:21]: > On Fri, 23 Mar 2007, Bernhard Walle wrote: > > > * Guennadi Liakhovetski <[EMAIL PROTECTED]> [2007-03-23 23:15]: > > > On Fri, 23 Mar 2007, Bernhard Walle wrote: > > > > > > > irqpoll is broken on some architectures that don't use the IRQ 0 for > > > > the timer > > > > interrupt like IA64. This patch adds a IRQF_IRQPOLL_IRQ flag. > > > > > > emn, please, cannot we come up with a better name - one that doesn't have > > > "IRQ" 3 (!) times in it?... > > > > Well, it was suggested by Andrew. What's your suggestion? > > Purely esthetically (ehm, am I saying that?...) your original suggestion > to use IRQF_IRQPOLL looks better to me. Or even IRQF_POLL, but, maybe, > it's a bit too much of a simplification. I personally don't care very much about the naming. So I think it's Andrews final decision ... Thanks, Bernhard - 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/
Re: [patch 0/7] Add IRQF_IRQPOLL_IRQ flag to allow irqpoll
On Fri, 23 Mar 2007, Bernhard Walle wrote: > * Guennadi Liakhovetski <[EMAIL PROTECTED]> [2007-03-23 23:15]: > > On Fri, 23 Mar 2007, Bernhard Walle wrote: > > > > > irqpoll is broken on some architectures that don't use the IRQ 0 for the > > > timer > > > interrupt like IA64. This patch adds a IRQF_IRQPOLL_IRQ flag. > > > > emn, please, cannot we come up with a better name - one that doesn't have > > "IRQ" 3 (!) times in it?... > > Well, it was suggested by Andrew. What's your suggestion? Purely esthetically (ehm, am I saying that?...) your original suggestion to use IRQF_IRQPOLL looks better to me. Or even IRQF_POLL, but, maybe, it's a bit too much of a simplification. Thanks Guennadi --- Guennadi Liakhovetski - 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/
Re: [patch 0/7] Add IRQF_IRQPOLL_IRQ flag to allow irqpoll
* Guennadi Liakhovetski <[EMAIL PROTECTED]> [2007-03-23 23:15]: > On Fri, 23 Mar 2007, Bernhard Walle wrote: > > > irqpoll is broken on some architectures that don't use the IRQ 0 for the > > timer > > interrupt like IA64. This patch adds a IRQF_IRQPOLL_IRQ flag. > > emn, please, cannot we come up with a better name - one that doesn't have > "IRQ" 3 (!) times in it?... Well, it was suggested by Andrew. What's your suggestion? Thanks, Bernhard - 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/
Re: [patch 0/7] Add IRQF_IRQPOLL_IRQ flag to allow irqpoll
On Fri, 23 Mar 2007, Bernhard Walle wrote: > irqpoll is broken on some architectures that don't use the IRQ 0 for the timer > interrupt like IA64. This patch adds a IRQF_IRQPOLL_IRQ flag. emn, please, cannot we come up with a better name - one that doesn't have "IRQ" 3 (!) times in it?... Thanks Guennadi --- Guennadi Liakhovetski - 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/
[patch 0/7] Add IRQF_IRQPOLL_IRQ flag to allow irqpoll
irqpoll is broken on some architectures that don't use the IRQ 0 for the timer interrupt like IA64. This patch adds a IRQF_IRQPOLL_IRQ flag. Each architecture is handled in a separate pach. As I left the irq == 0 as condition, this should not break existing architectures that use timer_irq == 0 and that I did't address with that patch (because I don't know). Signed-off-by: Bernhard Walle <[EMAIL PROTECTED]> - 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/
