From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34548) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bkqrE-0001Zh-6s for qemu-devel@nongnu.org; Fri, 16 Sep 2016 06:59:00 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bkqrA-0008J3-7t for qemu-devel@nongnu.org; Fri, 16 Sep 2016 06:59:00 -0400 Message-ID: <1474023532.5756.19.camel@redhat.com> From: Gerd Hoffmann Date: Fri, 16 Sep 2016 12:58:52 +0200 In-Reply-To: <1473977179.8689.370.camel@kernel.crashing.org> References: <1473977179.8689.370.camel@kernel.crashing.org> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Mime-Version: 1.0 Subject: Re: [Qemu-devel] Adding resolutions to the VGA driver List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Benjamin Herrenschmidt Cc: Programmingkid , "list@suse.de:PowerPC list:PowerPC" , qemu-devel qemu-devel On Fr, 2016-09-16 at 08:06 +1000, Benjamin Herrenschmidt wrote: > On Thu, 2016-09-15 at 13:21 -0400, Programmingkid wrote: > > There has been talk about what resolutions to add support for in the > > VGA driver. What do you think of this list: Which driver? Linux kernel driver just uses the kernel's internal mode database, beside that you can add anything you want (for example via xrandr). > We should add check for the vram amount. There's only 16M emulated > iirc, we need to check the combination resolution/depth fits. Yes, vram size should be checked, at runtime. 16M is just the default, you can have anything from 1M to 256M. cheers, Gerd