On Wed, Jan 17, 2007 at 10:09:08AM +0059, Jiri Slaby wrote: > [EMAIL PROTECTED] wrote: > [...] > > +Tips on when/where to use the above attributes: > > + o The module_init()/module_exit() functions (and all > > + initialization functions called _only_ from these) > > + should be marked __init/__exit. > > + > > + o Do not mark the struct pci_driver. > > + > > + o The ID table array should be marked __devinitdata. > > Is that correct? It panics somewehere IIRC?
If it's marked __initdata it can panic if we try to access the data when adding a new PCI device after the system is up and running (pci hotplug or dynamic pci ids support.) That's why it needs to be either no marking or __devinitdata (which resolves to nothing if CONFIG_HOTPLUG is enabled.) thanks, greg k-h - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/