commit d633a61aabe6f66cbf37342be56303087f768bbb
author Daniel Boulby <daniel.boulby@arm.com> Fri Mar 07 18:08:04 2025 +0000
committer Joao Alves <joao.alves@arm.com> Wed Mar 12 10:16:01 2025 +0100
tree 5b5057a1dbbc90b0acb7c13a3fc889cc94d97dcd
parent a10727d44e423f27cb48004d061b0f0d05d41214

refactor(interrupts): make enable bitmap funcs private

To ensure that the interrupt count is kept in sync when enabling and
disabling secure interrupts make the functions that manipulate the
interrupt_enabled bitmap private.

This requires modifying the unit tests to use the
vcpu_virt_interrupt_enable function so use this opportunity to add a
TearDown function for the vCPU unit tests to make things cleaner.

Signed-off-by: Daniel Boulby <daniel.boulby@arm.com>
Change-Id: Ie707547677199837df023e964c751ffe93fdac9b

src/ffa/hypervisor/interrupts.c

diff --git a/src/ffa/hypervisor/interrupts.c b/src/ffa/hypervisor/interrupts.c
index d056382..1877983 100644
--- a/src/ffa/hypervisor/interrupts.c
+++ b/src/ffa/hypervisor/interrupts.c
@@ -38,15 +38,9 @@
 void ffa_interrupts_enable_virtual_interrupts(struct vcpu_locked current_locked,
 					      struct vm_locked vm_locked)
 {
-	struct vcpu *current;
-	struct interrupts *interrupts;
-
-	current = current_locked.vcpu;
-	interrupts = &current->interrupts;
-
 	if (vm_locked.vm->notifications.enabled) {
-		vcpu_virt_interrupt_set_enabled(interrupts,
-						HF_NOTIFICATION_PENDING_INTID);
+		vcpu_virt_interrupt_enable(current_locked,
+					   HF_NOTIFICATION_PENDING_INTID, true);
 	}
 }