public inbox for linux-ia64@vger.kernel.org
 help / color / mirror / Atom feed
* [2.6 patch] IA64 irq.c: remove CONFIG_X86 code
@ 2004-11-19  0:19 Adrian Bunk
  2004-11-19  6:42 ` Luck, Tony
  0 siblings, 1 reply; 2+ messages in thread
From: Adrian Bunk @ 2004-11-19  0:19 UTC (permalink / raw)
  To: davidm; +Cc: linux-ia64, linux-kernel

I don't see hos this code could ever be used.

Am I correct or did I miss something?


diffstat output:
 arch/ia64/kernel/irq.c |   33 ---------------------------------
 1 files changed, 33 deletions(-)


Signed-off-by: Adrian Bunk <bunk@fs.tum.de>

--- linux-2.6.10-rc2-mm2-full/arch/ia64/kernel/irq.c.old	2004-11-19 01:14:22.000000000 +0100
+++ linux-2.6.10-rc2-mm2-full/arch/ia64/kernel/irq.c	2004-11-19 01:16:31.000000000 +0100
@@ -132,23 +132,7 @@
  * each architecture has to answer this themselves, it doesn't deserve
  * a generic callback i think.
  */
-#ifdef CONFIG_X86
-	printk(KERN_ERR "unexpected IRQ trap at vector %02x\n", irq);
-#ifdef CONFIG_X86_LOCAL_APIC
-	/*
-	 * Currently unexpected vectors happen only on SMP and APIC.
-	 * We _must_ ack these because every local APIC has only N
-	 * irq slots per priority level, and a 'hanging, unacked' IRQ
-	 * holds up an irq slot - in excessive cases (when multiple
-	 * unexpected vectors occur) that might lock up the APIC
-	 * completely.
-	 */
-	ack_APIC_irq();
-#endif
-#endif
-#ifdef CONFIG_IA64
 	printk(KERN_ERR "Unexpected irq vector 0x%x on CPU %u!\n", irq, smp_processor_id());
-#endif
 }
 
 /* startup is the same as "enable", shutdown is same as "disable" */
@@ -166,11 +150,6 @@
 };
 
 atomic_t irq_err_count;
-#ifdef CONFIG_X86_IO_APIC
-#ifdef APIC_MISMATCH_DEBUG
-atomic_t irq_mis_count;
-#endif
-#endif
 
 /*
  * Generic, controller-independent functions:
@@ -220,19 +199,7 @@
 			if (cpu_online(j))
 				seq_printf(p, "%10u ", nmi_count(j));
 		seq_putc(p, '\n');
-#ifdef CONFIG_X86_LOCAL_APIC
-		seq_puts(p, "LOC: ");
-		for (j = 0; j < NR_CPUS; j++)
-			if (cpu_online(j))
-				seq_printf(p, "%10u ", irq_stat[j].apic_timer_irqs);
-		seq_putc(p, '\n');
-#endif
 		seq_printf(p, "ERR: %10u\n", atomic_read(&irq_err_count));
-#ifdef CONFIG_X86_IO_APIC
-#ifdef APIC_MISMATCH_DEBUG
-		seq_printf(p, "MIS: %10u\n", atomic_read(&irq_mis_count));
-#endif
-#endif
 	}
 	return 0;
 }

^ permalink raw reply	[flat|nested] 2+ messages in thread

* RE: [2.6 patch] IA64 irq.c: remove CONFIG_X86 code
  2004-11-19  0:19 [2.6 patch] IA64 irq.c: remove CONFIG_X86 code Adrian Bunk
@ 2004-11-19  6:42 ` Luck, Tony
  0 siblings, 0 replies; 2+ messages in thread
From: Luck, Tony @ 2004-11-19  6:42 UTC (permalink / raw)
  To: Adrian Bunk, davidm; +Cc: linux-ia64, linux-kernel

>I don't see how this code could ever be used.
>
>Am I correct or did I miss something?
>
>
>diffstat output:
> arch/ia64/kernel/irq.c |   33 ---------------------------------
> 1 files changed, 33 deletions(-)

Thanks to a helpful hint from Christoph earlier this week I put
together a patch that throws away over 900 lines from the ia64
irq.c

http://marc.theaimsgroup.com/?l=linux-ia64&m\x110072859114157&w=2

It's queued up in an internal tree waiting for 2.6.11 to open.

Some parts of your patch may still be needed.

-Tony

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2004-11-19  6:42 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-11-19  0:19 [2.6 patch] IA64 irq.c: remove CONFIG_X86 code Adrian Bunk
2004-11-19  6:42 ` Luck, Tony

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox