[libvirt] [v3] qemu: Support vram for video of qxl type

[Daniel P. Berrange]

On Sun, Mar 06, 2011 at 10:04:48PM +0800, Osier Yang wrote:
> For qemu names the primary vga as "qxl-vga":
> 
>   1) if vram is specified for 2nd qxl device:
> 
>     -vga qxl -global qxl-vga.vram_size=$SIZE \
>     -device qxl,id=video1,vram_size=$SIZE,...
> 
>   2) if vram is not specified for 2nd qxl device, (use the default
>      set by global):
> 
>     -vga qxl -global qxl-vga.vram_size=$SIZE \
>     -device qxl,id=video1,...
> 
> For qemu names all qxl devices as "qxl":
> 
>   1) if vram is specified for 2nd qxl device:
> 
>     -vga qxl -global qxl.vram_size=$SIZE \
>     -device qxl,id=video1,vram_size=$SIZE ...
> 
>   2) if vram is not specified for 2nd qxl device:
> 
>     -vga qxl -global qxl-vga.vram_size=$SIZE \
>     -device qxl,id=video1,...
> 
> "-global" is the only way to define vram_size for the primary qxl
> device, regardless of how qemu names it, (It's not good a good
> way, as original idea of "-global" is to set a global default for
> a driver property, but to specify vram for first qxl device, we
> have to use it).
> 
> For other qxl devices, as they are represented by "-device", could
> specify it directly and seperately for each, and it overrides the
> default set by "-global" if specified.
> 
> v1 - v2:
>   * modify "virDomainVideoDefaultRAM" so that it returns 16M as the
>     default vram_size for qxl device.
> 
>   * vram_size * 1024 (qemu accepts bytes for vram_size).
> 
>   * apply default vram_size for qxl device for which vram_size is
>     not specified.
> 
>   * modify "graphics-spice" tests (more sensiable vram_size)
> 
>   * Add an argument of virDomainDefPtr type for qemuBuildVideoDevStr,
>     to use virDomainVideoDefaultRAM in qemuBuildVideoDevStr).
> 
> v2 - v3:
>   * Modify default video memory size for qxl device from 16M to 24M
> 
>   * Update codes to be consistent with changes on qemu_capabilities.*
> ---
>  src/conf/domain_conf.c                             |    4 ++
>  src/qemu/qemu_capabilities.c                       |    2 +
>  src/qemu/qemu_capabilities.h                       |    1 +
>  src/qemu/qemu_command.c                            |   18 ++++++++++
>  tests/qemuhelptest.c                               |    1 +
>  .../qemuxml2argv-graphics-spice-qxl-vga.args       |    7 ++++
>  .../qemuxml2argv-graphics-spice-qxl-vga.xml        |   36 ++++++++++++++++++++
>  .../qemuxml2argv-graphics-spice.args               |    4 +-
>  .../qemuxml2argv-graphics-spice.xml                |    4 +-
>  tests/qemuxml2argvtest.c                           |    4 ++
>  tests/qemuxml2xmltest.c                            |    1 +
>  11 files changed, 78 insertions(+), 4 deletions(-)
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-graphics-spice-qxl-vga.args
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-graphics-spice-qxl-vga.xml

ACK

Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|