Re: [libvirt] [PATCH v3 03/17] conf: Add <disk model='virtio-{non-}transitional'/>

On Fri, 2019-02-08 at 17:11 -0500, Cole Robinson wrote: [...] s/, see/. See/ [...] I'd reword this a bit to highlight the fact that protocol compatibility is the main point of these devices, and slot assignment stems from it. Something like This device can work both with virtio 0.9 and virtio 1.0 guest drivers, so it's the best choice when compatibility with older guest operating systems is desired. libvirt will plug the device into a conventional PCI slot. In the same vein as the above, This device can only work with virtio 1.0 guest drivers, and it's the recommended option unless compatibility with older guest operating systems is necessary. libvirt will plug the device into either a PCI Express slot or a conventional PCI slot based on the machine type, resulting in a more optimized PCI topology. And here too, This device will work like a <code>virtio-non-transitional</code> device when plugged into a PCI Express slot, and like a <code>virtio-transitional</code> device otherwise; libvirt will pick one or the other based on the machine type. This is the best choice when compatibility with libvirt versions older than 5.1.0 is necessary, but it's otherwise not recommended to use it. The idea behind these suggestions is that we should help the user understand the various trade-offs and pick the value that better suits their needs. [...] s/index='0' // [...] Here too. Reviewed-by: Andrea Bolognani <abologna(a)redhat.com&gt; -- Andrea Bolognani / Red Hat / Virtualization