From: ynorov@caviumnetworks.com (Yury Norov)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 36/37] KVM: arm/arm64: Move VGIC APR save/restore to vgic put/load
Date: Sun, 26 Nov 2017 18:09:25 +0300 [thread overview]
Message-ID: <20171126150925.n2c24dqjh2h4g4wb@yury-thinkpad> (raw)
In-Reply-To: <20171012104141.26902-37-christoffer.dall@linaro.org>
On Thu, Oct 12, 2017 at 12:41:40PM +0200, Christoffer Dall wrote:
> The APRs can only have bits set when the guest acknowledges an interrupt
> in the LR and can only have a bit cleared when the guest EOIs an
> interrupt in the LR. Therefore, if we have no LRs with any
> pending/active interrupts, the APR cannot change value and there is no
> need to clear it on every exit from the VM (hint: it will have already
> been cleared when we exited the guest the last time with the LRs all
> EOIed).
>
> The only case we need to take care of is when we migrate the VCPU away
> from a CPU or migrate a new VCPU onto a CPU, or when we return to
> userspace to capture the state of the VCPU for migration. To make sure
> this works, factor out the APR save/restore functionality into separate
> functions called from the VCPU (and by extension VGIC) put/load hooks.
>
> Signed-off-by: Christoffer Dall <christoffer.dall@linaro.org>
> ---
> arch/arm/include/asm/kvm_hyp.h | 2 +
> arch/arm64/include/asm/kvm_hyp.h | 2 +
> virt/kvm/arm/hyp/vgic-v3-sr.c | 123 +++++++++++++++++++++------------------
> virt/kvm/arm/vgic/vgic-v2.c | 7 +--
> virt/kvm/arm/vgic/vgic-v3.c | 5 ++
> 5 files changed, 77 insertions(+), 62 deletions(-)
[...]
> @@ -361,6 +304,72 @@ void __hyp_text __vgic_v3_restore_state(struct kvm_vcpu *vcpu)
> ICC_SRE_EL2);
> }
>
> +void __hyp_text __vgic_v3_save_aprs(struct kvm_vcpu *vcpu)
> +{
> + struct vgic_v3_cpu_if *cpu_if;
> + u64 val;
> + u32 nr_pre_bits;
> +
> + vcpu = kern_hyp_va(vcpu);
> + cpu_if = &vcpu->arch.vgic_cpu.vgic_v3;
> +
> + val = read_gicreg(ICH_VTR_EL2);
> + nr_pre_bits = vtr_to_nr_pre_bits(val);
> +
> + switch (nr_pre_bits) {
> + case 7:
> + cpu_if->vgic_ap0r[3] = __vgic_v3_read_ap0rn(3);
> + cpu_if->vgic_ap0r[2] = __vgic_v3_read_ap0rn(2);
> + case 6:
> + cpu_if->vgic_ap0r[1] = __vgic_v3_read_ap0rn(1);
> + default:
> + cpu_if->vgic_ap0r[0] = __vgic_v3_read_ap0rn(0);
> + }
> +
> + switch (nr_pre_bits) {
> + case 7:
> + cpu_if->vgic_ap1r[3] = __vgic_v3_read_ap1rn(3);
> + cpu_if->vgic_ap1r[2] = __vgic_v3_read_ap1rn(2);
> + case 6:
> + cpu_if->vgic_ap1r[1] = __vgic_v3_read_ap1rn(1);
> + default:
> + cpu_if->vgic_ap1r[0] = __vgic_v3_read_ap1rn(0);
> + }
> +}
What for 2 switch-cases here? At first glance, you can join them:
switch (nr_pre_bits) {
case 7:
cpu_if->vgic_ap0r[3] = __vgic_v3_read_ap0rn(3);
cpu_if->vgic_ap0r[2] = __vgic_v3_read_ap0rn(2);
cpu_if->vgic_ap1r[3] = __vgic_v3_read_ap1rn(3);
cpu_if->vgic_ap1r[2] = __vgic_v3_read_ap1rn(2);
case 6:
cpu_if->vgic_ap0r[1] = __vgic_v3_read_ap0rn(1);
cpu_if->vgic_ap1r[1] = __vgic_v3_read_ap1rn(1);
default:
cpu_if->vgic_ap1r[0] = __vgic_v3_read_ap1rn(0);
cpu_if->vgic_ap0r[0] = __vgic_v3_read_ap0rn(0);
}
> +
> +void __hyp_text __vgic_v3_restore_aprs(struct kvm_vcpu *vcpu)
> +{
> + struct vgic_v3_cpu_if *cpu_if;
> + u64 val;
> + u32 nr_pre_bits;
> +
> + vcpu = kern_hyp_va(vcpu);
> + cpu_if = &vcpu->arch.vgic_cpu.vgic_v3;
> +
> + val = read_gicreg(ICH_VTR_EL2);
> + nr_pre_bits = vtr_to_nr_pre_bits(val);
> +
> + switch (nr_pre_bits) {
> + case 7:
> + __vgic_v3_write_ap0rn(cpu_if->vgic_ap0r[3], 3);
> + __vgic_v3_write_ap0rn(cpu_if->vgic_ap0r[2], 2);
> + case 6:
> + __vgic_v3_write_ap0rn(cpu_if->vgic_ap0r[1], 1);
> + default:
> + __vgic_v3_write_ap0rn(cpu_if->vgic_ap0r[0], 0);
> + }
> +
> + switch (nr_pre_bits) {
> + case 7:
> + __vgic_v3_write_ap1rn(cpu_if->vgic_ap1r[3], 3);
> + __vgic_v3_write_ap1rn(cpu_if->vgic_ap1r[2], 2);
> + case 6:
> + __vgic_v3_write_ap1rn(cpu_if->vgic_ap1r[1], 1);
> + default:
> + __vgic_v3_write_ap1rn(cpu_if->vgic_ap1r[0], 0);
> + }
And here
Thanks,
Yury
> +}
> +
> void __hyp_text __vgic_v3_init_lrs(void)
> {
> int max_lr_idx = vtr_to_max_lr_idx(read_gicreg(ICH_VTR_EL2));
> diff --git a/virt/kvm/arm/vgic/vgic-v2.c b/virt/kvm/arm/vgic/vgic-v2.c
> index df7e03b..c04c3475 100644
> --- a/virt/kvm/arm/vgic/vgic-v2.c
> +++ b/virt/kvm/arm/vgic/vgic-v2.c
> @@ -428,7 +428,6 @@ void vgic_v2_save_state(struct kvm_vcpu *vcpu)
> {
> struct kvm *kvm = vcpu->kvm;
> struct vgic_dist *vgic = &kvm->arch.vgic;
> - struct vgic_v2_cpu_if *cpu_if = &vcpu->arch.vgic_cpu.vgic_v2;
> void __iomem *base = vgic->vctrl_base;
> u64 used_lrs = vcpu->arch.vgic_cpu.used_lrs;
>
> @@ -436,11 +435,8 @@ void vgic_v2_save_state(struct kvm_vcpu *vcpu)
> return;
>
> if (used_lrs) {
> - cpu_if->vgic_apr = readl_relaxed(base + GICH_APR);
> save_lrs(vcpu, base);
> writel_relaxed(0, base + GICH_HCR);
> - } else {
> - cpu_if->vgic_apr = 0;
> }
> }
>
> @@ -458,7 +454,6 @@ void vgic_v2_restore_state(struct kvm_vcpu *vcpu)
>
> if (used_lrs) {
> writel_relaxed(cpu_if->vgic_hcr, base + GICH_HCR);
> - writel_relaxed(cpu_if->vgic_apr, base + GICH_APR);
> for (i = 0; i < used_lrs; i++) {
> writel_relaxed(cpu_if->vgic_lr[i],
> base + GICH_LR0 + (i * 4));
> @@ -472,6 +467,7 @@ void vgic_v2_load(struct kvm_vcpu *vcpu)
> struct vgic_dist *vgic = &vcpu->kvm->arch.vgic;
>
> writel_relaxed(cpu_if->vgic_vmcr, vgic->vctrl_base + GICH_VMCR);
> + writel_relaxed(cpu_if->vgic_apr, vgic->vctrl_base + GICH_APR);
> }
>
> void vgic_v2_put(struct kvm_vcpu *vcpu)
> @@ -480,4 +476,5 @@ void vgic_v2_put(struct kvm_vcpu *vcpu)
> struct vgic_dist *vgic = &vcpu->kvm->arch.vgic;
>
> cpu_if->vgic_vmcr = readl_relaxed(vgic->vctrl_base + GICH_VMCR);
> + cpu_if->vgic_apr = readl_relaxed(vgic->vctrl_base + GICH_APR);
> }
> diff --git a/virt/kvm/arm/vgic/vgic-v3.c b/virt/kvm/arm/vgic/vgic-v3.c
> index 0900649..df3a222 100644
> --- a/virt/kvm/arm/vgic/vgic-v3.c
> +++ b/virt/kvm/arm/vgic/vgic-v3.c
> @@ -16,6 +16,7 @@
> #include <linux/kvm.h>
> #include <linux/kvm_host.h>
> #include <kvm/arm_vgic.h>
> +#include <asm/kvm_hyp.h>
> #include <asm/kvm_mmu.h>
> #include <asm/kvm_asm.h>
>
> @@ -544,6 +545,8 @@ void vgic_v3_load(struct kvm_vcpu *vcpu)
> */
> if (likely(cpu_if->vgic_sre))
> kvm_call_hyp(__vgic_v3_write_vmcr, cpu_if->vgic_vmcr);
> +
> + kvm_call_hyp(__vgic_v3_restore_aprs, vcpu);
> }
>
> void vgic_v3_put(struct kvm_vcpu *vcpu)
> @@ -552,4 +555,6 @@ void vgic_v3_put(struct kvm_vcpu *vcpu)
>
> if (likely(cpu_if->vgic_sre))
> cpu_if->vgic_vmcr = kvm_call_hyp(__vgic_v3_read_vmcr);
> +
> + kvm_call_hyp(__vgic_v3_save_aprs, vcpu);
> }
> --
> 2.9.0
next prev parent reply other threads:[~2017-11-26 15:09 UTC|newest]
Thread overview: 127+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-12 10:41 [PATCH 00/37] Optimize KVM/ARM for VHE systems Christoffer Dall
2017-10-12 10:41 ` [PATCH 01/37] KVM: arm64: Avoid storing the vcpu pointer on the stack Christoffer Dall
2017-10-12 15:49 ` Marc Zyngier
2017-10-12 17:02 ` Christoffer Dall
2017-10-13 11:31 ` Marc Zyngier
2017-11-23 20:59 ` Christoffer Dall
2017-11-27 11:11 ` James Morse
2017-11-29 18:20 ` Christoffer Dall
2017-11-06 17:22 ` Andrew Jones
2017-11-07 8:24 ` Christoffer Dall
2017-10-12 10:41 ` [PATCH 02/37] KVM: arm64: Rework hyp_panic for VHE and non-VHE Christoffer Dall
2017-10-12 15:55 ` Marc Zyngier
2017-10-12 17:06 ` Christoffer Dall
2017-10-12 10:41 ` [PATCH 03/37] KVM: arm64: Move HCR_INT_OVERRIDE to default HCR_EL2 guest flag Christoffer Dall
2017-10-12 16:20 ` Marc Zyngier
2017-10-12 10:41 ` [PATCH 04/37] KVM: arm/arm64: Get rid of vcpu->arch.irq_lines Christoffer Dall
2017-10-12 16:24 ` Marc Zyngier
2017-11-06 17:58 ` Andrew Jones
2017-11-14 12:17 ` Julien Thierry
2017-11-16 16:11 ` Julien Thierry
2017-11-26 16:04 ` Christoffer Dall
2017-10-12 10:41 ` [PATCH 05/37] KVM: Record the executing ioctl number on the vcpu struct Christoffer Dall
2017-10-13 17:13 ` Radim Krčmář
2017-10-13 17:31 ` Christoffer Dall
2017-10-13 18:38 ` Radim Krčmář
2017-10-13 18:51 ` Christoffer Dall
2017-11-07 10:45 ` Andrew Jones
2017-11-22 20:28 ` Christoffer Dall
2017-10-12 10:41 ` [PATCH 06/37] KVM: arm/arm64: Only load/put VCPU state for KVM_RUN Christoffer Dall
2017-10-12 10:41 ` [PATCH 07/37] KVM: arm/arm64: Add kvm_vcpu_load_sysregs and kvm_vcpu_put_sysregs Christoffer Dall
2017-11-07 10:56 ` Andrew Jones
2017-11-07 11:10 ` Andrew Jones
2017-11-22 20:34 ` Christoffer Dall
2017-11-23 11:08 ` Andrew Jones
2017-10-12 10:41 ` [PATCH 08/37] KVM: arm64: Defer restoring host VFP state to vcpu_put Christoffer Dall
2017-11-07 13:15 ` Andrew Jones
2017-11-26 16:24 ` Christoffer Dall
2017-11-15 16:04 ` Andrew Jones
2017-11-26 16:17 ` Christoffer Dall
2017-11-27 8:32 ` Andrew Jones
2017-11-25 7:52 ` Yury Norov
2017-11-26 16:17 ` Christoffer Dall
2017-11-26 18:58 ` Yury Norov
2017-11-26 19:18 ` Christoffer Dall
2017-11-27 6:25 ` Yury Norov
2017-11-30 19:07 ` Marc Zyngier
2017-10-12 10:41 ` [PATCH 09/37] KVM: arm64: Move debug dirty flag calculation out of world switch Christoffer Dall
2017-11-07 14:09 ` Andrew Jones
2017-11-25 8:09 ` Yury Norov
2017-12-01 17:25 ` Christoffer Dall
2017-12-03 13:17 ` Andrew Jones
2017-10-12 10:41 ` [PATCH 10/37] KVM: arm64: Slightly improve debug save/restore functions Christoffer Dall
2017-11-07 14:22 ` Andrew Jones
2017-12-01 17:51 ` Christoffer Dall
2017-11-14 16:42 ` Julien Thierry
2017-12-01 15:19 ` Christoffer Dall
2017-12-06 15:38 ` Julien Thierry
2017-10-12 10:41 ` [PATCH 11/37] KVM: arm64: Improve debug register save/restore flow Christoffer Dall
2017-11-07 14:48 ` Andrew Jones
2017-12-01 17:52 ` Christoffer Dall
2017-12-03 13:49 ` Andrew Jones
2017-12-03 20:47 ` Christoffer Dall
2017-10-12 10:41 ` [PATCH 12/37] KVM: arm64: Factor out fault info population and gic workarounds Christoffer Dall
2017-11-07 15:12 ` Andrew Jones
2017-10-12 10:41 ` [PATCH 13/37] KVM: arm64: Introduce VHE-specific kvm_vcpu_run Christoffer Dall
2017-11-07 15:25 ` Andrew Jones
2017-12-01 18:10 ` Christoffer Dall
2017-10-12 10:41 ` [PATCH 14/37] KVM: arm64: Remove kern_hyp_va() use in VHE switch function Christoffer Dall
2017-11-07 16:07 ` Andrew Jones
2017-10-12 10:41 ` [PATCH 15/37] KVM: arm64: Don't deactivate VM on VHE systems Christoffer Dall
2017-11-07 16:14 ` Andrew Jones
2017-12-03 19:27 ` Christoffer Dall
2017-10-12 10:41 ` [PATCH 16/37] KVM: arm64: Remove noop calls to timer save/restore from VHE switch Christoffer Dall
2017-11-07 16:25 ` Andrew Jones
2017-12-03 19:27 ` Christoffer Dall
2017-10-12 10:41 ` [PATCH 17/37] KVM: arm64: Move userspace system registers into separate function Christoffer Dall
2017-11-08 9:32 ` Andrew Jones
2017-12-03 19:36 ` Christoffer Dall
2017-10-12 10:41 ` [PATCH 18/37] KVM: arm64: Rewrite sysreg alternatives to static keys Christoffer Dall
2017-10-12 10:41 ` [PATCH 19/37] KVM: arm64: Introduce separate VHE/non-VHE sysreg save/restore functions Christoffer Dall
2017-11-08 10:31 ` Andrew Jones
2017-10-12 10:41 ` [PATCH 20/37] KVM: arm64: Unify non-VHE host/guest sysreg save and restore functions Christoffer Dall
2017-11-08 10:39 ` Andrew Jones
2017-12-03 19:41 ` Christoffer Dall
2017-10-12 10:41 ` [PATCH 21/37] KVM: arm64: Don't save the host ELR_EL2 and SPSR_EL2 on VHE systems Christoffer Dall
2017-11-08 17:03 ` Andrew Jones
2017-12-03 19:45 ` Christoffer Dall
2017-10-12 10:41 ` [PATCH 22/37] KVM: arm64: Change 32-bit handling of VM system registers Christoffer Dall
2017-11-13 16:25 ` Andrew Jones
2017-10-12 10:41 ` [PATCH 23/37] KVM: arm64: Prepare to handle traps on deferred VM sysregs Christoffer Dall
2017-11-13 17:54 ` Andrew Jones
2017-12-03 19:50 ` Christoffer Dall
2017-12-04 10:05 ` Andrew Jones
2017-10-12 10:41 ` [PATCH 24/37] KVM: arm64: Prepare to handle traps on deferred EL0 sysregs Christoffer Dall
2017-11-15 9:25 ` Julien Thierry
2017-12-03 19:51 ` Christoffer Dall
2017-10-12 10:41 ` [PATCH 25/37] KVM: arm64: Prepare to handle traps on remaining deferred EL1 sysregs Christoffer Dall
2017-11-13 18:56 ` Andrew Jones
2017-12-03 20:29 ` Christoffer Dall
2017-10-12 10:41 ` [PATCH 26/37] KVM: arm64: Prepare to handle traps on deferred AArch32 sysregs Christoffer Dall
2017-11-13 19:07 ` Andrew Jones
2017-12-03 20:35 ` Christoffer Dall
2017-10-12 10:41 ` [PATCH 27/37] KVM: arm64: Defer saving/restoring system registers to vcpu load/put on VHE Christoffer Dall
2017-10-12 10:41 ` [PATCH 28/37] KVM: arm64: Move common VHE/non-VHE trap config in separate functions Christoffer Dall
2017-11-25 10:43 ` Yury Norov
2017-11-25 10:49 ` Russell King - ARM Linux
2017-10-12 10:41 ` [PATCH 29/37] KVM: arm64: Configure FPSIMD traps on vcpu load/put for VHE Christoffer Dall
2017-10-12 10:41 ` [PATCH 30/37] KVM: arm64: Configure c15, PMU, and debug register traps on cpu " Christoffer Dall
2017-10-12 10:41 ` [PATCH 31/37] KVM: arm64: Separate activate_traps and deactive_traps for VHE and non-VHE Christoffer Dall
2017-10-12 10:41 ` [PATCH 32/37] KVM: arm/arm64: Handle VGICv2 save/restore from the main VGIC code Christoffer Dall
2017-11-15 17:50 ` Andre Przywara
2017-11-26 10:29 ` Yury Norov
2017-11-26 19:46 ` Christoffer Dall
2017-11-30 12:09 ` Yury Norov
2017-11-26 19:37 ` Christoffer Dall
2017-10-12 10:41 ` [PATCH 33/37] KVM: arm/arm64: Move arm64-only vgic-v2-sr.c file to arm64 Christoffer Dall
2017-11-15 17:52 ` Andre Przywara
2017-10-12 10:41 ` [PATCH 34/37] KVM: arm/arm64: Handle VGICv3 save/restore from the main VGIC code on VHE Christoffer Dall
2017-10-12 10:41 ` [PATCH 35/37] KVM: arm/arm64: Get rid of vgic_elrsr Christoffer Dall
2017-11-26 14:39 ` Yury Norov
2017-11-26 19:53 ` Christoffer Dall
2017-10-12 10:41 ` [PATCH 36/37] KVM: arm/arm64: Move VGIC APR save/restore to vgic put/load Christoffer Dall
2017-11-26 15:09 ` Yury Norov [this message]
2017-11-26 19:55 ` Christoffer Dall
2017-10-12 10:41 ` [PATCH 37/37] KVM: arm/arm64: Avoid VGICv3 save/restore on VHE with no IRQs Christoffer Dall
2017-11-30 18:33 ` Yury Norov
2017-12-03 20:38 ` Christoffer Dall
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=20171126150925.n2c24dqjh2h4g4wb@yury-thinkpad \
--to=ynorov@caviumnetworks.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
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;
as well as URLs for NNTP newsgroup(s).