| Message ID | 1370644273-10495-18-git-send-email-yinghai@kernel.org (mailing list archive) |
|---|---|
| State | New, archived |
| Delegated to: | Bjorn Helgaas |
| Headers | show |
On Fri, 7 Jun 2013, Yinghai Lu wrote: > We pre-reserve irq range for hot-added ioapic, and later only > some are used via realloc. > So during hot-remove, we need to clear bits in allocated_irqs > for both case. > > Check if the irq_desc is there at first, and bail out early if > irq_desc is not allocated yet. > We can use irq_free_descs to clear allocated_irqs bits for > preserved irqs only. This changelog is a nightmare as usual. It has nothing to do with pre reservation and hot-added ioapics. This is generic code and does not care at all about x86 specific crappola. Your change is adding a generic sanity check into free_desc(). So first of all the patch subject is bogus: genirq: Bail out early in free_desc() That's missing _WHY_ it bails out early. And then the changelog itself drivels about completely irrelevant nonsense instead of explaining the change itself. So what I want to see here is something like this: "genirq: Do not free unallocated irq descriptors Hot-added interrupt controllers can reserve a range of interrupt numbers, but only allocate some of them. To simplify the release on hot-remove allow them to iterate over the reserved range and let the free_desc() code return early when the descriptor does not exist." Can you see the difference? I told you more than once, that I'm not accepting your sloppy crap anymore. I'm simply not buying your claim that you think "chinese" when writing "english". You are simply too lazy to give a rats ass about it. That applies to your code and to your changelogs in the same way. I'm really tired of dealing with shit like this. No thanks tglx > Signed-off-by: Yinghai Lu <yinghai@kernel.org> > --- > kernel/irq/irqdesc.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/kernel/irq/irqdesc.c b/kernel/irq/irqdesc.c > index b48f65b..32f099e 100644 > --- a/kernel/irq/irqdesc.c > +++ b/kernel/irq/irqdesc.c > @@ -170,6 +170,9 @@ static void free_desc(unsigned int irq) > { > struct irq_desc *desc = irq_to_desc(irq); > > + if (!desc) > + return; > + > unregister_irq_proc(irq, desc); > > mutex_lock(&sparse_irq_lock); > -- > 1.8.1.4 > > -- 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/kernel/irq/irqdesc.c b/kernel/irq/irqdesc.c index b48f65b..32f099e 100644 --- a/kernel/irq/irqdesc.c +++ b/kernel/irq/irqdesc.c @@ -170,6 +170,9 @@ static void free_desc(unsigned int irq) { struct irq_desc *desc = irq_to_desc(irq); + if (!desc) + return; + unregister_irq_proc(irq, desc); mutex_lock(&sparse_irq_lock);
We pre-reserve irq range for hot-added ioapic, and later only some are used via realloc. So during hot-remove, we need to clear bits in allocated_irqs for both case. Check if the irq_desc is there at first, and bail out early if irq_desc is not allocated yet. We can use irq_free_descs to clear allocated_irqs bits for preserved irqs only. Signed-off-by: Yinghai Lu <yinghai@kernel.org> --- kernel/irq/irqdesc.c | 3 +++ 1 file changed, 3 insertions(+)