Re: [Xen-devel] [PATCH 2/3] tools:libxl: Add qxl vga interface support v2

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Wed, 2012-10-24 at 09:06 +0100, ZhouPeng wrote:

> >> +For B<qxl> option, the default is 128MB. If B<videoram> is set greater
> >> +than 128MB, it will be trimmed to 128MB; if set less than 128MB, an
> >> +error will be triggered.
> >
> >
> > Is this a fixed value in qemu, or is this something that can be changed via
> > the command-line?
> Can be changed.
> 
> But Ian Campbell and me have talked on sth related with this
> closely before. And we both agree to just support the default is
> enough in most time. You can get it from the url below quickly, pls
> read from bottom up, which can save time.
> 
> http://lists.xen.org/archives/html/xen-devel/2012-07/msg00098.html

We discussed what to do if the supplied value was too small but I don't
recall this behaviour of clamping a value which is larger to a specific
value.

If we reject values which are <128M, and clamp anything which is >128M
to 128M then what is the point of the option?

Ian.


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