Re: [libvirt] [PATCH] qemu: Support ram bar size for qxl devices

On Thu, Jan 17, 2013 at 09:35:22PM +0200, Alon Levy wrote: > Adds a qxl-ram attribute globaly to the video.model element, that > changes > the resulting qemu command line only if video.type == "qxl". > > That attribute gets a default value of 64*1024 only if model.type > is > "qxl". In effect not changing any xml or argv for non qxl devices. > > For qxl devices a new property is set: > -global qxl-vga.ram_size=<ram>*1024 > or > -global qxl.ram_size=<ram>*1024 > > For the main and secondary qxl devices respectively. > > The default for the qxl ram bar is the same as the default for the > qxl > vram bar, 64*1024. > --- > I've added a qxl-ram attribute. There is no precedent for adding am > attribute > prefixed like this, so I'm open for any other suggestion on how to > do it. What is the difference between this new RAM size and what we currently set for ram size. That will influence how we pick a good name, ie one with any qxl prefix, which is not something we can use.

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