genirq: Prepare proc for real sparse irq support
authorThomas Gleixner <tglx@linutronix.de>
Thu, 30 Sep 2010 00:46:07 +0000 (02:46 +0200)
committerThomas Gleixner <tglx@linutronix.de>
Tue, 12 Oct 2010 14:39:07 +0000 (16:39 +0200)
/proc/irq never removes any entries, but when irq descriptors can be
freed for real this is necessary. Otherwise we'd reference a freed
descriptor in /proc/irq/N

Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Reviewed-by: Ingo Molnar <mingo@elte.hu>
kernel/irq/internals.h
kernel/irq/irqdesc.c
kernel/irq/proc.c

index e281e45fbb55aad97db2664839ec26a8be80ca83..8eb01e379ccc745de14f12855b755cf575dc21ba 100644 (file)
@@ -32,10 +32,12 @@ void replace_irq_desc(unsigned int irq, struct irq_desc *desc);
 
 #ifdef CONFIG_PROC_FS
 extern void register_irq_proc(unsigned int irq, struct irq_desc *desc);
+extern void unregister_irq_proc(unsigned int irq, struct irq_desc *desc);
 extern void register_handler_proc(unsigned int irq, struct irqaction *action);
 extern void unregister_handler_proc(unsigned int irq, struct irqaction *action);
 #else
 static inline void register_irq_proc(unsigned int irq, struct irq_desc *desc) { }
+static inline void unregister_irq_proc(unsigned int irq, struct irq_desc *desc) { }
 static inline void register_handler_proc(unsigned int irq,
                                         struct irqaction *action) { }
 static inline void unregister_handler_proc(unsigned int irq,
index 4eea48b4f5767b53b5b2420138303f22faddcdf5..6312a2c8397187ba9f4586cf91007a685f177506 100644 (file)
@@ -205,6 +205,8 @@ static void free_desc(unsigned int irq)
        struct irq_desc *desc = irq_to_desc(irq);
        unsigned long flags;
 
+       unregister_irq_proc(irq, desc);
+
        raw_spin_lock_irqsave(&sparse_irq_lock, flags);
        delete_irq_desc(irq);
        raw_spin_unlock_irqrestore(&sparse_irq_lock, flags);
index d9fddf918b41e87b5b989c9e613f2419f7a3cce8..01b1d3a88983f92c189e0c9b27f23ffe0e73f88f 100644 (file)
@@ -297,6 +297,24 @@ void register_irq_proc(unsigned int irq, struct irq_desc *desc)
                         &irq_spurious_proc_fops, (void *)(long)irq);
 }
 
+void unregister_irq_proc(unsigned int irq, struct irq_desc *desc)
+{
+       char name [MAX_NAMELEN];
+
+       if (!root_irq_dir || !desc->dir)
+               return;
+#ifdef CONFIG_SMP
+       remove_proc_entry("smp_affinity", desc->dir);
+       remove_proc_entry("affinity_hint", desc->dir);
+       remove_proc_entry("node", desc->dir);
+#endif
+       remove_proc_entry("spurious", desc->dir);
+
+       memset(name, 0, MAX_NAMELEN);
+       sprintf(name, "%u", irq);
+       remove_proc_entry(name, root_irq_dir);
+}
+
 #undef MAX_NAMELEN
 
 void unregister_handler_proc(unsigned int irq, struct irqaction *action)