On 08/01/2011 06:27 PM, Marcelo Tosatti wrote:
As noted by the original commit. Probably kept in qemu-kvm due to a mismerge. commit 7d0adcd6d57149d4e9ec66f3a1c171df2bea2775 Author: aurel32<aurel32@c046a42c-6fe2-441c-8c8c-71466251a162> Date: Fri Oct 24 13:12:42 2008 +0000 hw/pc: don't register the memory hole as unassigned twice Since revision 5228, we don't register the memory hole (0xa00000 to 0xfffff) anymore. As a consequence, we don't need to register it again as unassigned. Signed-off-by: Aurelien Jarno<aurelien@xxxxxxxxxxx> Signed-off-by: Marcelo Tosatti<mtosatti@xxxxxxxxxx> Index: qemu-kvm/hw/pc.c =================================================================== --- qemu-kvm.orig/hw/pc.c +++ qemu-kvm/hw/pc.c @@ -1013,8 +1013,6 @@ void pc_memory_init(const char *kernel_f isa_bios_size = bios_size; if (isa_bios_size> (128 * 1024)) isa_bios_size = 128 * 1024; - cpu_register_physical_memory(0xd0000, (192 * 1024) - isa_bios_size, - IO_MEM_UNASSIGNED); cpu_register_physical_memory(0x100000 - isa_bios_size, isa_bios_size, (bios_offset + bios_size - isa_bios_size) | IO_MEM_ROM);
I noticed the same thing on the large upstream merge - this code is now gone.
-- error compiling committee.c: too many arguments to function -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html