From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1LDOUu-00056i-4V for qemu-devel@nongnu.org; Thu, 18 Dec 2008 14:25:24 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1LDOUg-00055e-Tr for qemu-devel@nongnu.org; Thu, 18 Dec 2008 14:25:23 -0500 Received: from [199.232.76.173] (port=50726 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1LDOUg-00055Z-JQ for qemu-devel@nongnu.org; Thu, 18 Dec 2008 14:25:10 -0500 Received: from yx-out-1718.google.com ([74.125.44.156]:59069) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1LDOUf-0005mu-Ba for qemu-devel@nongnu.org; Thu, 18 Dec 2008 14:25:10 -0500 Received: by yx-out-1718.google.com with SMTP id 3so217968yxi.82 for ; Thu, 18 Dec 2008 11:25:04 -0800 (PST) Message-ID: <494AA38C.1000507@codemonkey.ws> Date: Thu, 18 Dec 2008 13:25:00 -0600 From: Anthony Liguori MIME-Version: 1.0 Subject: Re: [Qemu-devel] [PATCH 0 of 7] DisplayState interface change References: <494AA0B2.8070702@eu.citrix.com> In-Reply-To: <494AA0B2.8070702@eu.citrix.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Stefano Stabellini wrote: > Hi all, > this is the forth update of the "DisplayState interface change" series. > I'm pretty happy with the state of this patch series. I'm going to do thorough testing of each patch but if all goes well, I'd like to apply them. I think there's room to improve things but I think it's good enough now to take into the tree. Please take a look through these patches and speak up if you think someone needs to be changed before applying them. I'll wait at least until the middle of next week before applying them to give everyone time to look closely. Regards, Anthony Liguori > This update fixes the few remaining issue raised last time. > > The series is made of 7 patches: > > 1) accessors again > some other substitutions in hw/sm501.c; > > 2) remove bgr > the new DisplayState interface does not contain any host specific > display detail, it is an abstraction of the backend display, hence we > don't need to memorize the bgr flag in DisplayState. > The frontend must be able to handle a bgr display by itself, in fact sdl > is perfectly capable of that; > > 3) DisplayState interface change > this is the big patch that actually changes the interface; > > 4) vnc improvements > this patch introduces DisplaySurfaces in vnc, simplifying the code. > > 5) graphical_console_init change > this is the patch that changes the graphical_console_init function > to return an allocated DisplayState instead of a QEMUConsole, as Anthony > suggested. > This patch does *not* include any required changes to any devices, these > changes come with the two following patches. > > 6) machine changes > this patch changes the QEMUMachine init functions of all the machine types > not to take a DisplayState as an argument because is not needed any > more; > > 7) graphic device changes > this patch updates the graphic device code to use the new > graphical_console_init function. > > Patch number 5 requires also 6 and 7 to compile. > > More comments on single patches. > > Cheers, > > Stefano > > > > > >