Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/modules



Well, you could look for <machine/pci_machdep.h> and compile them for
all the platforms that have PCI, but is the additional complexity worth
it compared to the benefit? Disabling them for now is the easiest way
because no matter what, the way they attach or the module is designed
will change.

I agree in principle, but I am greatly concerned about the definition of "for now". If they get disabled, they'll get forgotten. So few people seem to care about modules, and those that try to create modules often screw something up because they didn't actually build and load as a module.

So, I will undertake the effort to identify the pci-capable platforms, update the conditional compilation stuff in the modules/Makefile, and update all of the appropriate modules/md.* sets lists.


+------------------+--------------------------+------------------------+
| Paul Goyette     | PGP Key fingerprint:     | E-mail addresses:      |
| (Retired)        | FA29 0E3B 35AF E8AE 6651 | paul at whooppee.com   |
| Kernel Developer | 0786 F758 55DE 53BA 7731 | pgoyette at netbsd.org |
+------------------+--------------------------+------------------------+


Home | Main Index | Thread Index | Old Index