Andrea Bolognani wrote: > On Wed, 2017-02-01 at 20:25 +0400, Roman Bogorodskiy wrote: > > Like it usually happens, I fixed one thing and broke another: > > in 803966c76 address allocation was fixed for SATA disks, but > > broke that for virtio disks, because it dropped disk address > > assignment completely. It's not needed for SATA disks anymore, > > but still needed for the virtio ones. > > > > Bring that back and add a couple of tests to make sure it won't > > happen again. > > I didn't actually test this[1], but both the code and the > tests look reasonable enough, plus 'make check' and 'make > syntax-check' are all green[2] on FreeBSD, so: > > ACK Pushed, thanks! > [1] Can you run bhyve guests inside a FreeBSD KVM guest? I guess that generally you cannot because it requires hardware virtualization support (i.e. POPCNT CPU feature and maybe something else). Probably it's possible with KVM's nested virtualization, I've been wanting to check that for a while but somehow always forget about that when I have time. > [2] Well, mostly. We really should fix qemuxml2argvtest once > and for all, and it looks like we're sooo close now! True. Roman Bogorodskiy
Attachment:
signature.asc
Description: PGP signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
