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. cheers, Gerd