Re: [PATCH] kvm: bios: Put AP boot up code to 0x1000

[Avi Kivity <avi@qumranet.com>]

Yang, Sheng wrote:
> On Thursday 31 July 2008 14:41:04 Avi Kivity wrote:
>   
>> Yang, Sheng wrote:
>>     
>>> On Thursday 31 July 2008 10:43:20 Yang, Sheng wrote:
>>>       
>>>> From ba05b3c821400127074f65ee4d172fabbe3524cd Mon Sep 17
>>>> 00:00:00 2001 From: Sheng Yang <sheng.yang@intel.com>
>>>> Date: Thu, 31 Jul 2008 10:39:24 +0800
>>>> Subject: [PATCH] kvm: bios: Put AP boot up code to 0x1000
>>>>
>>>> Rather than 0x10000, which can be overrided by userspace program
>>>> like grub.
>>>>         
>> How about 0xex000?  And avoid copying the bios to low memory?
>>
>> Gleb tried it and it worked.  We'd have to change qemu to mark the
>> memory as RAM, not ROM, but that's what real hardware does as well
>> (shadowing the ROM into RAM).
>>     
>
> I tried 0xea000, and it's fine. But avoid copying is better. 
>
> But I don't understand the RAM/ROM issue in qemu here...
>   

IIRC the rombios32.c writes to the memory it is in, so it expects RAM, 
not ROM.

kvm doesn't support ROM, so it would work.  Qemu doesn't, so it would fail.

> And do you mean you have patch on hand now? :)
>
>   

No, but maybe Gleb does.

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