[PATCH v6 00/20] KVM GICv3 emulation
Andre Przywara
andre.przywara at arm.com
Wed Jan 14 02:48:34 PST 2015
Hi Tomasz, Eric,
On 14/01/15 08:51, Eric Auger wrote:
> Hi Tomasz
>
> I confirm I have the same issue and this git was working fine before for me.
Thanks for noticing me and sorry for that:
You are right, the access to the git repositories on linux-arm.org via
git does not work anymore for some reason. While this is being fixed, I
made a branch on github:
https://github.com/apritzel/linux.git
Pull the kvm-gicv3/v6 branch from there.
Cheers,
Andre.
> On 01/14/2015 09:50 AM, Tomasz Nowicki wrote:
>> Hi Andre,
>>
>> I heve just tried and got:
>>
>> $ git clone git://linux-arm.org/linux-ap.git
>> Cloning into 'linux-ap'...
>> fatal: Could not read from remote repository.
>>
>> Please make sure you have the correct access rights
>> and the repository exists.
>>
>> Regards,
>> Tomasz
>>
>>
>> On 14.01.2015 01:04, Andre Przywara wrote:
>>> 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
>>>>>
>>>>
>>>
>
>
More information about the linux-arm-kernel
mailing list