qemu-trivial
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-trivial] [PATCH 2/2] KVM: PPC: Add dummy kvm_arch_init_irq_rou


From: Michael Tokarev
Subject: Re: [Qemu-trivial] [PATCH 2/2] KVM: PPC: Add dummy kvm_arch_init_irq_routing()
Date: Sat, 08 Jun 2013 11:24:21 +0400
User-agent: Mozilla/5.0 (X11; Linux i686 on x86_64; rv:17.0) Gecko/17.0 Icedove/17.0

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).

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?

Thanks,

/mjt




reply via email to

[Prev in Thread] Current Thread [Next in Thread]