Re: [PATCH v6 03/11] arm64: realm: Query IPA size from the RMM

From: Gavin Shan
Date: Mon Oct 07 2024 - 19:33:42 EST


On 10/5/24 12:42 AM, Steven Price wrote:
The top bit of the configured IPA size is used as an attribute to
control whether the address is protected or shared. Query the
configuration from the RMM to assertain which bit this is.

Reviewed-by: Catalin Marinas <catalin.marinas@xxxxxxx>
Reviewed-by: Gavin Shan <gshan@xxxxxxxxxx>
Co-developed-by: Suzuki K Poulose <suzuki.poulose@xxxxxxx>
Signed-off-by: Suzuki K Poulose <suzuki.poulose@xxxxxxx>
Signed-off-by: Steven Price <steven.price@xxxxxxx>
---
Changes since v4:
* Make PROT_NS_SHARED check is_realm_world() to reduce impact on
non-CCA systems.
Changes since v2:
* Drop unneeded extra brackets from PROT_NS_SHARED.
* Drop the explicit alignment from 'config' as struct realm_config now
specifies the alignment.
---
arch/arm64/include/asm/pgtable-prot.h | 4 ++++
arch/arm64/include/asm/rsi.h | 2 +-
arch/arm64/kernel/rsi.c | 8 ++++++++
3 files changed, 13 insertions(+), 1 deletion(-)


[...]

diff --git a/arch/arm64/include/asm/rsi.h b/arch/arm64/include/asm/rsi.h
index e4c01796c618..acba065eb00e 100644
--- a/arch/arm64/include/asm/rsi.h
+++ b/arch/arm64/include/asm/rsi.h
@@ -27,7 +27,7 @@ static inline int rsi_set_memory_range(phys_addr_t start, phys_addr_t end,
while (start != end) {
ret = rsi_set_addr_range_state(start, end, state, flags, &top);
- if (WARN_ON(ret || top < start || top > end))
+ if (ret || top < start || top > end)
return -EINVAL;
start = top;
}

I think the changes belong to PATCH[02/11] :)

Thanks,
Gavin