Bug 807413 - [PATCH] switch default display device to QXL
Summary: [PATCH] switch default display device to QXL
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: qemu
Version: 17
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Fedora Virtualization Maintainers
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-03-27 17:43 UTC by Bill Nottingham
Modified: 2014-03-17 03:30 UTC (History)
13 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-04-24 14:49:30 UTC
Type: ---


Attachments (Terms of Use)

Description Bill Nottingham 2012-03-27 17:43:05 UTC
Description of problem:

Because cirrus causes kittens to kill kittens in a horrible kitten-killing arena.
(Others can elucidate on cirrus' lameness if so desired.)

Version-Release number of selected component (if applicable):

qemu-1.0-6.fc17

How reproducible:

100%

Steps to Reproduce:
1. Look at qemu defaults
2. See 'cirrus'
  
Actual results:

Dead kittens.

Expected results:

Spice.

Additional info:

diff -up qemu-kvm-1.0/vl.c.foo qemu-kvm-1.0/vl.c
--- qemu-kvm-1.0/vl.c.foo	2012-03-27 13:38:39.721397415 -0400
+++ qemu-kvm-1.0/vl.c	2012-03-27 13:38:57.708397915 -0400
@@ -3221,7 +3221,7 @@ int main(int argc, char **argv, char **e
             add_device_config(DEV_VIRTCON, "vc:80Cx24C");
     }
     if (default_vga)
-        vga_interface_type = VGA_CIRRUS;
+        vga_interface_type = VGA_QXL;
 
     socket_init();

Comment 1 Daniel Berrangé 2012-03-27 18:01:23 UTC
I don't think we can do this because some OS (eg Fedora 16) have QXL drivers which are pretty fubar and thus should not be used.

We could at least switch to the STDVGA driver though, which should be an improvement.

Any use of QXL should be controlled by libosinfo (or equivalent per-OS config databases in virt-install, etc)

Comment 2 Bill Nottingham 2012-03-27 18:09:07 UTC
OSes can fall back to vesa (and ours should be fixed).

Comment 3 Daniel Berrangé 2012-03-27 18:41:41 UTC
We can't retroactively fix existing released media. This kind of issue is exactly why we maintain databases of per-OS default hardware drivers.

Comment 4 Cole Robinson 2012-04-24 14:49:30 UTC
Yeah, closing this WONTFIX. Changing the default at the qemu level just isn't an option unless the change was upstream first (and it's highly unlikely upstream would make this change).

As danpb mentions, the way forward with this is getting VM building tools to use libosinfo.


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