MIPS: KVM: Use va in kvm_get_inst()
authorJames Hogan <james.hogan@imgtec.com>
Thu, 9 Jun 2016 13:19:21 +0000 (14:19 +0100)
committerPaolo Bonzini <pbonzini@redhat.com>
Tue, 14 Jun 2016 09:02:48 +0000 (11:02 +0200)
Like other functions, make use of a local unsigned long va, for the
virtual address of the PC. This reduces the amount of verbose casting of
the opc pointer to an unsigned long.

Signed-off-by: James Hogan <james.hogan@imgtec.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: Radim Krčmář <rkrcmar@redhat.com>
Cc: Ralf Baechle <ralf@linux-mips.org>
Cc: linux-mips@linux-mips.org
Cc: kvm@vger.kernel.org
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
arch/mips/kvm/mmu.c

index ad3125fa9c614d8b0aa9d5617379c7aa5ba89053..208f70409ccb44009297046dfe0af1c9318ac977 100644 (file)
@@ -327,17 +327,18 @@ u32 kvm_get_inst(u32 *opc, struct kvm_vcpu *vcpu)
 {
        struct mips_coproc *cop0 = vcpu->arch.cop0;
        unsigned long paddr, flags, vpn2, asid;
+       unsigned long va = (unsigned long)opc;
        u32 inst;
        int index;
 
-       if (KVM_GUEST_KSEGX((unsigned long) opc) < KVM_GUEST_KSEG0 ||
-           KVM_GUEST_KSEGX((unsigned long) opc) == KVM_GUEST_KSEG23) {
+       if (KVM_GUEST_KSEGX(va) < KVM_GUEST_KSEG0 ||
+           KVM_GUEST_KSEGX(va) == KVM_GUEST_KSEG23) {
                local_irq_save(flags);
-               index = kvm_mips_host_tlb_lookup(vcpu, (unsigned long) opc);
+               index = kvm_mips_host_tlb_lookup(vcpu, va);
                if (index >= 0) {
                        inst = *(opc);
                } else {
-                       vpn2 = (unsigned long) opc & VPN2_MASK;
+                       vpn2 = va & VPN2_MASK;
                        asid = kvm_read_c0_guest_entryhi(cop0) &
                                                KVM_ENTRYHI_ASID;
                        index = kvm_mips_guest_tlb_lookup(vcpu, vpn2 | asid);
@@ -354,10 +355,8 @@ u32 kvm_get_inst(u32 *opc, struct kvm_vcpu *vcpu)
                        inst = *(opc);
                }
                local_irq_restore(flags);
-       } else if (KVM_GUEST_KSEGX(opc) == KVM_GUEST_KSEG0) {
-               paddr =
-                   kvm_mips_translate_guest_kseg0_to_hpa(vcpu,
-                                                         (unsigned long) opc);
+       } else if (KVM_GUEST_KSEGX(va) == KVM_GUEST_KSEG0) {
+               paddr = kvm_mips_translate_guest_kseg0_to_hpa(vcpu, va);
                inst = *(u32 *) CKSEG0ADDR(paddr);
        } else {
                kvm_err("%s: illegal address: %p\n", __func__, opc);