Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1334557 - libxl: default video errors with 'videoram must be at least 4MB for CIRRUS'
Summary: libxl: default video errors with 'videoram must be at least 4MB for CIRRUS'
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: libvirt
Version: 24
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Libvirt Maintainers
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-05-10 02:13 UTC by Konrad Rzeszutek Wilk
Modified: 2016-05-23 20:02 UTC (History)
13 users (show)

Fixed In Version: libvirt-1.3.3.1-2.fc24
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-05-23 20:02:31 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Output from: virt-install -v -n F24-64 --memory 1024 --cdrom /root/Fedora-Server-dvd-x86_64-24_Alpha-1.7.iso --disk /var/lib/libvirt/images/f24-i686.img --network bridge=switch,mac=00:0f:4b:01:00:87 (5.88 KB, text/plain)
2016-05-10 17:05 UTC, Konrad Rzeszutek Wilk
no flags Details
stderr output (5.88 KB, text/plain)
2016-05-16 19:28 UTC, Konrad Rzeszutek Wilk
no flags Details

Description Konrad Rzeszutek Wilk 2016-05-10 02:13:20 UTC
Description of problem:

When 
Traceback (most recent call last):
  File "/usr/share/virt-manager/virtManager/asyncjob.py", line 90, in cb_wrapper
    callback(asyncjob, *args, **kwargs)
  File "/usr/share/virt-manager/virtManager/create.py", line 2277, in _do_async_install
    guest.start_install(meter=meter)
  File "/usr/share/virt-manager/virtinst/guest.py", line 501, in start_install
    noboot)
  File "/usr/share/virt-manager/virtinst/guest.py", line 416, in _create_guest
    dom = self.conn.createLinux(start_xml or final_xml, 0)
  File "/usr/lib64/python2.7/site-packages/libvirt.py", line 3701, in createLinux
    if ret is None:raise libvirtError('virDomainCreateLinux() failed', conn=self)
libvirtError: unsupported configuration: videoram must be at least 4MB for CIRRUS

NOTE: This is after I used the work-around in Bug 1334554


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

[root@tst032 bin]# rpm -qa | grep virt-manage
virt-manager-common-1.3.2-3.fc24.noarch
virt-manager-1.3.2-3.fc24.noarch

How reproducible:

100% 
Steps to Reproduce:
1. Install Fedora 24
2. Install Xen : dnf install xen
3. Reboot in Xen
4. Run virt-manager and try to install F24 guest.
5. Use work-around from  Bug 1334554


Actual results:

See above - the Python  error from libvirt.

Expected results:

I expected an VNC window of the guest.

Additional info:

Comment 1 Konrad Rzeszutek Wilk 2016-05-10 02:32:37 UTC
This is also a problem with virt-install:

[root@tst032 F19-64]# virt-install   -v -n F24-64 --memory 1024 --cdrom /root/Fedora-Server-dvd-x86_64-24_Alpha-1.7.iso --disk /var/lib/libvirt/images/f24-i686.img --network bridge=switch,mac=00:0f:4b:01:00:87

Starting install...
ERROR    unsupported configuration: videoram must be at least 4MB for CIRRUS
Domain installation does not appear to have been successful.
If it was, you can restart your domain by running:
  virsh --connect xen:/// start F24-64
otherwise, please restart your installation.
[root@tst032 F19-64]#   virsh --connect xen:/// start F24-64
error: failed to get domain 'F24-64'
error: Domain not found

Comment 2 Cole Robinson 2016-05-10 15:40:33 UTC
Thanks for the report, please provide virt-install --debug when reproducing

Comment 3 Konrad Rzeszutek Wilk 2016-05-10 17:05:54 UTC
Created attachment 1155800 [details]
Output from: virt-install   -v -n F24-64 --memory 1024 --cdrom /root/Fedora-Server-dvd-x86_64-24_Alpha-1.7.iso --disk /var/lib/libvirt/images/f24-i686.img --network bridge=switch,mac=00:0f:4b:01:00:87

Comment 4 Cole Robinson 2016-05-10 18:01:31 UTC
> libvirtError: unsupported configuration: videoram must be at least 4MB for
> CIRRUS

That error comes from src/libxl/libxl_conf.c:libxlMakeVideo . virt-install isn't adding an explicit video ddevice, so the default video dev is being added, and I suspect some part of the code isn't setting video ram correctly, or libxl should be updated to expect ram=0 or whatever the value is by default.

Unfortunately the libvirt unit test suite doesn't seem to touch this code, I tried editing one of the tests in xlconfigdata/ but it didn't trigger

