summaryrefslogtreecommitdiff
path: root/spm/cactus/plat/arm/fvp/fdts/cactus.dts
diff options
context:
space:
mode:
authorJ-Alves <joao.alves@arm.com>2021-07-28 13:57:09 +0100
committerJ-Alves <joao.alves@arm.com>2021-10-04 09:17:20 +0100
commit60ea4c3507f00781be41822271ba869da28d997e (patch)
tree36c74bab6c3f0cf70c37ba68505e645a12ab7522 /spm/cactus/plat/arm/fvp/fdts/cactus.dts
parent912ab1a418d7cb6b79ab8cef7b726eb432d289d7 (diff)
fix(cactus): memory sharing tests failure MP
The last test to share memory functionality tested the FFA_MEMORY_DONATE interface. After donating the memory, the original owner didn't update its stage-1 translation at EL1. Hence, when booting a MP setup, there would be a data abort, as at stage-2 translation the memory region is now owned by the receiver of the test. Share memory region was being allocated part of the BSS section, was being mapped as a static region in the xlat libraries. A static region is not meant to be unmapped. To fix the above points: - allocated a memory region in the partition manifest of all Cactus SPs for the sake of the memory sharing tests. - at stage-1 before using it for the first memory share operation, the sender would map its segment as a dynamic region. - if test included a FFA_MEMORY_DONATE call, after donation the owner unmaps the memory region from its stage-1 translation. Signed-off-by: J-Alves <joao.alves@arm.com> Change-Id: I039c0d0d1633688b0e9d6ba1e2bbebfd518fc72b
Diffstat (limited to 'spm/cactus/plat/arm/fvp/fdts/cactus.dts')
-rw-r--r--spm/cactus/plat/arm/fvp/fdts/cactus.dts8
1 files changed, 8 insertions, 0 deletions
diff --git a/spm/cactus/plat/arm/fvp/fdts/cactus.dts b/spm/cactus/plat/arm/fvp/fdts/cactus.dts
index 7d74e1e..759adc4 100644
--- a/spm/cactus/plat/arm/fvp/fdts/cactus.dts
+++ b/spm/cactus/plat/arm/fvp/fdts/cactus.dts
@@ -57,6 +57,14 @@
attributes = <0x3>; /* read-write */
};
+ /* Memory to be shared in memory sharing tests. */
+ share-memory {
+ description = "share-memory";
+ pages-count = <1>;
+ base-address =<0x00000000 0x7500000>;
+ attributes = <0x3>; /* read-write */
+ };
+
/* Without optional base-address */
test-memory {
description = "test-memory";