Bug 1030812

Can you provide the specific qemu-kvm command line that you used?

Yes, I'm running the VM using virt-manager. 
The command line used:

/usr/libexec/qemu-kvm -name rhel7beta -S -machine pc-i440fx-rhel7.0.0,accel=kvm,usb=off -m 2048 -realtime mlock=off -smp 2,sockets=2,cores=1,threads=1 -uuid 4ebd40ec-1e77-49a2-bd89-d65523848043 -no-user-config -nodefaults -chardev socket,id=charmonitor,path=/var/lib/libvirt/qemu/rhel7beta.monitor,server,nowait -mon chardev=charmonitor,id=monitor,mode=control -rtc base=utc -no-shutdown -boot strict=on -device piix3-usb-uhci,id=usb,bus=pci.0,addr=0x1.0x2 -device virtio-serial-pci,id=virtio-serial0,bus=pci.0,addr=0x5 -drive file=/var/lib/libvirt/images/rhel7beta.img,if=none,id=drive-virtio-disk0,format=raw,cache=none -device virtio-blk-pci,scsi=off,bus=pci.0,addr=0x6,drive=drive-virtio-disk0,id=virtio-disk0,bootindex=1 -netdev tap,fd=23,id=hostnet0,vhost=on,vhostfd=24 -device virtio-net-pci,netdev=hostnet0,id=net0,mac=52:54:00:02:19:1c,bus=pci.0,addr=0x3 -chardev pty,id=charserial0 -device isa-serial,chardev=charserial0,id=serial0 -chardev spicevmc,id=charchannel0,name=vdagent -device virtserialport,bus=virtio-serial0.0,nr=1,chardev=charchannel0,id=channel0,name=com.redhat.spice.0 -device usb-tablet,id=input0 -spice port=5900,addr=127.0.0.1,disable-ticketing,seamless-migration=on -vga qxl -global qxl-vga.ram_size=67108864 -global qxl-vga.vram_size=67108864 -device intel-hda,id=sound0,bus=pci.0,addr=0x4 -device hda-duplex,id=sound0-codec0,bus=sound0.0,cad=0 -device virtio-balloon-pci,id=balloon0,bus=pci.0,addr=0x7

Can you reproduce with virt-viewer 0.5.7-3? It's possible that this is related to bug 1022769...

Yes I can. I've tested this with virt-viewer-0.5.7-4.el7.x86_64 and the bug is still there.

It *appears* this bug is connected with the following code in spice-gtk3:

    if (c->surface_id != 0) {
        g_warning("FIXME: only support monitor config with primary surface 0, "
                  "but given config surface %d", c->surface_id);
        goto whole;
    }

If I comment it out I can't reproduce this bug any more.

Marc-Andre, Christophe -- any ideas?

(In reply to Søren Sandmann Pedersen from comment #6)
> It *appears* this bug is connected with the following code in spice-gtk3:
> 
>     if (c->surface_id != 0) {
>         g_warning("FIXME: only support monitor config with primary surface
> 0, "
>                   "but given config surface %d", c->surface_id);
>         goto whole;
>     }
> 
> If I comment it out I can't reproduce this bug any more.
> 

When we get a monitor config with a surface != 0, we don't have any support for that. There is no code to handle such case, and in general I expect this to be a bit tricky to get right (changing surface etc). 

I suppose this is still using the xf86-qxl driver, right? (not the drm) Any idea why we have monitor config with surface_id != 0?

This is using kernel mode setting, if that's what you mean.

(In reply to Søren Sandmann Pedersen from comment #8)
> This is using kernel mode setting, if that's what you mean.

shouldn't this bug be moved to kernel then?

That doesn't answer my question: why we have monitor config with surface_id != 0? Afaik, we shouldn't reach this, but perhaps drm/kms driver is doing something new?

I thought that the multi-monitor work added the ability to set any surface as scan-out.

Cc'ing Dave and Alon.

This problem is in drm/kms driver, as noted by Marc-André. The fix is already pushed upstream (52571ad5f4c57067ac593a6bdb1f7a35ed032d27) and has to be backported to rhel-7.1.

Shall I just move this bug to kernel?

fyi, this landed upstream in 3.15-rc3, and is part of the rhel71 drm backport

*** This bug has been marked as a duplicate of bug 1101269 ***