Re: [Xen-devel] libxl videoram for cirrus graphics

[Frediano Ziglio <frediano.ziglio@xxxxxxxxxx>]

On Tue, 2013-09-17 at 12:54 +0000, Rob Hoes wrote:
> Hi,
> 
> The videoram setting in xl config files is documented as follows:
> 
> "
> videoram=MBYTES
> 
>     Sets the amount of RAM which the emulated video card will contain, which 
> in turn limits the resolutions and bit depths which will be available. The 
> default amount of video ram for stdvga is 8MB which is sufficient for e.g. 
> 1600x1200 at 32bpp and videoram option is currently working only when using 
> the qemu-xen-traditional device-model.
> 
>     When using the emulated Cirrus graphics card (vga="cirrus") the amount of 
> video ram is fixed at 4MB which is sufficient for 1024x768 at 32 bpp and 
> videoram option is currently working only when using the upstream qemu-xen 
> device-model.
> "
> 
> XenServer also uses a default of 4MB video ram for cirrus graphics.
> 
> However, when I use xl to create a VM with cirrus graphics, I get 8MB of 
> video RAM by default. Moreover, setting it to videoram=4 in the config file 
> leads to complaints from libxl:
> 
>     libxl_create.c:228:libxl__domain_build_info_setdefault: videoram must be 
> at least 8 mb
> 
> Is there a bug in the logic in libxl_create.c?
> 
> Cheers,
> Rob
> 

cirrus card support maximum of 4mb (which is also the default) so there
is IMHO no point in reporting an error, just a warning would be enough

Frediano



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel