Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.
RHEL Engineering is moving the tracking of its product development work on RHEL 6 through RHEL 9 to Red Hat Jira (issues.redhat.com). If you're a Red Hat customer, please continue to file support cases via the Red Hat customer portal. If you're not, please head to the "RHEL project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs in the statuses "NEW", "ASSIGNED", and "POST" are being migrated throughout September 2023. Bugs of Red Hat partners with an assigned Engineering Partner Manager (EPM) are migrated in late September as per pre-agreed dates. Bugs against components "kernel", "kernel-rt", and "kpatch" are only migrated if still in "NEW" or "ASSIGNED". If you cannot log in to RH Jira, please consult article #7032570. That failing, please send an e-mail to the RH Jira admins at rh-issues@redhat.com to troubleshoot your issue as a user management inquiry. The email creates a ServiceNow ticket with Red Hat. Individual Bugzilla bugs that are migrated will be moved to status "CLOSED", resolution "MIGRATED", and set with "MigratedToJIRA" in "Keywords". The link to the successor Jira issue will be found under "Links", have a little "two-footprint" icon next to it, and direct you to the "RHEL project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/RHEL-XXXX", where "X" is a digit). This same link will be available in a blue banner at the top of the page informing you that that bug has been migrated.

Bug 686236

Summary: [vdsm] Passes wrong argument to qemu while using spice: -global qxl-vga.vram_size=67108864
Product: Red Hat Enterprise Linux 6 Reporter: Jakub Libosvar <jlibosva>
Component: vdsmAssignee: Dan Kenigsberg <dkenigsb>
Status: CLOSED DUPLICATE QA Contact: yeylon <yeylon>
Severity: urgent Docs Contact:
Priority: unspecified    
Version: 6.1CC: abaron, bazulay, dallan, danken, iheim, srevivo, ykaul
Target Milestone: rcKeywords: TestBlocker
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-03-15 17:20:29 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
VM log & vdsm log none

Description Jakub Libosvar 2011-03-15 14:15:39 UTC
Created attachment 484779 [details]
VM log & vdsm log

Description of problem:
When using spice console, qemu process is run with parameter -global qxl-vga.vram_size=67108864 which is considered as invalid option and due to this parameter VM won't start

Version-Release number of selected component (if applicable):
qemu-kvm-0.12.1.2-2.150.el6.x86_64
vdsm-4.9-53.el6.x86_64
libvirt-0.8.7-11.el6.x86_64

How reproducible:
Always

Steps to Reproduce:
1. Create a new VM (in my case RHEL 6 x64, with virtio disk)
2. Set console as Spice
3. Run VM
  
Actual results:
run fails due to bad qemu parameters

Expected results:
VM is running

Additional info:
vnc works

critical parameters qemu is run with: -vga qxl -global qxl-vga.vram_size=67108864

Parameters passed from vdsm:
                        <model heads="1" type="qxl" vram="65536"/>
                </video>
                <channel type="spicevmc">
                        <target name="com.redhat.spice.0" type="virtio"/>
                </channel>
                <graphics autoport="yes" keymap="en-us" listen="0" passwd="*****" passwdValidTo="1970-01-01T00:00:01" port="-1" tlsPort="-1" type="spice">
                        <channel mode="secure" name="main"/>
                        <channel mode="secure" name="inputs"/>
                </graphics>

VM log and vdsm log is attached

Comment 2 Yaniv Kaul 2011-03-15 14:35:52 UTC
Looks like VDSM should ask libvirt for vram=64 (as libvirt translates it from megs to bytes).

Comment 4 Dan Kenigsberg 2011-03-15 14:58:08 UTC
Lovely. Is this an intended change in libvirt-0.8.7-11?

Comment 6 Dave Allan 2011-03-15 16:57:10 UTC
Dan, I don't believe that's intentional.  We do not change the units of the XML fields.  Assuming that's the case, it will be fixed along with https://bugzilla.redhat.com/show_bug.cgi?id=673578#c18 in -12.

Comment 7 Dan Kenigsberg 2011-03-15 17:20:29 UTC
(In reply to comment #6)
> Dan, I don't believe that's intentional.  We do not change the units of the XML
> fields.  Assuming that's the case, it will be fixed along with
> https://bugzilla.redhat.com/show_bug.cgi?id=673578#c18 in -12.

Cool. If vram= attribute is to be specified in KiB as it used to, there's nothing for vdsm to do here.

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

Comment 8 Dave Allan 2011-03-17 01:38:45 UTC
(In reply to comment #7)
> (In reply to comment #6)
> > Dan, I don't believe that's intentional.  We do not change the units of the XML
> > fields.  Assuming that's the case, it will be fixed along with
> > https://bugzilla.redhat.com/show_bug.cgi?id=673578#c18 in -12.
> 
> Cool. If vram= attribute is to be specified in KiB as it used to, there's
> nothing for vdsm to do here.

Actually, the field *is* in KiB, and this bug is precisely a dup of 673578; qemu isn't barfing on the value (67108864 / 1024 = 65536, which is what was specified in the XML) it's barfing on the mangling of the field name, which is verified in -12 now and you should be good to go.

Comment 10 Dave Allan 2011-03-17 19:05:32 UTC
vram is documented as being in KiB at:

http://libvirt.org/formatdomain.html#elementsVideo