[libvirt] [PATCH] libxl: Implement basic video device selection
eblake at redhat.com
Wed Jul 16 21:16:20 UTC 2014
On 07/16/2014 03:05 PM, Jim Fehlig wrote:
>> + b_info->video_memkb = def->videos->vram ?
>> + def->videos->vram :
>> + LIBXL_MEMKB_DEFAULT;
> While testing this, I noticed that libvirt will set vram to 9216 if not
> specified. E.g.
The 9216 default for qemu is absolutely stupid. No real hardware has a
limit of 9M (8M or 16M are more likely). Please feel free to not
perpetuate that stupidity into libxl.
> With type='vga', libxl will then fail to start the domain
> libxl: error: libxl_create.c:253:libxl__domain_build_info_setdefault:
> videoram must be at least 16 MB for STDVGA on QEMU_XEN
> This could be handled in libxlDomainDeviceDefPostParse(), where we can
> check for sane vram values for the various VIR_DOMAIN_VIDEO_TYPE_*, or
> set sane defaults if vram is not specified.
Sounds like for libxl, a sane default is 16M.
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 604 bytes
Desc: OpenPGP digital signature
More information about the libvir-list