Re: [BUG] qemu git error with virgl

[Igor Mammedov <imammedo@redhat.com>]

On Sun, 3 Jan 2021 18:28:11 +0100
Philippe Mathieu-Daudé <philmd@redhat.com> wrote:

> Cc'ing Gerd + patch author/reviewer.
> 
> On 1/2/21 2:11 PM, Torsten Wohlfarth wrote:
> > Hello,
> > 
> > i can't start any system if i use virgl. I get the following error:
> > 
> > qemu-x86_64: ../ui/console.c:1791: dpy_gl_ctx_create: Assertion
> > `con->gl' failed.

Does following fix issue:
  [PULL 12/55] vl: initialize displays _after_ exiting preconfiguration

> > ./and.sh: line 27: 3337167 Aborted                 qemu-x86_64 -m 4096
> > -smp cores=4,sockets=1 -cpu host -machine pc-q35-4.0,accel=kvm -device
> > virtio-vga,virgl=on,xres=1280,yres=800 -display sdl,gl=on -device
> > intel-hda,id=sound0,msi=on -device
> > hda-micro,id=sound0-codec0,bus=sound0.0,cad=0 -device qemu-xhci,id=xhci
> > -device usb-tablet,bus=xhci.0 -net
> > nic,macaddr=52:54:00:12:34:62,model=e1000 -net
> > tap,ifname=$INTERFACE,script=no,downscript=no -drive
> > file=/media/daten2/image/lineageos.qcow2,if=virtio,index=1,media=disk,cache=none,aio=threads
> > 
> > Set 'tap3' nonpersistent
> > 
> > i have bicected the issue:
> >   
> > towo:Defiant> git bisect good  
> > b4e1a342112e50e05b609e857f38c1f2b7aafdc4 is the first bad commit
> > commit b4e1a342112e50e05b609e857f38c1f2b7aafdc4
> > Author: Paolo Bonzini <pbonzini@redhat.com>
> > Date:   Tue Oct 27 08:44:23 2020 -0400
> > 
> >     vl: remove separate preconfig main_loop
> > 
> >     Move post-preconfig initialization to the x-exit-preconfig. If
> > preconfig
> >     is not requested, just exit preconfig mode immediately with the QMP
> >     command.
> > 
> >     As a result, the preconfig loop will run with accel_setup_post
> >     and os_setup_post restrictions (xen_restrict, chroot, etc.)
> >     already done.
> > 
> >     Reviewed-by: Igor Mammedov <imammedo@redhat.com>
> >     Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> > 
> >  include/sysemu/runstate.h |  1 -
> >  monitor/qmp-cmds.c        |  9 -----
> >  softmmu/vl.c              | 95
> > ++++++++++++++++++++---------------------------
> >  3 files changed, 41 insertions(+), 64 deletions(-)
> > 
> > Regards,
> > 
> > Torsten Wohlfarth
> > 
> > 
> >   
> 
>