From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1NiaP1-000688-IJ
	for qemu-devel@nongnu.org; Fri, 19 Feb 2010 16:28:47 -0500
Received: from [199.232.76.173] (port=36537 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1NiaP0-00066q-St
	for qemu-devel@nongnu.org; Fri, 19 Feb 2010 16:28:46 -0500
Received: from Debian-exim by monty-python.gnu.org with spam-scanned (Exim
	4.60) (envelope-from <anthony@codemonkey.ws>) id 1NiaOz-0002X8-Ep
	for qemu-devel@nongnu.org; Fri, 19 Feb 2010 16:28:46 -0500
Received: from mail-yw0-f197.google.com ([209.85.211.197]:32786)
	by monty-python.gnu.org with esmtp (Exim 4.60)
	(envelope-from <anthony@codemonkey.ws>) id 1NiaOz-0002X2-5o
	for qemu-devel@nongnu.org; Fri, 19 Feb 2010 16:28:45 -0500
Received: by ywh35 with SMTP id 35so491231ywh.4
	for <qemu-devel@nongnu.org>; Fri, 19 Feb 2010 13:28:44 -0800 (PST)
Message-ID: <4B7F028A.8050900@codemonkey.ws>
Date: Fri, 19 Feb 2010 15:28:42 -0600
From: Anthony Liguori <anthony@codemonkey.ws>
MIME-Version: 1.0
Subject: Re: [Qemu-devel] [PATCH 1/3] use lazy initialization for display_state
References: <1265844597-29941-1-git-send-email-pbonzini@redhat.com>
	<1265844597-29941-2-git-send-email-pbonzini@redhat.com>
In-Reply-To: <1265844597-29941-2-git-send-email-pbonzini@redhat.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: qemu-devel@nongnu.org

On 02/10/2010 05:29 PM, Paolo Bonzini wrote:
> Ensure initialization of a dumb display, if needed, by making
> all accesses go through get_displaystate.
>
> Signed-off-by: Paolo Bonzini<pbonzini@redhat.com>
>    

Applied all.  Thanks.

Regards,

Anthony Liguori

> ---
>   vl.c |   29 +++++++++++++++--------------
>   1 files changed, 15 insertions(+), 14 deletions(-)
>
> diff --git a/vl.c b/vl.c
> index 5ddf1fe..94aeb5e 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -2568,6 +2568,16 @@ struct DisplayAllocator default_allocator = {
>       defaultallocator_free_displaysurface
>   };
>
> +/* dumb display */
> +
> +static void dumb_display_init(void)
> +{
> +    DisplayState *ds = qemu_mallocz(sizeof(DisplayState));
> +    ds->allocator =&default_allocator;
> +    ds->surface = qemu_create_displaysurface(ds, 640, 480);
> +    register_displaystate(ds);
> +}
> +
>   void register_displaystate(DisplayState *ds)
>   {
>       DisplayState **s;
> @@ -2580,6 +2590,9 @@ void register_displaystate(DisplayState *ds)
>
>   DisplayState *get_displaystate(void)
>   {
> +    if (!display_state) {
> +        dumb_display_init();
> +    }
>       return display_state;
>   }
>
> @@ -2589,16 +2602,6 @@ DisplayAllocator *register_displayallocator(DisplayState *ds, DisplayAllocator *
>       return ds->allocator;
>   }
>
> -/* dumb display */
> -
> -static void dumb_display_init(void)
> -{
> -    DisplayState *ds = qemu_mallocz(sizeof(DisplayState));
> -    ds->allocator =&default_allocator;
> -    ds->surface = qemu_create_displaysurface(ds, 640, 480);
> -    register_displaystate(ds);
> -}
> -
>   /***********************************************************/
>   /* I/O handling */
>
> @@ -5871,10 +5874,8 @@ int main(int argc, char **argv, char **envp)
>       if (qemu_opts_foreach(&qemu_device_opts, device_init_func, NULL, 1) != 0)
>           exit(1);
>
> -    if (!display_state)
> -        dumb_display_init();
>       /* just use the first displaystate for the moment */
> -    ds = display_state;
> +    ds = get_displaystate();
>
>       if (display_type == DT_DEFAULT) {
>   #if defined(CONFIG_SDL) || defined(CONFIG_COCOA)
> @@ -5932,7 +5933,7 @@ int main(int argc, char **argv, char **envp)
>           qemu_mod_timer(nographic_timer, qemu_get_clock(rt_clock));
>       }
>
> -    text_consoles_set_display(display_state);
> +    text_consoles_set_display(ds);
>
>       if (qemu_opts_foreach(&qemu_mon_opts, mon_init_func, NULL, 1) != 0)
>           exit(1);
>