Re: KVM-58 -- existing VM just hangs with black screen

Avi Kivity
2008-01-03T10:22:57Z

[-- Attachment #1: Type: text/plain, Size: 629 bytes --]

Jeremy Katz wrote:
> On Wed, 2008-01-02 at 19:13 +0200, Avi Kivity wrote:
>
>> Can you try the option '-no-kvm' both with kvm-57 and kvm-58?
>>
>
> -no-kvm works. I can hit this just booting a live CD both on a 32bit
> and a 64bit host. It looks like compatibility with old kernels is
> broken (the kvm package only contains the userspace bits, the kernel
> bits come from the kernel which is currently 2.6.24-rc6). Building an
> updated kernel module makes things start to work on my laptop
>

Right. I fixed this for kvm-60, see attached patch.

--
error compiling committee.c: too many arguments to function

[-- Attachment #2: fix-old-kernels.patch --]
[-- Type: text/x-patch, Size: 1126 bytes --]

commit 6b8bb99a9cde386d72b4b7c22b92f4bdec333dab
Author: Avi Kivity <avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
Date: Thu Jan 3 12:20:21 2008 +0200

kvm: qemu: restore IO_MEM_ROM mark to bios when registering the memory with kvm

We removed the IO_MEM_ROM mark for qemu, since the tpr optimization needs to
update the bios. However, kvm_cpu_register_physical_memory() uses the mark
for other purposes, so keep it. The removal broke bootstrap on older kernels.

Signed-off-by: Avi Kivity <avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org>

diff --git a/qemu/hw/pc.c b/qemu/hw/pc.c
index 5a1b7d4..179585f 100644
--- a/qemu/hw/pc.c
+++ b/qemu/hw/pc.c
@@ -865,8 +865,7 @@ static void pc_init1(ram_addr_t ram_size, int vga_ram_size,
if (kvm_allowed)
kvm_cpu_register_physical_memory(0x100000 - isa_bios_size,
isa_bios_size,
- (bios_offset + bios_size - isa_bios_size)
- /* | IO_MEM_ROM */);
+ (bios_offset + bios_size - isa_bios_size) | IO_MEM_ROM);
#endif

[-- Attachment #3: Type: text/plain, Size: 228 bytes --]

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

[-- Attachment #4: Type: text/plain, Size: 186 bytes --]