Re: [PATCH v2] qemu: Pass the number of heads even with -vga qxl

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

 



On Thu, Jun 15, 2017 at 01:42:33PM +0200, Martin Kletzander wrote:
> When added in multiple previous commits, it was used only with -device
> qxl(-vga), but for some QEMUs (< 1.6) we need to add this
> functionality when using -vga qxl as well.
> 
> Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1283207
> 
> Signed-off-by: Martin Kletzander <mkletzan@xxxxxxxxxx>
> ---
> v2:
>  - Do not change the domain definition
>  - Adjust the code so that it looks the same as other chunks before above
> 
> v1:
>  - https://www.redhat.com/archives/libvir-list/2017-June/msg00606.html
> 
>  src/qemu/qemu_command.c                            |  7 ++++
>  .../qemuxml2argv-video-vga-qxl-heads.args          | 30 ++++++++++++++
>  .../qemuxml2argv-video-vga-qxl-heads.xml           | 47 ++++++++++++++++++++++
>  tests/qemuxml2argvtest.c                           |  3 ++
>  4 files changed, 87 insertions(+)
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-video-vga-qxl-heads.args
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-video-vga-qxl-heads.xml

Reviewed-by: Pavel Hrdina <phrdina@xxxxxxxxxx>

Attachment: signature.asc
Description: Digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[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