[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] xics-kvm: fix reset function
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH] xics-kvm: fix reset function |
Date: |
Wed, 12 Mar 2014 15:42:54 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.3.0 |
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?
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);
>> }
>>
>>
>
>
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg