Re: [PATCH] 2.6.8.1 modprobe tg3 oopses

[Thomas Duffy]

On Sat, 2004-08-21 at 17:36 -0700, Randy.Dunlap wrote:
> I wouldn't expect this to be compiler-dependent.  There's an obvious
> problem with add_pin_to_irq().  It shouldn't be __init.  Patch below.
> (I thought that I had already mailed this one time, but I don't
> see it anywhere.)
> 
> arch/i386/kernel/io_apic.c |    2 +-

There also needs to be a similar patch for arch/x86_64/kernel/io_apic.c.
Doing this fixed the oops for me.

Thanks,

-tduffy

--- arch/x86_64/kernel/io_apic.c.orig	2004-08-23 15:28:16.574961000 -0700
+++ arch/x86_64/kernel/io_apic.c	2004-08-23 15:10:41.319960000 -0700
@@ -80,7 +80,7 @@
  * shared ISA-space IRQs, so we have to support them. We are super
  * fast in the common case, and fast for shared ISA-space IRQs.
  */
-static void __init add_pin_to_irq(unsigned int irq, int apic, int pin)
+static void add_pin_to_irq(unsigned int irq, int apic, int pin)
 {
 	static int first_free_entry = NR_IRQS;
 	struct irq_pin_list *entry = irq_2_pin + irq;
z

Attachment: signature.asc
Description: This is a digitally signed message part