[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] cirrus_vga: adding sanity check for vram size
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH] cirrus_vga: adding sanity check for vram size |
Date: |
Mon, 12 May 2014 11:03:52 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.5.0 |
On 05/09/2014 05:18 AM, Dr. David Alan Gilbert wrote:
> * address@hidden (address@hidden) wrote:
>> From: Gonglei <address@hidden>
>>
>> when configure a invalid vram size for cirrus card, such as less
>> 2 MB, which will crash qemu. Follow the real hardware, the cirrus
>> card has 4 MB video memory. Also for backward compatibility, accept
>> 8 MB and 16 MB vram size.
>
> virt-manager/libvirt seems to default to 9 MByte of Vram for cirrus,
> so this would break a lot of setups.
I think it was virt-manager that made the mistake, but it is indeed a
historical wart that we are now stuck with (unless you argue that taking
the user's request and silently rounding up to the next power of 2 will
not be a guest-visible change).
>
> Looking at datasheets on the web seems to say the chips actually went
> down to 1 MB or less.
>
> I think before doing this change, it would be good to understand where
> the weird 9MB in libvirt/virt-manager came from, and what the limits of
> the emulator/drivers are.
>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-devel] [PATCH] cirrus_vga: adding sanity check for vram size [checkpatch false positive?], (continued)
Re: [Qemu-devel] [PATCH] cirrus_vga: adding sanity check for vram size, Dr. David Alan Gilbert, 2014/05/09