提交 14454a1b 编写于 作者: K Kenji Kaneshige 提交者: Len Brown

[ACPI] iosapic_register_intr() now returns error instead of panic

error condition is passed along by acpi_register_gsi().
Signed-off-by: NKenji Kaneshige <kaneshige.kenji@jp.fujitsu.com>
Signed-off-by: NAndrew Morton <akpm@osdl.org>
Signed-off-by: NLen Brown <len.brown@intel.com>
上级 58e02762
...@@ -562,7 +562,7 @@ static inline int vector_is_shared (int vector) ...@@ -562,7 +562,7 @@ static inline int vector_is_shared (int vector)
return (iosapic_intr_info[vector].count > 1); return (iosapic_intr_info[vector].count > 1);
} }
static void static int
register_intr (unsigned int gsi, int vector, unsigned char delivery, register_intr (unsigned int gsi, int vector, unsigned char delivery,
unsigned long polarity, unsigned long trigger) unsigned long polarity, unsigned long trigger)
{ {
...@@ -577,7 +577,7 @@ register_intr (unsigned int gsi, int vector, unsigned char delivery, ...@@ -577,7 +577,7 @@ register_intr (unsigned int gsi, int vector, unsigned char delivery,
index = find_iosapic(gsi); index = find_iosapic(gsi);
if (index < 0) { if (index < 0) {
printk(KERN_WARNING "%s: No IOSAPIC for GSI %u\n", __FUNCTION__, gsi); printk(KERN_WARNING "%s: No IOSAPIC for GSI %u\n", __FUNCTION__, gsi);
return; return -ENODEV;
} }
iosapic_address = iosapic_lists[index].addr; iosapic_address = iosapic_lists[index].addr;
...@@ -588,7 +588,7 @@ register_intr (unsigned int gsi, int vector, unsigned char delivery, ...@@ -588,7 +588,7 @@ register_intr (unsigned int gsi, int vector, unsigned char delivery,
rte = iosapic_alloc_rte(); rte = iosapic_alloc_rte();
if (!rte) { if (!rte) {
printk(KERN_WARNING "%s: cannot allocate memory\n", __FUNCTION__); printk(KERN_WARNING "%s: cannot allocate memory\n", __FUNCTION__);
return; return -ENOMEM;
} }
rte_index = gsi - gsi_base; rte_index = gsi - gsi_base;
...@@ -603,7 +603,7 @@ register_intr (unsigned int gsi, int vector, unsigned char delivery, ...@@ -603,7 +603,7 @@ register_intr (unsigned int gsi, int vector, unsigned char delivery,
struct iosapic_intr_info *info = &iosapic_intr_info[vector]; struct iosapic_intr_info *info = &iosapic_intr_info[vector];
if (info->trigger != trigger || info->polarity != polarity) { if (info->trigger != trigger || info->polarity != polarity) {
printk (KERN_WARNING "%s: cannot override the interrupt\n", __FUNCTION__); printk (KERN_WARNING "%s: cannot override the interrupt\n", __FUNCTION__);
return; return -EINVAL;
} }
} }
...@@ -623,6 +623,7 @@ register_intr (unsigned int gsi, int vector, unsigned char delivery, ...@@ -623,6 +623,7 @@ register_intr (unsigned int gsi, int vector, unsigned char delivery,
__FUNCTION__, vector, idesc->handler->typename, irq_type->typename); __FUNCTION__, vector, idesc->handler->typename, irq_type->typename);
idesc->handler = irq_type; idesc->handler = irq_type;
} }
return 0;
} }
static unsigned int static unsigned int
...@@ -710,7 +711,7 @@ int ...@@ -710,7 +711,7 @@ int
iosapic_register_intr (unsigned int gsi, iosapic_register_intr (unsigned int gsi,
unsigned long polarity, unsigned long trigger) unsigned long polarity, unsigned long trigger)
{ {
int vector, mask = 1; int vector, mask = 1, err;
unsigned int dest; unsigned int dest;
unsigned long flags; unsigned long flags;
struct iosapic_rte_info *rte; struct iosapic_rte_info *rte;
...@@ -735,8 +736,11 @@ iosapic_register_intr (unsigned int gsi, ...@@ -735,8 +736,11 @@ iosapic_register_intr (unsigned int gsi,
/* If vector is running out, we try to find a sharable vector */ /* If vector is running out, we try to find a sharable vector */
vector = assign_irq_vector_nopanic(AUTO_ASSIGN); vector = assign_irq_vector_nopanic(AUTO_ASSIGN);
if (vector < 0) if (vector < 0) {
vector = iosapic_find_sharable_vector(trigger, polarity); vector = iosapic_find_sharable_vector(trigger, polarity);
if (vector < 0)
Return -ENOSPC;
}
spin_lock_irqsave(&irq_descp(vector)->lock, flags); spin_lock_irqsave(&irq_descp(vector)->lock, flags);
spin_lock(&iosapic_lock); spin_lock(&iosapic_lock);
...@@ -750,8 +754,13 @@ iosapic_register_intr (unsigned int gsi, ...@@ -750,8 +754,13 @@ iosapic_register_intr (unsigned int gsi,
} }
dest = get_target_cpu(gsi, vector); dest = get_target_cpu(gsi, vector);
register_intr(gsi, vector, IOSAPIC_LOWEST_PRIORITY, err = register_intr(gsi, vector, IOSAPIC_LOWEST_PRIORITY,
polarity, trigger); polarity, trigger);
if (err < 0) {
spin_unlock(&iosapic_lock);
spin_unlock_irqrestore(&irq_descp(vector)->lock, flags);
return err;
}
/* /*
* If the vector is shared and already unmasked for * If the vector is shared and already unmasked for
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册