Comment 5 Cole Robinson 2016-05-10 18:07:48 UTC
(In reply to Cole Robinson from comment #4)
> > libvirtError: unsupported configuration: videoram must be at least 4MB for
> > CIRRUS
> 
> That error comes from src/libxl/libxl_conf.c:libxlMakeVideo . virt-install
> isn't adding an explicit video ddevice, so the default video dev is being
> added, and I suspect some part of the code isn't setting video ram
> correctly, or libxl should be updated to expect ram=0 or whatever the value
> is by default.
> 
> Unfortunately the libvirt unit test suite doesn't seem to touch this code, I
> tried editing one of the tests in xlconfigdata/ but it didn't trigger

Comment 6 Jim Fehlig 2016-05-11 05:06:00 UTC
This bug was introduced by libvirt commit 6879be48. I've sent a patch upstream that fixes the issue in my testing

https://www.redhat.com/archives/libvir-list/2016-May/msg00700.html

Comment 7 Cole Robinson 2016-05-16 13:33:27 UTC
In libvirt 1.3.3-maint now

Comment 8 Konrad Rzeszutek Wilk 2016-05-16 14:19:48 UTC
Do you have a test-build I could try out?

Thanks!

Comment 9 Cole Robinson 2016-05-16 18:22:17 UTC
I kicked off a scratch build here;

http://koji.fedoraproject.org/koji/taskinfo?taskID=14108914

Comment 10 Konrad Rzeszutek Wilk 2016-05-16 19:24:19 UTC
OK, I downloaded it and installed it.

Now I get:

nable to complete install: 'unsupported configuration: videoram must be at least 8MB for CIRRUS'

Traceback (most recent call last):
  File "/usr/share/virt-manager/virtManager/asyncjob.py", line 90, in cb_wrapper
    callback(asyncjob, *args, **kwargs)
  File "/usr/share/virt-manager/virtManager/create.py", line 2277, in _do_async_install
    guest.start_install(meter=meter)
  File "/usr/share/virt-manager/virtinst/guest.py", line 501, in start_install
    noboot)
  File "/usr/share/virt-manager/virtinst/guest.py", line 416, in _create_guest
    dom = self.conn.createLinux(start_xml or final_xml, 0)
  File "/usr/lib64/python2.7/site-packages/libvirt.py", line 3701, in createLinux
    if ret is None:raise libvirtError('virDomainCreateLinux() failed', conn=self)
libvirtError: unsupported configuration: videoram must be at least 8MB for CIRRUS



Umm?

Comment 11 Konrad Rzeszutek Wilk 2016-05-16 19:28:40 UTC
Created attachment 1158086 [details]
stderr output

When running:
 virt-install --debug   -v -n F24-64 --memory 1024 --cdrom /root/Fedora-Server-dvd-x86_64-24_Alpha-1.7.iso --disk /var/lib/libvirt/images/f24-i686.img --network bridge=switch,mac=00:0f:4b:01:00:87 2> ~/1334557_stdout.txt

This is with:
[root@tst032 ~]# rpm -qi libvirt
Name        : libvirt
Version     : 1.3.3.1
Release     : 1.fc24
Architecture: x86_64
Install Date: Mon 16 May 2016 03:16:05 PM EDT
Group       : Development/Libraries
Size        : 0
License     : LGPLv2+
Signature   : (none)
Source RPM  : libvirt-1.3.3.1-1.fc24.src.rpm
Build Date  : Mon 16 May 2016 10:52:29 AM EDT
Build Host  : buildvm-07.phx2.fedoraproject.org
Relocations : (not relocatable)
Packager    : Fedora Project
Vendor      : Fedora Project
URL         : http://libvirt.org/
Summary     : Library providing a simple virtualization API
Description :
Libvirt is a C toolkit to interact with the virtualization capabilities
of recent versions of Linux (and other OSes). The main package includes
the libvirtd server exporting the virtualization support.
[root@tst032 ~]# 

which is the libvirt RPMs built from http://koji.fedoraproject.org/koji/taskinfo?taskID=14108918 scratch build.

Comment 12 Jim Fehlig 2016-05-16 19:59:53 UTC
I suppose some patches are missing. On top of 1.3.4 I have e4d131b8, 3e428670, 538012c8, 96b21fb0, and b90c4b5f to fix this bug. Note that 96b21fb0 and b90c4b5f are only adjusting tests to allow 'make check' to pass.

Comment 13 Cole Robinson 2016-05-16 20:01:54 UTC
I pulled in all of those. Konrad can you double check that libvirtd was restarted? I could have botched the build though

Comment 14 Konrad Rzeszutek Wilk 2016-05-16 20:54:24 UTC
I rebooted the box after I did the install. And just to make sure there were no odd things around I first did 'dnf remove libvirt' and then did 'dnf install libvirt' (with adjusting my local repo to be picked over the updates which had the same version).

Pls keep in mind that the error is different - initially it was 4MB, now it is 8MB so there is a fix in there.

Comment 15 Konrad Rzeszutek Wilk 2016-05-16 20:58:09 UTC
Just rebooted the box once more just in case. Same error.

# virt-install   -v -n F24-64 --memory 1024 --cdrom /root/Fedora-Server-dvd-x86_64-24_Alpha-1.7.iso --disk /var/lib/libvirt/images/f24-i686.img --network bridge=switch,mac=00:0f:4b:01:00:87

Starting install...
ERROR    unsupported configuration: videoram must be at least 8MB for CIRRUS
Domain installation does not appear to have been successful.
If it was, you can restart your domain by running:
  virsh --connect xen:/// start F24-64
otherwise, please restart your installation.
[root@tst032 ~]# May 16 16:57:24 tst032.dumpdata.com libvirtd[1668]: 1672: error : libxlMakeVideo:2022 : unsupported configuration: videoram must be at least 8MB for CIRRUS

Comment 16 Cole Robinson 2016-05-19 13:36:30 UTC
I just confirmed that the test build fixes the issues for me. What RPMs did you update? Make sure libvirt-daemon and libvirt-daemon-driver-libxl were included. You can use 'koji download-task 14108918' to download all the RPMs

Comment 17 Fedora Update System 2016-05-20 21:47:17 UTC
libvirt-1.3.3.1-2.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-a87b87586a

Comment 18 Fedora Update System 2016-05-22 03:55:53 UTC
libvirt-1.3.3.1-2.fc24 has been pushed to the Fedora 24 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-a87b87586a

Comment 19 Konrad Rzeszutek Wilk 2016-05-23 15:51:50 UTC
I am not sure why it did not work for me before.

But I ended up removing all libvirt and then re-intalling (this this using the new repo). And it all worked!

Thank you!

Comment 20 Fedora Update System 2016-05-23 20:02:21 UTC
libvirt-1.3.3.1-2.fc24 has been pushed to the Fedora 24 stable repository. If problems still persist, please make note of it in this bug report.


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