Re: CVS commit: src/sys/arch/amd64/conf

[Masao Uebayashi <uebayasi%gmail.com@localhost>]

On Sun, Oct 12, 2014 at 12:05 AM, Manuel Bouyer <bouyer%antioche.eu.org@localhost> wrote:
> On Sat, Oct 11, 2014 at 09:50:03AM +0000, Masao Uebayashi wrote:
>> Module Name:  src
>> Committed By: uebayasi
>> Date:         Sat Oct 11 09:50:03 UTC 2014
>>
>> Modified Files:
>>       src/sys/arch/amd64/conf: XEN3_DOM0 std.xen
>>
>> Log Message:
>> Don't include std.ath_hal for XEN3_DOMU.
>
> Why ?
> We still support PCI pass-through, so we could have a ath in a domU

In that case you have to enable xpci/pci at first.

Including "std.ath_hal" means that you pull in ath device code in your
kernel.  But you don't have no parent buses.  This might be OK for
ath(4), but in general, it means that config(1) doesn't resolve
dependency, and your kernel may be bloated by unnecessary code.

(config(1) is not great in reporting dependency errors yet.)