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_size > 0 && qxl->vram_size < 16 * 1024 * 1024) { > > Is there any reason to keep the 16MB minimum size? I think we can just > drop it and simplify the logic here. See the patch series just sent.
I agree. Will review your patchset. > > cheers, > Gerd >