Re: [Qemu-devel] [PATCH] hw/qxl: allow vram to be sized to 4096 bytes

2012-02-17 Thread Alon Levy
On Fri, Feb 17, 2012 at 04:11:59PM +0100, Gerd Hoffmann wrote: > Hi, > > > -if (qxl->vram_size < 16 * 1024 * 1024) { > > +/* a request of 0 vram_size is translated to the smallest vram that > > doesn't > > + * break current off screen surfaces using drivers */ > > +if (qxl->vram

Re: [Qemu-devel] [PATCH] hw/qxl: allow vram to be sized to 4096 bytes

2012-02-17 Thread Gerd Hoffmann
Hi, > -if (qxl->vram_size < 16 * 1024 * 1024) { > +/* a request of 0 vram_size is translated to the smallest vram that > doesn't > + * break current off screen surfaces using drivers */ > +if (qxl->vram_size > 0 && qxl->vram_size < 16 * 1024 * 1024) { Is there any reason to kee

[Qemu-devel] [PATCH] hw/qxl: allow vram to be sized to 4096 bytes

2012-02-16 Thread Alon Levy
This allows us to reduce the size of the surfaces bar minimally while not breaking current drivers. To completely disable the bar, or make it zero sized, would break them. Signed-off-by: Alon Levy --- hw/qxl.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/hw/qxl.c

[Qemu-devel] [PATCH] hw/qxl: allow vram to be sized to 4096 bytes

2011-11-09 Thread Alon Levy
This allows us to reduce the size of the surfaces bar minimally while not breaking current drivers. To completely disable the bar, or make it zero sized, would break them. Signed-off-by: Alon Levy --- Causes endless ooms for the linux driver, but it's mainly meant for the windows driver, where it