Re: [PATCH 5/7] qemu_command: Switch from VIR_ENUM_IMPL(qemuDeviceVideoSecondary) to explicit switch()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jun 11, 2021 at 17:04:58 +0200, Michal Privoznik wrote:
> This may look like a step backwards, but it isn't. The point is
> that in near future the chosen model will depend on more than
> just video type.
> 
> Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
> ---
>  src/qemu/qemu_command.c | 41 +++++++++++++++++++++--------------------
>  1 file changed, 21 insertions(+), 20 deletions(-)
> @@ -4229,7 +4210,27 @@ qemuDeviceVideoGetModel(virQEMUCaps *qemuCaps,
>                  break;
>              }
>          } else {
> -            model = qemuDeviceVideoSecondaryTypeToString(video->type);
> +            switch (video->type) {

Same as previous patch.

> +            case VIR_DOMAIN_VIDEO_TYPE_QXL:
> +                model = "qxl";
> +                break;
> +            case VIR_DOMAIN_VIDEO_TYPE_VIRTIO:
> +                model = "virtio-gpu";
> +                break;
> +            case VIR_DOMAIN_VIDEO_TYPE_DEFAULT:




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux