Skip to content

Commit c70faab

Browse files
nehebavpatel
authored andcommitted
RISC-V: KVM: Use flexible array for APLIC IRQ state
Store the per-source APLIC IRQ state in the APLIC allocation instead of allocating it separately. This ties the IRQ state lifetime directly to the APLIC state, removes a separate allocation failure path, and lets __counted_by() describe the array bounds. Assisted-by: Codex:GPT-5.5 Signed-off-by: Rosen Penev <rosenp@gmail.com> Reviewed-by: Anup Patel <anup@brainfault.org> Link: https://lore.kernel.org/r/20260511032144.361520-1-rosenp@gmail.com Signed-off-by: Anup Patel <anup@brainfault.org>
1 parent 60aa673 commit c70faab

1 file changed

Lines changed: 3 additions & 12 deletions

File tree

arch/riscv/kvm/aia_aplic.c

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ struct aplic {
3535

3636
u32 nr_irqs;
3737
u32 nr_words;
38-
struct aplic_irq *irqs;
38+
struct aplic_irq irqs[] __counted_by(nr_irqs);
3939
};
4040

4141
static u32 aplic_read_sourcecfg(struct aplic *aplic, u32 irq)
@@ -581,19 +581,14 @@ int kvm_riscv_aia_aplic_init(struct kvm *kvm)
581581
return 0;
582582

583583
/* Allocate APLIC global state */
584-
aplic = kzalloc_obj(*aplic);
584+
aplic = kzalloc_flex(*aplic, irqs, kvm->arch.aia.nr_sources + 1);
585585
if (!aplic)
586586
return -ENOMEM;
587587
kvm->arch.aia.aplic_state = aplic;
588588

589589
/* Setup APLIC IRQs */
590590
aplic->nr_irqs = kvm->arch.aia.nr_sources + 1;
591591
aplic->nr_words = DIV_ROUND_UP(aplic->nr_irqs, 32);
592-
aplic->irqs = kzalloc_objs(*aplic->irqs, aplic->nr_irqs);
593-
if (!aplic->irqs) {
594-
ret = -ENOMEM;
595-
goto fail_free_aplic;
596-
}
597592
for (i = 0; i < aplic->nr_irqs; i++)
598593
raw_spin_lock_init(&aplic->irqs[i].lock);
599594

@@ -606,7 +601,7 @@ int kvm_riscv_aia_aplic_init(struct kvm *kvm)
606601
&aplic->iodev);
607602
mutex_unlock(&kvm->slots_lock);
608603
if (ret)
609-
goto fail_free_aplic_irqs;
604+
goto fail_free_aplic;
610605

611606
/* Setup default IRQ routing */
612607
ret = kvm_riscv_setup_default_irq_routing(kvm, aplic->nr_irqs);
@@ -619,8 +614,6 @@ int kvm_riscv_aia_aplic_init(struct kvm *kvm)
619614
mutex_lock(&kvm->slots_lock);
620615
kvm_io_bus_unregister_dev(kvm, KVM_MMIO_BUS, &aplic->iodev);
621616
mutex_unlock(&kvm->slots_lock);
622-
fail_free_aplic_irqs:
623-
kfree(aplic->irqs);
624617
fail_free_aplic:
625618
kvm->arch.aia.aplic_state = NULL;
626619
kfree(aplic);
@@ -638,8 +631,6 @@ void kvm_riscv_aia_aplic_cleanup(struct kvm *kvm)
638631
kvm_io_bus_unregister_dev(kvm, KVM_MMIO_BUS, &aplic->iodev);
639632
mutex_unlock(&kvm->slots_lock);
640633

641-
kfree(aplic->irqs);
642-
643634
kvm->arch.aia.aplic_state = NULL;
644635
kfree(aplic);
645636
}

0 commit comments

Comments
 (0)