On 04/28/2014 03:38 AM, Paolo Bonzini wrote: > Il 04/04/2014 22:46, Eric Blake ha scritto: >>> > >>> > I'm still not seeing what's wrong with depending on the version >>> > number since other features are also depending on the version >>> > number. >> Every feature where we have to guess based on version number is due to a >> bug in qemu for not providing enough information for libvirt to probe >> for the feature's existence. We hate those features, and I have been >> lobbying hard on the qemu list that all NEW features should be >> discoverable. rdma is one such feature - I recall you making changes in >> your series there so that it is discoverable in response to my early >> review comments - so now please USE that methodology from libvirt. >> > > I think that relied on the QAPI introspection. The MigrationCapability > rdma-pin-all will be present only if QEMU supports RDMA migration. > > But QAPI introspection is not there yet. :( I know that qapi introspection is not there yet, but 'query-migrate-capabilities' IS there, and the presence or absence of rdma-pin-all in the set of queried capabilities IS a reliable witness of whether rdma support is available. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list