On Fri, 2019-02-08 at 17:11 -0500, Cole Robinson wrote:
[...]
> - if (hostdev->source.subsys.type ==
> VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_SCSI_HOST)
> + if (hostdev->source.subsys.type ==
> VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_SCSI_HOST) {
> + switch ((virDomainHostdevSubsysSCSIVHostModelType)
> hostdev->source.subsys.u.scsi_host.model) {
> + case
> VIR_DOMAIN_HOSTDEV_SUBSYS_SCSI_VHOST_MODEL_TYPE_VIRTIO_TRANSITIONAL:
> + /* Transitional devices only work in conventional PCI slots
> */
> + return pciFlags;
> + case VIR_DOMAIN_HOSTDEV_SUBSYS_SCSI_VHOST_MODEL_TYPE_VIRTIO:
> + case
> VIR_DOMAIN_HOSTDEV_SUBSYS_SCSI_VHOST_MODEL_TYPE_VIRTIO_NON_TRANSITIONAL:
> + case VIR_DOMAIN_HOSTDEV_SUBSYS_SCSI_VHOST_MODEL_TYPE_DEFAULT:
> + return virtioFlags;
> + case VIR_DOMAIN_HOSTDEV_SUBSYS_SCSI_VHOST_MODEL_TYPE_LAST:
> + break;
> + }
> + return 0;
> +
> + /* Transitional devices only work in conventional PCI slots */
> + if (hostdev->source.subsys.u.scsi_host.model ==
> VIR_DOMAIN_HOSTDEV_SUBSYS_SCSI_VHOST_MODEL_TYPE_VIRTIO_TRANSITIONAL)
> + return pciFlags;
> return virtioFlags;
These last 5 lines are a leftover from v2, please get rid of them.
Reviewed-by: Andrea Bolognani <abolo...@redhat.com>
--
Andrea Bolognani / Red Hat / Virtualization
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
