[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [Qemu-trivial] [PATCH 2/2] KVM: PPC: Add dummy kvm_arch_i
From: |
Alexey Kardashevskiy |
Subject: |
Re: [Qemu-ppc] [Qemu-trivial] [PATCH 2/2] KVM: PPC: Add dummy kvm_arch_init_irq_routing() |
Date: |
Sat, 08 Jun 2013 18:33:20 +1000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130514 Thunderbird/17.0.6 |
On 06/08/2013 05:24 PM, Michael Tokarev wrote:
> 06.06.2013 07:59, Alexey Kardashevskiy wrote:
>> From: Scott Wood <address@hidden>
>>
>> The common KVM code insists on calling kvm_arch_init_irq_routing()
>> as soon as it sees kernel header support for it (regardless of whether
>> QEMU supports it). Provide a dummy function to satisfy this.
>>
>> Unlike x86, PPC does not have one default irqchip, so there's no common
>> code that we'd stick here. Even if you ignore the routes themselves,
>> which even on x86 are not set up in this function, the initial XICS
>> kernel implementation will not support IRQ routing, so it's best to
>> leave even the general feature flags up to the specific irqchip code.
>
> As Scott Wood already pointed out, this should come in before the
> actual header update, which is no problem. We'll have to deal
> with a new warning (-Wmissing-prototypes) which can be dealt with
> by wrapping this function into #ifdef KVM_CAP_IRQ_ROUTING .. #endif
> (I can add this).
Ah. I'll repost it early next week.
>
> But how about other architectures? Before, this function were only
> defined for x86, now it is defined for two arches - x86 and ppc.
> Aren't other arches need this as well?
Good point, will check (and fix if needed). Thanks!
--
Alexey