KVM: PPC: Remove redundant kvm_run from vcpu_arch
authorTianjia Zhang <tianjia.zhang@linux.alibaba.com>
Mon, 27 Apr 2020 04:35:10 +0000 (12:35 +0800)
committerPaul Mackerras <paulus@ozlabs.org>
Wed, 27 May 2020 01:39:31 +0000 (11:39 +1000)
The 'kvm_run' field already exists in the 'vcpu' structure, which
is the same structure as the 'kvm_run' in the 'vcpu_arch' and
should be deleted.

Signed-off-by: Tianjia Zhang <tianjia.zhang@linux.alibaba.com>
Reviewed-by: Vitaly Kuznetsov <vkuznets@redhat.com>
Reviewed-by: Paul Mackerras <paulus@ozlabs.org>
Signed-off-by: Paul Mackerras <paulus@ozlabs.org>
arch/powerpc/include/asm/kvm_host.h
arch/powerpc/kvm/book3s_hv.c
arch/powerpc/kvm/book3s_hv_nested.c

index 337047ba4a566f511c593ed62bfca0d3a0f39c95..7e2d061d04451b4ff4a76fd38a25236aba0062b1 100644 (file)
@@ -795,7 +795,6 @@ struct kvm_vcpu_arch {
        struct mmio_hpte_cache_entry *pgfault_cache;
 
        struct task_struct *run_task;
-       struct kvm_run *kvm_run;
 
        spinlock_t vpa_update_lock;
        struct kvmppc_vpa vpa;
index 7f59c47a5b9d3c7df5b096e8e97cffc9e963e6f1..50fbfc1fc34b2f63cf3dc18964e3d8546133ce47 100644 (file)
@@ -2932,7 +2932,7 @@ static void post_guest_process(struct kvmppc_vcore *vc, bool is_master)
 
                ret = RESUME_GUEST;
                if (vcpu->arch.trap)
-                       ret = kvmppc_handle_exit_hv(vcpu->arch.kvm_run, vcpu,
+                       ret = kvmppc_handle_exit_hv(vcpu->run, vcpu,
                                                    vcpu->arch.run_task);
 
                vcpu->arch.ret = ret;
@@ -3917,7 +3917,6 @@ static int kvmppc_run_vcpu(struct kvm_run *kvm_run, struct kvm_vcpu *vcpu)
        spin_lock(&vc->lock);
        vcpu->arch.ceded = 0;
        vcpu->arch.run_task = current;
-       vcpu->arch.kvm_run = kvm_run;
        vcpu->arch.stolen_logged = vcore_stolen_time(vc, mftb());
        vcpu->arch.state = KVMPPC_VCPU_RUNNABLE;
        vcpu->arch.busy_preempt = TB_NIL;
@@ -3970,7 +3969,7 @@ static int kvmppc_run_vcpu(struct kvm_run *kvm_run, struct kvm_vcpu *vcpu)
                        if (signal_pending(v->arch.run_task)) {
                                kvmppc_remove_runnable(vc, v);
                                v->stat.signal_exits++;
-                               v->arch.kvm_run->exit_reason = KVM_EXIT_INTR;
+                               v->run->exit_reason = KVM_EXIT_INTR;
                                v->arch.ret = -EINTR;
                                wake_up(&v->arch.cpu_run);
                        }
@@ -4046,7 +4045,6 @@ int kvmhv_run_single_vcpu(struct kvm_run *kvm_run,
        vc = vcpu->arch.vcore;
        vcpu->arch.ceded = 0;
        vcpu->arch.run_task = current;
-       vcpu->arch.kvm_run = kvm_run;
        vcpu->arch.stolen_logged = vcore_stolen_time(vc, mftb());
        vcpu->arch.state = KVMPPC_VCPU_RUNNABLE;
        vcpu->arch.busy_preempt = TB_NIL;
index cad324312040b416da69d28a1f344fb624f4bdb9..d31c2e5c8b2ae8c965e30f9b326c52b2e18a039c 100644 (file)
@@ -290,8 +290,7 @@ long kvmhv_enter_nested_guest(struct kvm_vcpu *vcpu)
                        r = RESUME_HOST;
                        break;
                }
-               r = kvmhv_run_single_vcpu(vcpu->arch.kvm_run, vcpu, hdec_exp,
-                                         lpcr);
+               r = kvmhv_run_single_vcpu(vcpu->run, vcpu, hdec_exp, lpcr);
        } while (is_kvmppc_resume_guest(r));
 
        /* save L2 state for return */