[PATCH v3 0/4] Add resolution properties for QEMU video devices

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

 



From: Julio Faracco <jcfaracco@xxxxxxxxx>

This serie adds resolution ('x' and 'y') properties into XML definition
for QEMU video devices to specify a default resolution. This specific
case is not considering those attributes as a QEMU capabilities due to
complexity of code versus test complexity versus a real gain. This 
skeleton would work very well initially. After, it should be possible to
include them as a capabilities without changing this serie.

v1-v2: Adds suggestions of multiple members.
v2-v3: Adds Cole's suggestions.

Julio Faracco (4):
  docs: Add 'xres' and 'yres' into video XML definition
  qemu: Generate 'xres' and 'yres' for video devices
  conf: Adding XML resolution support for video models
  tests: Add separate tests for resolution 'x' and 'y'

 docs/formatdomain.html.in                     |  5 +-
 docs/schemas/domaincommon.rng                 | 10 +++
 src/conf/domain_conf.c                        | 74 ++++++++++++++++++-
 src/conf/domain_conf.h                        |  5 ++
 src/conf/virconftypes.h                       |  3 +
 src/qemu/qemu_command.c                       |  5 ++
 .../video-qxl-resolution.args                 | 32 ++++++++
 .../qemuxml2argvdata/video-qxl-resolution.xml | 42 +++++++++++
 tests/qemuxml2argvtest.c                      |  4 +
 .../video-qxl-resolution.xml                  | 42 +++++++++++
 tests/qemuxml2xmltest.c                       |  1 +
 11 files changed, 221 insertions(+), 2 deletions(-)
 create mode 100644 tests/qemuxml2argvdata/video-qxl-resolution.args
 create mode 100644 tests/qemuxml2argvdata/video-qxl-resolution.xml
 create mode 100644 tests/qemuxml2xmloutdata/video-qxl-resolution.xml

-- 
2.20.1

--
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