On Tue, Oct 20, 2015 at 03:51:05PM -0400, Paolo Bonzini wrote: > Should this be "select" or "depends on"? Not a blocker, can always be fixed > in 4.4. > Hmm, I don't know actually. I trusted Arnd to make the right call and given Marc's ack as well, I didn't pay too much attention to that particular detail.
Arnd, any comments? Thanks, -Christoffer > > > -----Original Message----- > From: Christoffer Dall [[email protected]] > Received: martedì, 20 ott 2015, 18:18 > To: Paolo Bonzini [[email protected]]; [email protected], > [email protected], [email protected] > CC: Marc Zyngier [[email protected]]; Arnd Bergmann [[email protected]]; > Christoffer Dall [[email protected]] > Subject: [GIT PULL 3/6] KVM: arm: use GIC support unconditionally > > From: Arnd Bergmann <[email protected]> > > The vgic code on ARM is built for all configurations that enable KVM, > but the parent_data field that it references is only present when > CONFIG_IRQ_DOMAIN_HIERARCHY is set: > > virt/kvm/arm/vgic.c: In function 'kvm_vgic_map_phys_irq': > virt/kvm/arm/vgic.c:1781:13: error: 'struct irq_data' has no member named > 'parent_data' > > This flag is implied by the GIC driver, and indeed the VGIC code only > makes sense if a GIC is present. This changes the CONFIG_KVM symbol > to always select GIC, which avoids the issue. > > Fixes: 662d9715840 ("arm/arm64: KVM: Kill CONFIG_KVM_ARM_{VGIC,TIMER}") > Signed-off-by: Arnd Bergmann <[email protected]> > Acked-by: Marc Zyngier <[email protected]> > Signed-off-by: Christoffer Dall <[email protected]> > --- > arch/arm/kvm/Kconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/kvm/Kconfig b/arch/arm/kvm/Kconfig > index 210ecca..356970f 100644 > --- a/arch/arm/kvm/Kconfig > +++ b/arch/arm/kvm/Kconfig > @@ -21,6 +21,7 @@ config KVM > depends on MMU && OF > select PREEMPT_NOTIFIERS > select ANON_INODES > + select ARM_GIC > select HAVE_KVM_CPU_RELAX_INTERCEPT > select HAVE_KVM_ARCH_TLB_FLUSH_ALL > select KVM_MMIO > -- > 2.1.2.330.g565301e.dirty > -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
