From: andre.przywara@arm•com (Andre Przywara)
To: linux-arm-kernel@lists•infradead.org
Subject: [PATCH v6 00/20] KVM GICv3 emulation
Date: Wed, 14 Jan 2015 00:04:31 +0000 [thread overview]
Message-ID: <54B5B28F.8050502@arm.com> (raw)
In-Reply-To: <54B546E9.30107@linaro.org>
On 01/13/2015 04:25 PM, Tomasz Nowicki wrote:
Hi Tomasz,
> 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!
Try:
git://linux-arm.org/linux-ap.git
(as referenced a bit hidden below ;-)
Cheers,
Andre.
>
> 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
>>
>
next prev parent reply other threads:[~2015-01-14 0:04 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-09 11:54 [PATCH v6 00/20] KVM GICv3 emulation Andre Przywara
2015-01-09 11:54 ` [PATCH v6 01/20] arm/arm64: KVM: rework MPIDR assignment and add accessors Andre Przywara
2015-01-09 11:54 ` [PATCH v6 02/20] arm/arm64: KVM: pass down user space provided GIC type into vGIC code Andre Przywara
2015-01-09 11:54 ` [PATCH v6 03/20] arm/arm64: KVM: refactor vgic_handle_mmio() function Andre Przywara
2015-01-09 11:54 ` [PATCH v6 04/20] arm/arm64: KVM: wrap 64 bit MMIO accesses with two 32 bit ones Andre Przywara
2015-01-09 11:54 ` [PATCH v6 05/20] arm/arm64: KVM: introduce per-VM ops Andre Przywara
2015-01-11 14:58 ` Christoffer Dall
2015-01-09 11:54 ` [PATCH v6 06/20] arm/arm64: KVM: move kvm_register_device_ops() into vGIC probing Andre Przywara
2015-01-09 11:54 ` [PATCH v6 07/20] arm/arm64: KVM: dont rely on a valid GICH base address Andre Przywara
2015-01-09 11:54 ` [PATCH v6 08/20] arm/arm64: KVM: make the maximum number of vCPUs a per-VM value Andre Przywara
2015-01-11 14:58 ` Christoffer Dall
2015-01-12 9:34 ` Andre Przywara
2015-01-09 11:54 ` [PATCH v6 09/20] arm/arm64: KVM: make the value of ICC_SRE_EL1 a per-VM variable Andre Przywara
2015-01-09 11:54 ` [PATCH v6 10/20] arm/arm64: KVM: refactor MMIO accessors Andre Przywara
2015-01-09 11:54 ` [PATCH v6 11/20] arm/arm64: KVM: refactor/wrap vgic_set/get_attr() Andre Przywara
2015-01-09 11:54 ` [PATCH v6 12/20] arm/arm64: KVM: add vgic.h header file Andre Przywara
2015-01-09 11:54 ` [PATCH v6 13/20] arm/arm64: KVM: split GICv2 specific emulation code from vgic.c Andre Przywara
2015-01-09 11:54 ` [PATCH v6 14/20] arm/arm64: KVM: add opaque private pointer to MMIO data Andre Przywara
2015-01-09 11:54 ` [PATCH v6 15/20] arm/arm64: KVM: add virtual GICv3 distributor emulation Andre Przywara
2015-01-11 15:15 ` Christoffer Dall
2015-01-12 9:57 ` Andre Przywara
2015-01-12 17:08 ` Christoffer Dall
2015-01-09 11:54 ` [PATCH v6 16/20] arm64: GICv3: introduce symbolic names for GICv3 ICC_SGI1R_EL1 fields Andre Przywara
2015-01-09 11:54 ` [PATCH v6 17/20] arm64: KVM: add SGI generation register emulation Andre Przywara
2015-01-09 11:54 ` [PATCH v6 18/20] KVM: introduce kvm_check_device_type() Andre Przywara
2015-01-09 12:33 ` Christoffer Dall
2015-01-09 13:42 ` Andre Przywara
2015-01-11 15:22 ` Christoffer Dall
2015-01-12 12:33 ` Andre Przywara
2015-01-12 17:11 ` Christoffer Dall
2015-01-14 12:33 ` Andre Przywara
2015-01-09 11:54 ` [PATCH v6 19/20] arm/arm64: KVM: enable kernel side of GICv3 emulation Andre Przywara
2015-01-11 15:32 ` Christoffer Dall
2015-01-09 11:54 ` [PATCH v6 20/20] arm/arm64: KVM: allow userland to request a virtual GICv3 Andre Przywara
2015-01-11 15:35 ` Christoffer Dall
2015-01-13 16:25 ` [PATCH v6 00/20] KVM GICv3 emulation Tomasz Nowicki
2015-01-14 0:04 ` Andre Przywara [this message]
2015-01-14 8:50 ` Tomasz Nowicki
2015-01-14 8:51 ` Eric Auger
2015-01-14 10:48 ` Andre Przywara
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=54B5B28F.8050502@arm.com \
--to=andre.przywara@arm$(echo .)com \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox