Re: [PATCH 2/3] Fix CPU hotplug

[Avi Kivity <avi@redhat.com>]

Gleb Natapov wrote:
>>>   struct pci_status {
>>> @@ -603,10 +603,12 @@ static uint32_t gpe_readb(void *opaque, uint32_t addr)
>>>      struct gpe_regs *g = opaque;
>>>      switch (addr) {
>>>          case PROC_BASE:
>>> -            val = g->up;
>>> -            break;
>>>          case PROC_BASE + 1:
>>> -            val = g->down;
>>> +            val = gpe_read_val(g->cpus_sts, addr);
>>> +            break;
>>> +        case PROC_BASE + 2:
>>> +        case PROC_BASE + 3:
>>> +            val = gpe_read_val(g->bios_cpus_sts, addr);
>>>              break;
>>>   
>>>       
>> Why can't the bios maintain bios_cpu_sts in RAM?
>>
>>     
> It can, just need to find a place for it. Currently our AML does not use
> RAM at all.

OperationRegion(..., SystemMemory, ...) should work.  It's better to 
avoid introducing unnecessary virtual hardware.

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