Ivan Mishonov wrote: > Yes, that makes sense. I'll try to find some time next week to redo my > code and send another patch. Since my time for working on libvirt is > very limited can you confirm that the LPC configuration should look like > this: > > <controller type="isa-bridge" index="0"> > <address type="pci" domain="0" bus="0" slot="NNN" function="0"/> > </controller> This looks reasonable to me. However, it adds some corner cases we need to handle: 1. I'm wondering if we should still default to 31 if this entry is not specified? We can generate this entry when post-processing XML, but I'm not sure what's the best way to handle upgrades for the existing domains... 2. According to bhyve(8) manual page, lpc is only supported on bus 0, so need to add 'isa-bridge' specific validation to check that. Roman Bogorodskiy
Attachment:
signature.asc
Description: PGP signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
