Re: [Qemu-devel] [PATCH] xics-kvm: fix reset function

[Alexey Kardashevskiy]

On 03/13/2014 01:42 AM, Andreas Färber wrote:
> Am 04.03.2014 08:03, schrieb Alexey Kardashevskiy:
>> On 02/13/2014 12:08 PM, Alexey Kardashevskiy wrote:
>>> Currently interrupt priorities are set to 0 (highest) at the very
>>> beginning of the guest execution which is not correct and makes the guest
>>> produce random interrupt error messages such as:
>>> "Interrupt 0x1001 (real) is invalid, disabling it".
>>> This also prevents interrupt states from correct migration.
>>>
>>> This initializes priority to 0xFF as the emulated XICS does.
>>
>>
>> Ping? Pretty simple patch suggested by Paul Mackerras.
> 
> Is this one still needed? Can we document a Suggested-by?

Yes.

Suggested-by: Paul Mackerras <address@hidden>



> Andreas
> 
>>
>>
>>
>>> Signed-off-by: Alexey Kardashevskiy <address@hidden>
>>> ---
>>>  hw/intc/xics_kvm.c | 9 +++++++++
>>>  1 file changed, 9 insertions(+)
>>>
>>> diff --git a/hw/intc/xics_kvm.c b/hw/intc/xics_kvm.c
>>> index c203646..eaf94d9 100644
>>> --- a/hw/intc/xics_kvm.c
>>> +++ b/hw/intc/xics_kvm.c
>>> @@ -269,6 +269,15 @@ static void ics_kvm_set_irq(void *opaque, int srcno, 
>>> int val)
>>>  
>>>  static void ics_kvm_reset(DeviceState *dev)
>>>  {
>>> +    ICSState *ics = ICS(dev);
>>> +    int i;
>>> +
>>> +    memset(ics->irqs, 0, sizeof(ICSIRQState) * ics->nr_irqs);
>>> +    for (i = 0; i < ics->nr_irqs; i++) {
>>> +        ics->irqs[i].priority = 0xff;
>>> +        ics->irqs[i].saved_priority = 0xff;
>>> +    }
>>> +
>>>      ics_set_kvm_state(ICS(dev), 1);
>>>  }
>>>  
>>>
>>
>>
> 
> 


-- 
Alexey