summaryrefslogtreecommitdiffstats
path: root/virt/kvm
diff options
context:
space:
mode:
authorDave Martin <Dave.Martin@arm.com>2018-05-02 13:36:48 +0100
committerMarc Zyngier <marc.zyngier@arm.com>2018-05-25 12:28:30 +0100
commit7846b3119e24fe8d726535d6aa7489253797700c (patch)
tree8b9aa463a866e3048b29d184937b35c5734a614f /virt/kvm
parentba4f4cb0e661ed4c68057d4dd831f54b99770b09 (diff)
downloadop-kernel-dev-7846b3119e24fe8d726535d6aa7489253797700c.zip
op-kernel-dev-7846b3119e24fe8d726535d6aa7489253797700c.tar.gz
KVM: arm64: Fold redundant exit code checks out of fixup_guest_exit()
The entire tail of fixup_guest_exit() is contained in if statements of the form if (x && *exit_code == ARM_EXCEPTION_TRAP). As a result, we can check just once and bail out of the function early, allowing the remaining if conditions to be simplified. The only awkward case is where *exit_code is changed to ARM_EXCEPTION_EL1_SERROR in the case of an illegal GICv2 CPU interface access: in that case, the GICv3 trap handling code is skipped using a goto. This avoids pointlessly evaluating the static branch check for the GICv3 case, even though we can't have vgic_v2_cpuif_trap and vgic_v3_cpuif_trap true simultaneously unless we have a GICv3 and GICv2 on the host: that sounds stupid, but I haven't satisfied myself that it can't happen. No functional change. Signed-off-by: Dave Martin <Dave.Martin@arm.com> Reviewed-by: Marc Zyngier <marc.zyngier@arm.com> Reviewed-by: Alex Bennée <alex.bennee@linaro.org> Acked-by: Christoffer Dall <christoffer.dall@arm.com> Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
Diffstat (limited to 'virt/kvm')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud