chore(hftest): use common secondary stack definition
Have multicore tests in different test setups refer to a common
definition of `secondary_ec_stack` for secondary execution context
stack allocations. Refactor `hftest_cpu_start` to no longer
receive `stack` and `stack_size` as arguments and to instead refer to
this common stack definition.
Signed-off-by: Kathleen Capella <kathleen.capella@arm.com>
Change-Id: I600f4c73e1c615c83a3c69003ce725087e066eab
diff --git a/test/vmapi/ffa_secure_partitions/secure_interrupts.c b/test/vmapi/ffa_secure_partitions/secure_interrupts.c
index 127c9be..080183b 100644
--- a/test/vmapi/ffa_secure_partitions/secure_interrupts.c
+++ b/test/vmapi/ffa_secure_partitions/secure_interrupts.c
@@ -24,8 +24,6 @@
#define LAST_SECONDARY_VCPU_ID (MAX_CPUS - 1)
#define MID_SECONDARY_VCPU_ID (MAX_CPUS / 2)
-alignas(4096) static uint8_t secondary_ec_stack[MAX_CPUS - 1][PAGE_SIZE];
-
struct secondary_cpu_entry_args {
ffa_id_t receiver_id;
ffa_vcpu_count_t vcpu_count;
@@ -551,9 +549,7 @@
args.vcpu_id = i;
HFTEST_LOG("Booting CPU %u - %x", i, cpu_id);
- EXPECT_EQ(hftest_cpu_start(cpu_id, secondary_ec_stack[i - 1],
- sizeof(secondary_ec_stack[0]),
- cpu_entry_sp_sleep_loop,
+ EXPECT_EQ(hftest_cpu_start(cpu_id, cpu_entry_sp_sleep_loop,
(uintptr_t)&args),
true);