Re: insmod segfault in pci_find_subsys()

[Toralf Lund <toralf@procaptura.com>]

Greg KH wrote:

> On Fri, Mar 18, 2005 at 10:12:05AM +0100, Toralf Lund wrote:
>  
>
> > Am I seeing an issue with the PCI functions here, or is it just that I 
> > fail to spot an obvious mistake in the module itself?
> >    
>
> I think it's a problem in your code.  I built and ran the following
> example module just fine (based on your example, which wasn't the
> smallest or cleanest...), with no oops.  Does this code work for you?
>  
OK, I've finally been able to test this, and no, it does not work. 
insmod segfaults and the system log says

kernel: Unable to handle kernel paging request at virtual address 533e3762



> Oh, and the pci_find* functions are depreciated, do not use them, they
> are going away in the near future.  Please use the pci_get* functions
> instead.
>
> thanks,
>
> greg k-h
>
> -----------------
> #include <linux/pci.h>
> #include <linux/module.h>
>
> MODULE_LICENSE("GPL");
>
> 	
> static void __exit exit(void)
> {  
> }
>
> static __init int init(void)
> {
> 	struct pci_dev *dev;
>
> 	printk(KERN_DEBUG "Scanning all devices...\n");
>
> 	dev = NULL;
> 	while ((dev = pci_find_device(PCI_ANY_ID, PCI_ANY_ID, dev))) {
> 		printk(KERN_DEBUG "Device %04hx:%04hx\n",
> 			dev->vendor, dev->device);
> 	}
> 	return 0;
> }
>
> module_init(init);
> module_exit(exit);
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
>  


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/