On 11/14/2014 04:25 PM, Gerd Hoffmann wrote:
On Fr, 2014-11-14 at 16:13 +0100, Pavel Hrdina wrote:
> On 11/14/2014 04:06 PM, Gerd Hoffmann wrote:
>> On Fr, 2014-11-14 at 14:54 +0100, Pavel Hrdina wrote:
>>> + if (video->vram % 1024) {
>>> + virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
>>> + "%s", _("value for 'vram'
must be multiple
>>> of 1024"));
>>> + goto error;
>>> + }
>>
>> pci bar size must be a power of two. Especially the 9216 default value
>> libvirt fills in there is *not* valid. This probably needs some extra
>> care to handle so existing guest xml files don't blow up things.
>>
>> cheers,
>> Gerd
>
> Default value for QXL is 64MB in libvirt, the 9MB is for other video
> devices. But we can probably add a special check for QXL device to
> ensure that ram or vram size is power of 2.
For stdvga and vmware vga it must be a power of two too.
cheers,
Gerd
--
libvir-list mailing list
libvir-list(a)redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
We cannot just start enforcing that the vram attribute should be power
of 2. We can ether silently modify the value from user to power of 2 or
let QEMU round the value up to nearest power of 2 value and just add
a note about this to documentation.
Pavel