Very Computer

your recent "ServeRAID driver update" patch to Linux 2.4.20-pre removed
the __devexit_p from the pointer to the __devexit function
ips_remove_device and added two other pointers to this function that don't
use __devexit_p. This results in the following compile error when
compiling the driver statically into a kernel without support for
hot-plugging:

<--  snip  -->

...
        --end-group \
        -o vmlinux
drivers/scsi/scsidrv.o(.data+0xc3f4): undefined reference to `local
symbols in discarded section .text.exit'
drivers/scsi/scsidrv.o(.data+0xc434): undefined reference to `local
symbols in discarded section .text.exit'
drivers/scsi/scsidrv.o(.data+0xc474): undefined reference to `local
symbols in discarded section .text.exit'
make: *** [vmlinux] Error 1

<--  snip  -->

The fix is simple:

--- drivers/scsi/ips.c.old      2002-09-05 19:38:13.000000000 +0200

        name:           ips_hot_plug_name,
        id_table:       ips_pci_table,
        probe:          ips_insert_device,
-       remove:         ips_remove_device,
+       remove:         __devexit_p(ips_remove_device),
    };

    struct pci_driver ips_pci_driver_5i = {
        name:           ips_hot_plug_name,
        id_table:       ips_pci_table_5i,
        probe:          ips_insert_device,
-       remove:         ips_remove_device,
+       remove:         __devexit_p(ips_remove_device),
    };

    struct pci_driver ips_pci_driver_i960 = {
        name:           ips_hot_plug_name,
        id_table:       ips_pci_table_i960,
        probe:          ips_insert_device,
-       remove:         ips_remove_device,
+       remove:         __devexit_p(ips_remove_device),
    };

 #endif

cu
Adrian

--

You only think this is a free country. Like the US the UK spends a lot of
time explaining its a free country because its a police state.
                                                                Alan Cox

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in

More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thanks for the info.  We have already been made aware of this mistake and
have corrected this in our current development code which just entered test
last week.  As soon as we get a couple weeks of successful testing under our
belts, we will be updating Linus with a new ips driver for 2.5.   In the
meantime, if anyone needs this and can't wait for the new driver to come
out, your patch should work just fine.  

Thanks again for keeping us posted.

Jack