Bug 664192 - Error testing qemu qxl support
Summary: Error testing qemu qxl support
Alias: None
Product: Fedora
Classification: Fedora
Component: qemu
Version: 14
Hardware: x86_64
OS: Linux
Target Milestone: ---
Assignee: Justin M. Forbes
QA Contact: Fedora Extras Quality Assurance
: 669850 (view as bug list)
Depends On:
TreeView+ depends on / blocked
Reported: 2010-12-19 01:10 UTC by Amador Pahim
Modified: 2013-01-09 11:44 UTC (History)
20 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2011-02-24 09:28:35 UTC
Type: ---

Attachments (Terms of Use)
Patch for qemu package to fixing help messsages (482 bytes, application/octet-stream)
2011-01-21 13:19 UTC, Mikhail Kulemin
no flags Details
failing domain (2.00 KB, text/xml)
2011-01-29 11:39 UTC, Julian Sikorski
no flags Details
final patch version (1.62 KB, patch)
2011-01-31 00:04 UTC, Julian Sikorski
no flags Details | Diff

Description Amador Pahim 2010-12-19 01:10:08 UTC
Description of problem:
Running libvirt 0.8.6, spice parameters does not work

Version-Release number of selected component:

Steps to Reproduce:
Change the VM video device from:
      <model type='cirrus' vram='9216' heads='1'/>
      <address type='pci' domain='0x0000' bus='0x00' slot='0x02' function='0x0'/>
      <model type='qxl' heads='1'/>
      <address type='pci' domain='0x0000' bus='0x00' slot='0x02' function='0x0'/>

Try to start VM with:
# virsh start Ubuntu

Actual results:
error: Failed to start domain Ubuntu
error: unsupported configuration: This QEMU does not support QXL graphics adapters

Expected results:
Domain Ubuntu started

Additional info:
To skip this test, I had to edit the "src/qemu/qemu_command.c", in libvirt source, commenting this lines:

/*   if ((def->videos[0]->type == VIR_DOMAIN_VIDEO_TYPE_QXL) &&
 *    !(qemuCmdFlags & QEMUD_CMD_FLAG_VGA_QXL)) {
 *    qemuReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
 *                    _("This QEMU does not support QXL graphics adapters"));
 *    goto error;
 *   } 

Comment 1 Amador Pahim 2010-12-20 11:19:49 UTC
Going deeper in this problem, I saw qemu does not show the vga qxl capability in help:

[root@pahim-pc libvirt]# qemu --version
QEMU emulator version 0.13.0 (qemu-kvm-0.13.0), Copyright (c) 2003-2008 Fabrice Bellard

[root@pahim-pc libvirt]# qemu -help | grep vga
-vga [std|cirrus|vmware|xenfb|none]

I think libvirt is OK, but qemu needs to show the correct information about capabilities.

Comment 2 Pavel Zhukov 2011-01-21 13:15:39 UTC
*** Bug 669850 has been marked as a duplicate of this bug. ***

Comment 3 Mikhail Kulemin 2011-01-21 13:19:56 UTC
Created attachment 474624 [details]
Patch for qemu package to fixing help messsages

Comment 4 Pavel Zhukov 2011-01-21 14:58:40 UTC
koji build with michail's patch
mock: ok

minor changes (only list of vga has been corrected, not more)
SRPM: http://landgraf.fedorapeople.org/qemu-0.13.0-2.fc14.src.rpm

please push it to updates

Comment 5 Julian Sikorski 2011-01-29 11:39:02 UTC
Created attachment 475940 [details]
failing domain

Sorry for posting this twice but I clicked the wrong bug initially. With this package and libvirt from virt-preview repo the domain (attached) won't start:
# LANG=C virsh start windows
error: Failed to start domain windows
error: wewnętrzny błąd Zakończono proces podczas odczytywania wyjścia dziennika
console: qemu-kvm: -spice port=5900,addr=,disable-ticketing: Invalid
parameter 'addr'
parse error: port=5900,addr=,disable-ticketing

Comment 6 Mikhail Kulemin 2011-01-29 13:01:20 UTC
This bug explain problem:

With patched libvirt spice works well.

Comment 7 Pavel Zhukov 2011-01-29 13:02:37 UTC
"addr" parse error is a bug of libvirt, not qemu. 
qemu "spice" section hasn't "addr" option.

Comment 8 Mikhail Kulemin 2011-01-29 13:11:43 UTC
It should be noted that in the last development version of qemu we can find "addr" option in spice parameters. So, I hope that in next update of qemu (In virt preview repo) that issue will be fixed.

Comment 9 Julian Sikorski 2011-01-31 00:04:10 UTC
Created attachment 476098 [details]
final patch version

This patch disables the failing test so that the srpm can actually be rebuilt.

Comment 10 Julian Sikorski 2011-01-31 00:07:24 UTC
Comment on attachment 476098 [details]
final patch version

Oops wrong bug...

Comment 11 Julian Sikorski 2011-02-11 23:21:13 UTC
Looks like the problem is fixed in version 2:0.14.0-0.1.201102107aa8c46.fc14 which recently appeared in the virt-preview repo.

Comment 12 Pavel Zhukov 2011-02-24 09:28:35 UTC
The problem fixed in rawhide.

Note You need to log in before you can comment on or make changes to this bug.