| Message ID | 1370644273-10495-7-git-send-email-yinghai@kernel.org |
|---|---|
| State | Not Applicable |
| Headers | show |
On Fri, Jun 07, 2013 at 03:30:52PM -0700, Yinghai Lu wrote: > create_irq() will return -1 when fail to allocate. The ia64 code here will return -ENOSPC. > create_irq_nr() will return 0 when fail to allocate. > > Will use it to fix one return value checking for dmar_msi irq. What about to unify the interface? Using -1 is kinda bad. Sebastian -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Sun, Jun 9, 2013 at 8:22 AM, Sebastian Andrzej Siewior <sebastian@breakpoint.cc> wrote: > On Fri, Jun 07, 2013 at 03:30:52PM -0700, Yinghai Lu wrote: >> create_irq() will return -1 when fail to allocate. > The ia64 code here will return -ENOSPC. > >> create_irq_nr() will return 0 when fail to allocate. >> >> Will use it to fix one return value checking for dmar_msi irq. > > What about to unify the interface? Using -1 is kinda bad. after some following patch, create_irq() in x86 get killed. Still need ia64 guys to kill create_irq() in arch/ia64. Thanks Yinghai -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
> Still need ia64 guys to kill create_irq() in arch/ia64.
Was there already a patch to do that? I'm afraid my eyes tend to glaze over when I
see [part 64/87: ia64 ...] and assume that its general cleanup that will flow through
with all the other parts of the patch series. Please point me at something that you
want me to apply.
If not - then what is the recommended replacement? There seem to be a half dozen
places in arch/ia64 where create_irq() is called.
-Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/arch/ia64/kernel/irq_ia64.c b/arch/ia64/kernel/irq_ia64.c index 1034884..38e46df 100644 --- a/arch/ia64/kernel/irq_ia64.c +++ b/arch/ia64/kernel/irq_ia64.c @@ -429,6 +429,16 @@ int create_irq(void) return irq; } +unsigned int create_irq_nr(unsigned int from, int node) +{ + int irq = create_irq(); + + if (irq < 0) + irq = 0; + + return irq; +} + void destroy_irq(unsigned int irq) { dynamic_irq_cleanup(irq);
create_irq() will return -1 when fail to allocate. create_irq_nr() will return 0 when fail to allocate. Will use it to fix one return value checking for dmar_msi irq. Signed-off-by: Yinghai Lu <yinghai@kernel.org> Cc: Tony Luck <tony.luck@intel.com> Cc: Fenghua Yu <fenghua.yu@intel.com> Cc: linux-ia64@vger.kernel.org --- arch/ia64/kernel/irq_ia64.c | 10 ++++++++++ 1 file changed, 10 insertions(+)