KVM: SVM: Refactor AVIC vcpu initialization into avic_init_vcpu()

Preparing the base code for subsequent changes. This does not change
existing logic.

Signed-off-by: Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>
Signed-off-by: Radim Krčmář <rkrcmar@redhat.com>

1 files changed, 20 insertions, 8 deletions

diff --git a/arch/x86/kvm/svm.c b/arch/x86/kvm/svm.c
index af54327b9017..cab020acb509 100644
--- a/arch/x86/kvm/svm.c
+++ b/arch/x86/kvm/svm.c
@@ -1600,6 +1600,23 @@ static void svm_vcpu_reset(struct kvm_vcpu *vcpu, bool init_event)
                 avic_update_vapic_bar(svm, APIC_DEFAULT_PHYS_BASE);
 }
 
+static int avic_init_vcpu(struct vcpu_svm *svm)
+{
+        int ret;
+
+        if (!avic)
+                return 0;
+
+        ret = avic_init_backing_page(&svm->vcpu);
+        if (ret)
+                return ret;
+
+        INIT_LIST_HEAD(&svm->ir_list);
+        spin_lock_init(&svm->ir_list_lock);
+
+        return ret;
+}
+
 static struct kvm_vcpu *svm_create_vcpu(struct kvm *kvm, unsigned int id)
 {
         struct vcpu_svm *svm;
@@ -1636,14 +1653,9 @@ static struct kvm_vcpu *svm_create_vcpu(struct kvm *kvm, unsigned int id)
         if (!hsave_page)
                 goto free_page3;
 
-        if (avic) {
-                err = avic_init_backing_page(&svm->vcpu);
-                if (err)
-                        goto free_page4;
-
-                INIT_LIST_HEAD(&svm->ir_list);
-                spin_lock_init(&svm->ir_list_lock);
-        }
+        err = avic_init_vcpu(svm);
+        if (err)
+                goto free_page4;
 
         /* We initialize this flag to true to make sure that the is_running
          * bit would be set the first time the vcpu is loaded.