On 09/23/2012 10:29 PM, Osier Yang wrote:
Hi,
I think we already got enough bugs which are about "try to assign a host device, then it fails for not assignable", clearly this "try and then fails" approach is not nice.
The reason for not assignable can be various. E.g.
* IOMMU or AMD-V is not enabled/supported
* platform doesn't support interrupt remapping
There is no checking for above reasons now, an assignment will get error till the qemu process started. It's nice if checking it much earlier, and list whether the host supports device assignment as a property of capabilities?
Yes, failing earlier if it can be proven that the passthrough will not ever succeed seems worthwhile.
It will also be nice if one could known what devices are assignable (list all assignable devices? this needs to enumerate and try to unbind/reset/restore the devices, which is not desireable, as it could breaks work on host, such as NIC). Or whether a device is assignable (this shouldn't cause any problem when doing checking, because it's the requested operation, one knowns what he does).
Assuming a new API to determine whether a host device is assignable
Why would we need a new API? We can already get at each virNodeDevicePtr (see 'virsh nodedev-list') and learn about each device (see 'virsh nodedev-dumpxml'); this seems like something that you would just add an addditional XML element to that existing API instead of needing a new API. -- Eric Blake eblake@redhat.com +1-919-301-3266 Libvirt virtualization library http://libvirt.org