Re: CVS commit: src/sys/modules

[Paul Goyette <paul%whooppee.com@localhost>]

On Tue, 27 Sep 2016, Christos Zoulas wrote:

> | > Some of these drivers require PCI on machines that don't have it.
> | > We need to re-think the bus-specific attachments on modules...
> |
> | Yes, I was afraid of this...
> |
> | An earlier suggestion (of having all the attachments provide a "bus"
> | interface (such as ld_bus) won't currently work because config(1) does
> | not allow the interface attribute as a pseudo-root...
>
> The expedient fix is to disable the pci dependency modules until
> we come up with something better (simpler than doing conditional
> compilation and playing games with the sets). I think that config
> will need some work no matter work. There is even more complexity
> for drivers that support multiple bus attachments. Do those present
> themselves as a single module, or multiple modules? The directory
> structure of modules is not scalable, I am not sure what to do
> about that yet.

I'd rather not completely disable these, as I fear that a more proper 
solution will be a long time coming (if ever).  So, rather than just 
disabling them, I'd prefer to make them specific to amd64 and i386.  The 
initial module (for combined nvme and ld_nvme) was (and still is) built 
only for these two platforms.

I look forward to a productive discussion on tech-kern for ways to move 
forward.



+------------------+--------------------------+------------------------+
| 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 |
+------------------+--------------------------+------------------------+