KVM: arm/arm64: Remove kvm_vgic_vcpu_early_init

kvm_vgic_vcpu_early_init gets called after kvm_vgic_cpu_init which
is confusing. The call path is as follows:
kvm_vm_ioctl_create_vcpu
|_ kvm_arch_cpu_create
   |_ kvm_vcpu_init
      |_ kvm_arch_vcpu_init
         |_ kvm_vgic_vcpu_init
|_ kvm_arch_vcpu_postcreate
   |_ kvm_vgic_vcpu_early_init

Static initialization currently done in kvm_vgic_vcpu_early_init()
can be moved to kvm_vgic_vcpu_init(). So let's move the code and
remove kvm_vgic_vcpu_early_init(). kvm_arch_vcpu_postcreate() does
nothing.

Signed-off-by: Eric Auger <eric.auger@redhat.com>
Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>

2 files changed, 37 insertions, 44 deletions

diff --git a/virt/kvm/arm/arm.c b/virt/kvm/arm/arm.c
index 39e777155e7c..126b98fbf9ba 100644
--- a/virt/kvm/arm/arm.c
+++ b/virt/kvm/arm/arm.c
@@ -292,7 +292,6 @@ out:
 
 void kvm_arch_vcpu_postcreate(struct kvm_vcpu *vcpu)
 {
-        kvm_vgic_vcpu_early_init(vcpu);
 }
 
 void kvm_arch_vcpu_free(struct kvm_vcpu *vcpu)
diff --git a/virt/kvm/arm/vgic/vgic-init.c b/virt/kvm/arm/vgic/vgic-init.c
index 8901b2d8fca1..272af9704952 100644
--- a/virt/kvm/arm/vgic/vgic-init.c
+++ b/virt/kvm/arm/vgic/vgic-init.c
@@ -44,7 +44,7 @@
  *
  * CPU Interface:
  *
- * - kvm_vgic_vcpu_early_init(): initialization of static data that
+ * - kvm_vgic_vcpu_init(): initialization of static data that
  *   doesn't depend on any sizing information or emulation type. No
  *   allocation is allowed there.
  */
@@ -67,46 +67,6 @@ void kvm_vgic_early_init(struct kvm *kvm)
         spin_lock_init(&dist->lpi_list_lock);
 }
 
-/**
- * kvm_vgic_vcpu_early_init() - Initialize static VGIC VCPU data structures
- * @vcpu: The VCPU whose VGIC data structures whould be initialized
- *
- * Only do initialization, but do not actually enable the VGIC CPU interface
- * yet.
- */
-void kvm_vgic_vcpu_early_init(struct kvm_vcpu *vcpu)
-{
-        struct vgic_cpu *vgic_cpu = &vcpu->arch.vgic_cpu;
-        int i;
-
-        INIT_LIST_HEAD(&vgic_cpu->ap_list_head);
-        spin_lock_init(&vgic_cpu->ap_list_lock);
-
-        /*
-         * Enable and configure all SGIs to be edge-triggered and
-         * configure all PPIs as level-triggered.
-         */
-        for (i = 0; i < VGIC_NR_PRIVATE_IRQS; i++) {
-                struct vgic_irq *irq = &vgic_cpu->private_irqs[i];
-
-                INIT_LIST_HEAD(&irq->ap_list);
-                spin_lock_init(&irq->irq_lock);
-                irq->intid = i;
-                irq->vcpu = NULL;
-                irq->target_vcpu = vcpu;
-                irq->targets = 1U << vcpu->vcpu_id;
-                kref_init(&irq->refcount);
-                if (vgic_irq_is_sgi(i)) {
-                        /* SGIs */
-                        irq->enabled = 1;
-                        irq->config = VGIC_CONFIG_EDGE;
-                } else {
-                        /* PPIs */
-                        irq->config = VGIC_CONFIG_LEVEL;
-                }
-        }
-}
-
 /* CREATION */
 
 /**
@@ -224,13 +184,47 @@ static int kvm_vgic_dist_init(struct kvm *kvm, unsigned int nr_spis)
 }
 
 /**
- * kvm_vgic_vcpu_init() - Register VCPU-specific KVM iodevs
+ * kvm_vgic_vcpu_init() - Initialize static VGIC VCPU data
+ * structures and register VCPU-specific KVM iodevs
+ *
  * @vcpu: pointer to the VCPU being created and initialized
+ *
+ * Only do initialization, but do not actually enable the
+ * VGIC CPU interface
  */
 int kvm_vgic_vcpu_init(struct kvm_vcpu *vcpu)
 {
-        int ret = 0;
+        struct vgic_cpu *vgic_cpu = &vcpu->arch.vgic_cpu;
         struct vgic_dist *dist = &vcpu->kvm->arch.vgic;
+        int ret = 0;
+        int i;
+
+        INIT_LIST_HEAD(&vgic_cpu->ap_list_head);
+        spin_lock_init(&vgic_cpu->ap_list_lock);
+
+        /*
+         * Enable and configure all SGIs to be edge-triggered and
+         * configure all PPIs as level-triggered.
+         */
+        for (i = 0; i < VGIC_NR_PRIVATE_IRQS; i++) {
+                struct vgic_irq *irq = &vgic_cpu->private_irqs[i];
+
+                INIT_LIST_HEAD(&irq->ap_list);
+                spin_lock_init(&irq->irq_lock);
+                irq->intid = i;
+                irq->vcpu = NULL;
+                irq->target_vcpu = vcpu;
+                irq->targets = 1U << vcpu->vcpu_id;
+                kref_init(&irq->refcount);
+                if (vgic_irq_is_sgi(i)) {
+                        /* SGIs */
+                        irq->enabled = 1;
+                        irq->config = VGIC_CONFIG_EDGE;
+                } else {
+                        /* PPIs */
+                        irq->config = VGIC_CONFIG_LEVEL;
+                }
+        }
 
         if (!irqchip_in_kernel(vcpu->kvm))
                 return 0;