feat: unify lower EL EA tests
Lower EL External Abort tests were implemented in such a way that after
triggering EA in tftf it gets trapped in EL3 and causing a crash in EL3
Because of the tests ending up in crash there are few problems:
- Need to have to seperate tests one each for sync EA and Serror.
- Unable to test the behaviour of system had the lower EL EA's been
properly handled in EL3
This patch removes two separate configurations into a single test under
Firmware first handling(FFH). TF-A build macro PLATFORM_TEST_EA_FFH
allows fvp to have a proper handling.
This test configuration tests FFH(SCR_EL3.EA = 1) without RAS.
No need to create a new lower el panic test as they are tested in other
scenarios also.
Signed-off-by: Manish Pandey <manish.pandey2@arm.com>
Change-Id: If31ca9ad5f859e219d7cd2482397c52f84587a32
diff --git a/group/tftf-l2-fvp/fvp-lower-el-ea,fvp-inject-serror:fvp-tftf-fip.tftf-aemv8a.crash-debug b/group/tftf-l2-fvp/fvp-ea-ffh,fvp-ea-ffh:fvp-tftf-fip.tftf-aemv8a-debug
similarity index 100%
rename from group/tftf-l2-fvp/fvp-lower-el-ea,fvp-inject-serror:fvp-tftf-fip.tftf-aemv8a.crash-debug
rename to group/tftf-l2-fvp/fvp-ea-ffh,fvp-ea-ffh:fvp-tftf-fip.tftf-aemv8a-debug
diff --git a/group/tftf-l2-fvp/fvp-lower-el-ea,fvp-inject-syncEA:fvp-tftf-fip.tftf-aemv8a.crash-debug b/group/tftf-l2-fvp/fvp-lower-el-ea,fvp-inject-syncEA:fvp-tftf-fip.tftf-aemv8a.crash-debug
deleted file mode 100644
index 8c2d647..0000000
--- a/group/tftf-l2-fvp/fvp-lower-el-ea,fvp-inject-syncEA:fvp-tftf-fip.tftf-aemv8a.crash-debug
+++ /dev/null
@@ -1,5 +0,0 @@
-#
-# Copyright (c) 2023, Arm Limited. All rights reserved.
-#
-# SPDX-License-Identifier: BSD-3-Clause
-#
diff --git a/script/tf-coverity/tf-cov-make b/script/tf-coverity/tf-cov-make
index 2b34065..a3f790e 100755
--- a/script/tf-coverity/tf-cov-make
+++ b/script/tf-coverity/tf-cov-make
@@ -150,6 +150,9 @@
FAULT_INJECTION_SUPPORT=1 HANDLE_EA_EL3_FIRST_NS=1 RAS_EXTENSION=1 \
SDEI_SUPPORT=1
+# EA handled in EL3 first
+clean_build $fvp_common_flags HANDLE_EA_EL3_FIRST_NS=1 PLATFORM_TEST_EA_FFH=1
+
# Hardware Assisted Coherency(DynamIQ)
clean_build $fvp_common_flags FVP_CLUSTER_COUNT=1 FVP_MAX_CPUS_PER_CLUSTER=8 \
HW_ASSISTED_COHERENCY=1 USE_COHERENT_MEM=0
diff --git a/tf_config/fvp-ea-ffh b/tf_config/fvp-ea-ffh
new file mode 100644
index 0000000..e7a71d1
--- /dev/null
+++ b/tf_config/fvp-ea-ffh
@@ -0,0 +1,4 @@
+CROSS_COMPILE=aarch64-none-elf-
+HANDLE_EA_EL3_FIRST_NS=1
+PLATFORM_TEST_EA_FFH=1
+PLAT=fvp
diff --git a/tf_config/fvp-lower-el-ea b/tf_config/fvp-lower-el-ea
deleted file mode 100644
index c4d6a5d..0000000
--- a/tf_config/fvp-lower-el-ea
+++ /dev/null
@@ -1,6 +0,0 @@
-CRASH_REPORTING=1
-CROSS_COMPILE=aarch64-none-elf-
-EL3_EXCEPTION_HANDLING=1
-ENABLE_BACKTRACE=1
-HANDLE_EA_EL3_FIRST_NS=1
-PLAT=fvp
diff --git a/tftf_config/fvp-inject-syncEA b/tftf_config/fvp-ea-ffh
similarity index 67%
rename from tftf_config/fvp-inject-syncEA
rename to tftf_config/fvp-ea-ffh
index 174a8b0..61c0777 100644
--- a/tftf_config/fvp-inject-syncEA
+++ b/tftf_config/fvp-ea-ffh
@@ -1,3 +1,3 @@
CROSS_COMPILE=aarch64-none-elf-
PLAT=fvp
-TESTS=inject-syncEA
+TESTS=ea-ffh
diff --git a/tftf_config/fvp-inject-serror b/tftf_config/fvp-inject-serror
deleted file mode 100644
index f1721f0..0000000
--- a/tftf_config/fvp-inject-serror
+++ /dev/null
@@ -1,3 +0,0 @@
-CROSS_COMPILE=aarch64-none-elf-
-PLAT=fvp
-TESTS=inject-serror