irq: remove >= nr_irqs checking with config_have_sparse_irq
remove irq limit checks - nr_irqs is dynamic and we expand anytime.
v2: fix checking about result irq_cfg_without_new, so could use msi again
v3: use irq_desc_without_new to check irq is valid
Signed-off-by: Yinghai Lu <yhlu.kernel@gmail.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
diff --git a/kernel/irq/chip.c b/kernel/irq/chip.c
index 2aa3d4b2..a4bb0da 100644
--- a/kernel/irq/chip.c
+++ b/kernel/irq/chip.c
@@ -27,13 +27,13 @@
struct irq_desc *desc;
unsigned long flags;
- if (irq >= nr_irqs) {
+ desc = irq_to_desc(irq);
+ if (!desc) {
WARN(1, KERN_ERR "Trying to initialize invalid IRQ%d\n", irq);
return;
}
/* Ensure we don't have left over values from a previous use of this irq */
- desc = irq_to_desc(irq);
spin_lock_irqsave(&desc->lock, flags);
desc->status = IRQ_DISABLED;
desc->chip = &no_irq_chip;
@@ -60,12 +60,12 @@
struct irq_desc *desc;
unsigned long flags;
- if (irq >= nr_irqs) {
+ desc = __irq_to_desc(irq);
+ if (!desc) {
WARN(1, KERN_ERR "Trying to cleanup invalid IRQ%d\n", irq);
return;
}
- desc = irq_to_desc(irq);
spin_lock_irqsave(&desc->lock, flags);
if (desc->action) {
spin_unlock_irqrestore(&desc->lock, flags);
@@ -92,7 +92,8 @@
struct irq_desc *desc;
unsigned long flags;
- if (irq >= nr_irqs) {
+ desc = __irq_to_desc(irq);
+ if (!desc) {
WARN(1, KERN_ERR "Trying to install chip for IRQ%d\n", irq);
return -EINVAL;
}
@@ -121,12 +122,12 @@
unsigned long flags;
int ret = -ENXIO;
- if (irq >= nr_irqs) {
+ desc = __irq_to_desc(irq);
+ if (!desc) {
printk(KERN_ERR "Trying to set irq type for IRQ%d\n", irq);
return -ENODEV;
}
- desc = irq_to_desc(irq);
if (type == IRQ_TYPE_NONE)
return 0;
@@ -149,13 +150,13 @@
struct irq_desc *desc;
unsigned long flags;
- if (irq >= nr_irqs) {
+ desc = __irq_to_desc(irq);
+ if (!desc) {
printk(KERN_ERR
"Trying to install controller data for IRQ%d\n", irq);
return -EINVAL;
}
- desc = irq_to_desc(irq);
spin_lock_irqsave(&desc->lock, flags);
desc->handler_data = data;
spin_unlock_irqrestore(&desc->lock, flags);
@@ -175,12 +176,13 @@
struct irq_desc *desc;
unsigned long flags;
- if (irq >= nr_irqs) {
+ desc = __irq_to_desc(irq);
+ if (!desc) {
printk(KERN_ERR
"Trying to install msi data for IRQ%d\n", irq);
return -EINVAL;
}
- desc = irq_to_desc(irq);
+
spin_lock_irqsave(&desc->lock, flags);
desc->msi_desc = entry;
if (entry)
@@ -201,8 +203,14 @@
struct irq_desc *desc;
unsigned long flags;
- desc = irq_to_desc(irq);
- if (irq >= nr_irqs || !desc->chip) {
+ desc = __irq_to_desc(irq);
+ if (!desc) {
+ printk(KERN_ERR
+ "Trying to install chip data for IRQ%d\n", irq);
+ return -EINVAL;
+ }
+
+ if (!desc->chip) {
printk(KERN_ERR "BUG: bad set_irq_chip_data(IRQ#%d)\n", irq);
return -EINVAL;
}
@@ -546,14 +554,13 @@
struct irq_desc *desc;
unsigned long flags;
- if (irq >= nr_irqs) {
+ desc = __irq_to_desc(irq);
+ if (!desc) {
printk(KERN_ERR
"Trying to install type control for IRQ%d\n", irq);
return;
}
- desc = irq_to_desc(irq);
-
if (!handle)
handle = handle_bad_irq;
else if (desc->chip == &no_irq_chip) {
@@ -611,14 +618,13 @@
struct irq_desc *desc;
unsigned long flags;
- if (irq >= nr_irqs) {
+ desc = __irq_to_desc(irq);
+ if (!desc) {
printk(KERN_ERR "Trying to mark IRQ%d non-probeable\n", irq);
return;
}
- desc = irq_to_desc(irq);
-
spin_lock_irqsave(&desc->lock, flags);
desc->status |= IRQ_NOPROBE;
spin_unlock_irqrestore(&desc->lock, flags);
@@ -629,14 +635,13 @@
struct irq_desc *desc;
unsigned long flags;
- if (irq >= nr_irqs) {
+ desc = __irq_to_desc(irq);
+ if (!desc) {
printk(KERN_ERR "Trying to mark IRQ%d probeable\n", irq);
return;
}
- desc = irq_to_desc(irq);
-
spin_lock_irqsave(&desc->lock, flags);
desc->status &= ~IRQ_NOPROBE;
spin_unlock_irqrestore(&desc->lock, flags);
diff --git a/kernel/irq/manage.c b/kernel/irq/manage.c
index 5070f55..c0b4d4d 100644
--- a/kernel/irq/manage.c
+++ b/kernel/irq/manage.c
@@ -31,10 +31,10 @@
*/
void synchronize_irq(unsigned int irq)
{
- struct irq_desc *desc = irq_to_desc(irq);
+ struct irq_desc *desc = __irq_to_desc(irq);
unsigned int status;
- if (irq >= nr_irqs)
+ if (!desc)
return;
do {
@@ -142,10 +142,11 @@
*/
void disable_irq_nosync(unsigned int irq)
{
- struct irq_desc *desc = irq_to_desc(irq);
+ struct irq_desc *desc;
unsigned long flags;
- if (irq >= nr_irqs)
+ desc = __irq_to_desc(irq);
+ if (!desc)
return;
spin_lock_irqsave(&desc->lock, flags);
@@ -171,9 +172,10 @@
*/
void disable_irq(unsigned int irq)
{
- struct irq_desc *desc = irq_to_desc(irq);
+ struct irq_desc *desc;
- if (irq >= nr_irqs)
+ desc = __irq_to_desc(irq);
+ if (!desc)
return;
disable_irq_nosync(irq);
@@ -213,10 +215,11 @@
*/
void enable_irq(unsigned int irq)
{
- struct irq_desc *desc = irq_to_desc(irq);
+ struct irq_desc *desc;
unsigned long flags;
- if (irq >= nr_irqs)
+ desc = __irq_to_desc(irq);
+ if (!desc)
return;
spin_lock_irqsave(&desc->lock, flags);
@@ -290,10 +293,14 @@
*/
int can_request_irq(unsigned int irq, unsigned long irqflags)
{
- struct irq_desc *desc = irq_to_desc(irq);
+ struct irq_desc *desc;
struct irqaction *action;
- if (irq >= nr_irqs || desc->status & IRQ_NOREQUEST)
+ desc = __irq_to_desc(irq);
+ if (!desc)
+ return 0;
+
+ if (desc->status & IRQ_NOREQUEST)
return 0;
action = desc->action;
@@ -352,14 +359,15 @@
*/
int setup_irq(unsigned int irq, struct irqaction *new)
{
- struct irq_desc *desc = irq_to_desc(irq);
+ struct irq_desc *desc;
struct irqaction *old, **p;
const char *old_name = NULL;
unsigned long flags;
int shared = 0;
int ret;
- if (irq >= nr_irqs)
+ desc = __irq_to_desc(irq);
+ if (!desc)
return -EINVAL;
if (desc->chip == &no_irq_chip)
@@ -518,10 +526,11 @@
unsigned long flags;
WARN_ON(in_interrupt());
- if (irq >= nr_irqs)
+
+ desc = __irq_to_desc(irq);
+ if (!desc)
return;
- desc = irq_to_desc(irq);
spin_lock_irqsave(&desc->lock, flags);
p = &desc->action;
for (;;) {
@@ -634,9 +643,11 @@
*/
if ((irqflags & IRQF_SHARED) && !dev_id)
return -EINVAL;
- if (irq >= nr_irqs)
+
+ desc = __irq_to_desc(irq);
+ if (!desc)
return -EINVAL;
- desc = irq_to_desc(irq);
+
if (desc->status & IRQ_NOREQUEST)
return -EINVAL;
if (!handler)