[Qemu-devel] [Bug 723871] Re: qemu-kvm-0.14.0 Aborts with -vga qxl

[Boris Derzhavets]

Also in meantime git://git.kernel.org/pub/scm/virt/kvm/qemu-kvm.git
contains commits:-

commit 196a778428989217b82de042725dc8eb29c8f8d8
Author: Gerd Hoffmann <address@hidden>
Date:   Wed Apr 27 16:06:10 2011 +0200

    spice: drop obsolete iothread locking
    
    We don't use qemu internals from spice server context any more.
    Thus we don't also need to grab the iothread mutex from spice
    server context.  And we don't have to temporarely release the
    lock to avoid deadlocks.  Drop all the calls.
    
    Signed-off-by: Gerd Hoffmann <address@hidden>

commit 075360945860ad9bdd491921954b383bf762b0e5
Author: Gerd Hoffmann <address@hidden>
Date:   Wed Apr 27 15:50:32 2011 +0200

    spice: don't call displaystate callbacks from spice server context.
    
    This patch moves the displaystate callback calls for setting the cursor
    and the mouse pointer from spice server to qemu (iothread) context.
    This allows us to simplify locking.
    
    Signed-off-by: Gerd Hoffmann <address@hidden>

commit e0c64d08d11736dcea7c5a6373e3e7f62db51d9e
Author: Gerd Hoffmann <address@hidden>
Date:   Wed Apr 27 15:21:51 2011 +0200

    spice: don't create updates in spice server context.
    
    This patch moves the creation of spice screen updates from the spice
    server context to qemu iothread context (display refresh timer to be
    exact).  This way we avoid accessing qemu internals (display surface)
    from spice thread context which in turn allows us to simplify locking.
    
    Signed-off-by: Gerd Hoffmann <address@hidden>

commit 14da8345b2f7c21bab20fd12b755a61d6277f171
Author: Jes Sorensen <address@hidden>
Date:   Tue Feb 1 15:53:23 2011 +0100

    Make spice dummy functions inline to fix calls not checking return values
    
    qemu_spice_set_passwd() and qemu_spice_set_pw_expire() dummy functions
    needs to be inline, in order to handle the case where they are called
    without checking the return value.
    
    Signed-off-by: Jes Sorensen <address@hidden>
    Signed-off-by: Gerd Hoffmann <address@hidden>

So , git clone git://git.kernel.org/pub/scm/virt/kvm/qemu-kvm.git
creates a directory allows to build  tar ball for debian "qemu-kvm"  packages , 
which work with no "QXL" patching at all.

-- 
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/723871

Title:
  qemu-kvm-0.14.0 Aborts with -vga qxl

Status in QEMU:
  Invalid
Status in “libvirt” package in Ubuntu:
  Triaged
Status in “qemu-kvm” package in Ubuntu:
  Fix Released

Bug description:
  Host CPU is Core i7 Q820.  KVM is from 2.6.35-gentoo-r5 kernel (x86_64).
  Host has spice-0.7.2 and spice-protocol-0.7.0.
  Guest is Windows XP SP3 with qxl driver 0.6.1, virtio-serial 1.1.6 and 
vdagent 0.6.3.

  qemu-kvm is started like so:
  qemu-system-x86_64 -cpu host -enable-kvm -pidfile /home/rick/qemu/hds/wxp.pid 
-drive 
file=/home/rick/qemu/hds/wxp.raw,if=virtio,media=disk,aio=native,snapshot=on -m 
768 -name WinXP -net nic,model=virtio -net user -localtime -usb -vga qxl 
-device virtio-serial -chardev spicevmc,name=vdagent,id=vdagent -device 
virtserialport,chardev=vdagent,name=com.redhat.spice.0 -spice 
port=1234,disable-ticketing -monitor stdio
  and crashes with:
  qemu-system-x86_64: /home/rick/qemu/src/qemu-kvm-0.14.0/qemu-kvm.c:1724: 
kvm_mutex_unlock: Assertion `!cpu_single_env' failed.
  Aborted

  If I use -no-kvm, it works fine.  If I use -vga std, it works fine.
  -enable-kvm and -vga qxl crashes.