Re: [GIT PULL 3/6] KVM: arm: use GIC support unconditionally

2015-10-21 Thread Arnd Bergmann
On Tuesday 20 October 2015 15:51:05 Paolo Bonzini wrote: > Should this be "select" or "depends on"? Not a blocker, can always be fixed > in 4.4. We have lots of 'select ARM_GIC' in the tree for platforms that use one, using 'depends on' will limit KVM support to being available only if at least

Re: [GIT PULL 3/6] KVM: arm: use GIC support unconditionally

2015-10-21 Thread Christoffer Dall
On Wed, Oct 21, 2015 at 03:45:20PM +0200, Arnd Bergmann wrote: > On Tuesday 20 October 2015 15:51:05 Paolo Bonzini wrote: > > Should this be "select" or "depends on"? Not a blocker, can always be fixed > > in 4.4. > > We have lots of 'select ARM_GIC' in the tree for platforms that use one, using

RE: [GIT PULL 3/6] KVM: arm: use GIC support unconditionally

2015-10-21 Thread Pavel Fedin
Hello! > The only platform I can think of that uses ARMv7ve without actually having > a GIC is BCM2836 (Raspberry Pi 2). Can we actually run KVM on a platform > like that? We can, with two limitations: 1. GIC has to be emulated in software. I have recently fixed support for this. The only

Re: [GIT PULL 3/6] KVM: arm: use GIC support unconditionally

2015-10-21 Thread Arnd Bergmann
On Wednesday 21 October 2015 15:58:44 Christoffer Dall wrote: > On Wed, Oct 21, 2015 at 03:45:20PM +0200, Arnd Bergmann wrote: > > On Tuesday 20 October 2015 15:51:05 Paolo Bonzini wrote: > > > Should this be "select" or "depends on"? Not a blocker, can always be > > > fixed in 4.4. > > > > We

Re: [GIT PULL 3/6] KVM: arm: use GIC support unconditionally

2015-10-21 Thread Christoffer Dall
mbia.edu, > kvm@vger.kernel.org, linux-arm-ker...@lists.infradead.org > CC: Marc Zyngier [marc.zyng...@arm.com]; Arnd Bergmann [a...@arndb.de]; > Christoffer Dall [christoffer.d...@linaro.org] > Subject: [GIT PULL 3/6] KVM: arm: use GIC support unconditionally > > From: Arnd Be

[GIT PULL 3/6] KVM: arm: use GIC support unconditionally

2015-10-20 Thread Christoffer Dall
From: Arnd Bergmann 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: