On Wed, Aug 23, 2017 at 09:10:09PM +0530, Varad Gautam wrote: > Hi Matthew, > > On Sat, Aug 19, 2017 at 2:02 PM, Matthew Garrett <mjg59@xxxxxxxxxxxxx> wrote: > > On Fri, Aug 18, 2017 at 09:19:11PM +0530, Varad Gautam wrote: > >> From: Stéphane Marchesin <marcheu@xxxxxxxxxxxx> > >> > >> initially reviewed for ChromiumOS at: > >> https://chromium-review.googlesource.com/339093 > >> Signed-off-by: Stéphane Marchesin <marcheu@xxxxxxxxxxxx> > > > > 1280x1024x24 fits in 4MB, 1280x1024x32 doesn't. That seems like it's > > going to be a visible change in behaviour. > > > > Right, 800x600 is the highest we can go for >24bpp, so we now switch > to that instead of 1280x1024. fb creation fails in > cirrus_check_framebuffer if the w*h*bpp doesn't fit, and plane updates > get rejected when atomic is enabled later on. So users who want 1280x1024 now have to change their configuration. Is that the intent? It seems odd to change user-visible behaviour for everyone just to fix ChromeOS in a VM. -- Matthew Garrett | mjg59@xxxxxxxxxxxxx _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
