[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: [Xen-devel] [Bug] Fix for QEMU 0.8.1 Window Freeze


  • To: "Christian Limpach" <Christian.Limpach@xxxxxxxxxxxxx>
  • From: "Huang2, Wei" <Wei.Huang2@xxxxxxx>
  • Date: Tue, 11 Jul 2006 18:19:49 -0500
  • Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
  • Delivery-date: Tue, 11 Jul 2006 16:20:25 -0700
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: AcalOh5Iz3wo6KwmRsCU5qdvPhDivQABOIXg
  • Thread-topic: [Xen-devel] [Bug] Fix for QEMU 0.8.1 Window Freeze

The discrepancy comes from the guest configuration file. With [sdl=0 and
vnc=1], vnc-fixes patch works fine. However, if DomU uses [sdl=1 and
vnc=0], I think vnc_update_client() will not be called.

-Wei
-----Original Message-----
From: Christian Limpach [mailto:christian.limpach@xxxxxxxxx] 
Sent: Tuesday, July 11, 2006 5:34 PM
To: Huang2, Wei
Cc: Christian Limpach; xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: Re: [Xen-devel] [Bug] Fix for QEMU 0.8.1 Window Freeze

On 7/11/06, Huang2, Wei <Wei.Huang2@xxxxxxx> wrote:
> We had trouble with the latest QEMU 0.8.1 tree
> (http://xenbits.xensource.com/chris/xs-tools-ioemu.pq.hg). Basically,
> QEMU window appeared but freezed immediately. This happened to both
> 32-bit and 64-bit hypervisors.
>
> The root cause of this problem is in vnc-fixes patch. It removes
> initialization of gui_timer (see below), which actually is used by
> gui_update() in vl.c. Retaining these two statements will fix the
> problem and QEMU can perform normally.

That's not the root cause because the gui_timer is obselete, or at
least it should be.

gui_update only calls display_state.dpy_refresh but the vnc-fixes
patch also changes vnc_update_client to call dpy_refresh and that gets
called from the vs->timer.

Could you check if somehow the vnc_update_client doesn't get called for
you?

    christian

>
>
> ===========
> diff -r 15da4d2106fe vl.c
> --- a/vl.c      Thu Jul 06 14:27:28 2006 +0100
> +++ b/vl.c      Thu Jul 06 20:19:49 2006 +0100
> @@ -5972,8 +5972,7 @@ int main(int argc, char **argv)
>                    kernel_filename, kernel_cmdline, initrd_filename,
>                    timeoffset);
>
> -    gui_timer = qemu_new_timer(rt_clock, gui_update, NULL);     <===
> -    qemu_mod_timer(gui_timer, qemu_get_clock(rt_clock));        <===
> +    display_state.dpy_refresh(&display_state);
>
>  #ifdef CONFIG_GDBSTUB
>      if (use_gdbstub) {
> ...
> ===========
>
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel
>




_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.