]> xenbits.xensource.com Git - libvirt.git/commit
Call per-device post-parse callback even on implicit video
authorJán Tomko <jtomko@redhat.com>
Wed, 11 May 2016 10:13:51 +0000 (12:13 +0200)
committerJán Tomko <jtomko@redhat.com>
Thu, 12 May 2016 06:22:08 +0000 (08:22 +0200)
commit3e4286703273b06a21ae07f3e76a66f9661199dc
treea985f1f9ee86865b4aacc3ae4b0c197a6e127722
parente4d131b8cb12679814b6fda159281f472b615524
Call per-device post-parse callback even on implicit video

Commit 6879be48 moved adding of an implicit video device after XML
parsing. As a result, libxlDomainDeviceDefPostParse() is no longer
called to set the default vram when adding an implicit device.
Commit 6879be48 assumes virDomainVideoDefaultRAM() will set the
default vram, but it returns 0 if the domain virtType is
VIR_DOMAIN_VIRT_XEN. Attempting to start an HVM domain with vram=0
results in

error: unsupported configuration: videoram must be at least 4MB for CIRRUS

The default vram setting for Xen HVM domains depends on the device
model used (qemu-xen vs qemu-traditional), hence setting the
default is deferred to libxlDomainDeviceDefPostParse().

Call the device post-parse callback even for implicit video,
to fill out the default vram even for VIR_DOMAIN_VIRT_XEN.

https://bugzilla.redhat.com/show_bug.cgi?id=1334557
Most-of-commit-message-by: Jim Fehlig <jfehlig@suse.com>
src/conf/domain_conf.c