On Mon, Dec 17, 2012 at 12:06 AM, Andrew Thompson <[email protected]> wrote: > On 17 December 2012 18:06, Jim Harris <[email protected]> wrote: >> >> >> >> On Sun, Dec 16, 2012 at 6:53 PM, Andrew Thompson <[email protected]> >> wrote: >>> >>> On 17 December 2012 13:17, Bryan Venteicher <[email protected]> wrote: >>> >>> > There's been lots of requests to have VirtIO in GENERIC for i386 and >>> > amd64. Anybody have any issues or concerns with this or the patch at >>> > [1]. This also removes the kludge that was introduced in r239009. >>> > >>> > I've compiled LINT for i386 and amd64 so hopefully there won't be any >>> > surprise breakages. >>> > >>> > [1] http://people.freebsd.org/~bryanv/patches/virtio.generic.patch >>> >>> >>> It would be great to have the drivers enabled. You do not need the >>> sys/conf/files changes, the common and arch files are combined. >>> >> >> Removing the virtio files from sys/conf/files ensures these drivers can >> only be specified in x86 kernel configuration files. r239009 added these >> lines to sys/conf/files, but Bryan's patch does it more correctly. > >
Yes, I think the patch is correct for what I intended - support for x86 only (for now). > Linux supports virtio on ARM so I dont think its necessarily x86 MD. I guess > it can be moved back later. > I think VirtIO on ARM (on QEMU) effectively requires VirtIO-MMIO, which we don't support yet. And virtio_pci is probably missing some bus_space_barriers() required for non-x86. Both are on my TODO, but nobody has prodded me about either yet. > > Andrew _______________________________________________ [email protected] mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to "[email protected]"
