Michael Tokarev <[email protected]> writes:
> Bjørn Mork wrote:
>> 
>> The modes were once added to KVMs local copy of vgabios:
>> http://git.kernel.org/?p=virt/kvm/qemu-kvm.git;a=commitdiff;h=ebfac597cf
>> but never made it upstream or into QEMU. 

I have to correct myself here.  The patch has made it upstream
http://cvs.savannah.gnu.org/viewvc/vgabios/vbetables-gen.c?root=vgabios&r1=1.4&r2=1.5
and is part of the latest release.

> I too noticed that the list of video modes available is quite a bit
> shorter than it used to be, but at that time I were dealing with
> another bug and didn't pay enough attention to that fact, and it
> were "successfully" forgotten.  But without your findings it'd be
> much more difficult to find and fix things.
>
> I'm adding the thing to the Debian qemu-kvm git repository now, to
> be fixed in the next release.

I'm afraid my primitive approach is not the proper way to fix it.
qemu-kvm should depend on vgabios and just install symlinks as qemu
does:

 bj...@nemi:~$ ls -la /usr/share/qemu/vgabios*
 lrwxrwxrwx 1 root root 22 2010-01-10 20:31 /usr/share/qemu/vgabios.bin -> 
../vgabios/vgabios.bin
 lrwxrwxrwx 1 root root 29 2010-01-10 20:31 /usr/share/qemu/vgabios-cirrus.bin 
-> ../vgabios/vgabios.cirrus.bin

Which is bug #489442.  Fixing that one will close both bugs.

BTW, I think I'm going to propose some variant of my debugging patch to
qemu upstream.  The "Could not open SDL display" is rather useless as a
fatal error message, given how easy it is to trigger this error with a
simple guest (mis)onfiguration.


Bjørn



--
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]

Reply via email to