From mboxrd@z Thu Jan 1 00:00:00 1970 From: tomasz.nowicki@linaro.org (Tomasz Nowicki) Date: Tue, 13 Jan 2015 17:25:13 +0100 Subject: [PATCH v6 00/20] KVM GICv3 emulation In-Reply-To: <1420804478-17354-1-git-send-email-andre.przywara@arm.com> References: <1420804478-17354-1-git-send-email-andre.przywara@arm.com> Message-ID: <54B546E9.30107@linaro.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Andre, Is there a git repo reference that I can fetch all those patches? http://www.linux-arm.org/git?p=linux-ap.git does not work for 'git-fetch'. Thanks in advance! Regards, Tomasz On 09.01.2015 12:54, Andre Przywara wrote: > This is version 6 of the GICv3 guest emulation series. > This series is now based on v3.19-rc3 plus Eric's explicit > VGIC_CTRL_INIT patches. > > The rebase on top of 3.19-rc and Eric's patches required some reworks > in 05/19 (introduce vm_ops), which rippled through to patches 08, 12, > 13 and 15. > > The Aarch32 version of the ICC_SGI_EL1 sysreg is now correctly > trapped, enabling 32-bit guests to use the GICv3 emulation (which > needs not-yet-upstream patches to enable GICv3 support on ARM). > > If a host's device tree has faulty properties for the GICV property > (unaligned base addresses or wrong region size), we just deny the > GICv2 emulation and still allow guests to use the GICv3 emulation. > > There is now a new patch 18/20, which introduces a > kvm_check_device_type() function to detect valid GIC emulation models > for guests. This is much saner and safer than the private GIC specific > way of achieving this functionality. > > Also I added the Acks from Marc (thanks for the review!), but dropped > the 05/19 Acked-by: due to the above mentioned changes. > > For a changelog summary see below, also each patch carries a > changelog. Patches 01-04, 06, 07, 09-11, 14, 16, 20 are unchanged. > > A git repo hosting all these patches lives in the kvm-gicv3/v6 branch > of: > http://www.linux-arm.org/git?p=linux-ap.git > git://linux-arm.org/linux-ap.git > ----- > > GICv3 is the ARM generic interrupt controller designed to overcome > some limits of the prevalent GICv2. Most notably it lifts the 8-CPU > limit. Though with Linux-3.17 Marc introduced support for hosts to > use a GICv3, the CPU limitation still applies to KVM guests, since > the current code emulates a GICv2 only. > Also, GICv2 backward compatibility being optional in GICv3, a number > of systems won't be able to run GICv2 guests. > > This patch series provides code to emulate a GICv3 distributor and > redistributor for any KVM guest. It requires a GICv3 in the host to > work. With those patches one can run guests efficiently on any GICv3 > host. It has the following features: > - Affinity routing (support for up to 255 VCPUs, more possible) > - System registers (as opposed to MMIO access) > - No ITS > - No priority support (as the GICv2 emulation) > - No save / restore support so far (will be added soon) > - Only Group1 interrupts support > > The first patches actually refactor the current VGIC code to make > room for a different VGIC model to be dropped in with Patch 15. > The remaining patches connect the new model to the kernel backend and > the userland facing code. > > The series goes on top of v3.19-rc3 plus Eric's init rework patches. > The necessary patches for kvmtool to enable the guest's GICv3 have > been posted here before [2], an updated version will follow soon. > > There was some testing on the fast model with some I/O and interrupt > affinity shuffling in a Linux guest with a varying number of VCPUs as > well as some testing on a Juno board (GICv2 only, to spot regressions). > > Please review and test. > I would be grateful for people to test for GICv2 regressions also > (so on a GICv2 host with current kvmtool/qemu), as there is quite > some refactoring on that front. > > Much of the code was inspired by MarcZ, also kudos to him for doing > the rather painful rebase on top of v3.17-rc1. > > Cheers, > Andre. > > [1] http://www.linux-arm.org/git?p=linux-ap.git > [2] https://lists.cs.columbia.edu/pipermail/kvmarm/2014-June/010086.html > > Changes v5 ... v6: > * adapt to v3.19-rc changes > * allow GICv3 emulation in case of wrong GIC device tree properties > * allow 32-bit guests to use GICv3 > * introduce new kvm_check_device_type() function > > Changes v4 ... v5: > * fix size of GICD_IROUTER region > * fix compilation on ARM with VGIC not configured > * properly fix the remaining FIXME at the GICv3 memory allocation > * minor style changes, comments and commit messages fixes > > Changes v3 ... v4: > * bug-fix in handling GICv3 redistributor CFG register > * move set/get_lr from gic_vm_ops back to vgic_ops (get rid of v3 06/19) > * getting rid of init_emul() at all > * rework guest GIC model initialization > * use non-atomic bit-set and bit-clear functions > * split up handle_mmio_misc* into multiple functions > * refine handling of some reserved registers > * use symbolic names for ICC_SGI1R_EL1 register fields (new patch 16/19) > * move private parameter from MMIO accessors to struct kvm_mmio_exit > * added documentation of new GICv3 guest device > * added lots of comments > * some renaming of identifiers > * minor changes in style and code flow of various functions > > Changes v2 ... v3: > * rebase to v3.18-rc2 > * adapt to new kvm_register_device() function > * split up vm_ops patch and the GICv2 split-off patch to ease review > * various smaller changes due to Christoffer's review > * fix compilation for arm > * remove support for trapping SGI sysreg accesses on arm hosts > > Changes v1 ... v2: > * rebase to v3.17-rc1, caused quite some changes to the init code > * new 9/15 patch to make 10/15 smaller > * fix wrongly ordered cp15 register trap entry (MarcZ) > * fix SGI broadcast (thanks to wanghaibin for spotting) > * fix broken bailout path in kvm_vgic_create (wanghaibin) > * check return value of init_emulation_ops() (wanghaibin) > * fix return value check in vgic_[sg]et_attr() > * add header inclusion guards > * remove double definition of VCPU_NOT_ALLOCATED > * some code move-around > * whitespace fixes > > Andre Przywara (20): > arm/arm64: KVM: rework MPIDR assignment and add accessors > arm/arm64: KVM: pass down user space provided GIC type into vGIC code > arm/arm64: KVM: refactor vgic_handle_mmio() function > arm/arm64: KVM: wrap 64 bit MMIO accesses with two 32 bit ones > arm/arm64: KVM: introduce per-VM ops > arm/arm64: KVM: move kvm_register_device_ops() into vGIC probing > arm/arm64: KVM: dont rely on a valid GICH base address > arm/arm64: KVM: make the maximum number of vCPUs a per-VM value > arm/arm64: KVM: make the value of ICC_SRE_EL1 a per-VM variable > arm/arm64: KVM: refactor MMIO accessors > arm/arm64: KVM: refactor/wrap vgic_set/get_attr() > arm/arm64: KVM: add vgic.h header file > arm/arm64: KVM: split GICv2 specific emulation code from vgic.c > arm/arm64: KVM: add opaque private pointer to MMIO data > arm/arm64: KVM: add virtual GICv3 distributor emulation > arm64: GICv3: introduce symbolic names for GICv3 ICC_SGI1R_EL1 fields > arm64: KVM: add SGI generation register emulation > KVM: introduce kvm_check_device_type() > arm/arm64: KVM: enable kernel side of GICv3 emulation > arm/arm64: KVM: allow userland to request a virtual GICv3 > > Documentation/virtual/kvm/devices/arm-vgic.txt | 22 +- > arch/arm/include/asm/kvm_emulate.h | 5 +- > arch/arm/include/asm/kvm_host.h | 3 + > arch/arm/include/asm/kvm_mmio.h | 1 + > arch/arm/kvm/Makefile | 1 + > arch/arm/kvm/arm.c | 23 +- > arch/arm/kvm/psci.c | 17 +- > arch/arm64/include/asm/kvm_emulate.h | 5 +- > arch/arm64/include/asm/kvm_host.h | 5 + > arch/arm64/include/asm/kvm_mmio.h | 1 + > arch/arm64/include/uapi/asm/kvm.h | 7 + > arch/arm64/kernel/asm-offsets.c | 1 + > arch/arm64/kvm/Makefile | 2 + > arch/arm64/kvm/sys_regs.c | 40 +- > arch/arm64/kvm/vgic-v3-switch.S | 14 +- > drivers/irqchip/irq-gic-v3.c | 14 +- > include/kvm/arm_vgic.h | 41 +- > include/linux/irqchip/arm-gic-v3.h | 44 + > include/linux/kvm_host.h | 3 + > include/uapi/linux/kvm.h | 2 + > virt/kvm/arm/vgic-v2-emul.c | 849 ++++++++++++++++++ > virt/kvm/arm/vgic-v2.c | 3 + > virt/kvm/arm/vgic-v3-emul.c | 1038 ++++++++++++++++++++++ > virt/kvm/arm/vgic-v3.c | 80 +- > virt/kvm/arm/vgic.c | 1095 ++++++------------------ > virt/kvm/arm/vgic.h | 123 +++ > virt/kvm/kvm_main.c | 33 +- > 27 files changed, 2555 insertions(+), 917 deletions(-) > create mode 100644 virt/kvm/arm/vgic-v2-emul.c > create mode 100644 virt/kvm/arm/vgic-v3-emul.c > create mode 100644 virt/kvm/arm/vgic.h >