common: memsize: fix occasionally failing alias probingtoradex_imx_lf_v2025.04

probe_ram_size_by_alias() detects whether a probe address still aliases
a lower address by writing through one address and reading through the
other.

On i.MX95 this occasionally reported a false non-alias when the alias
read happened immediately after the write.

A memory barrier alone, mb(), was tested but did not make the failure go
away. This suggests that ordering the CPU accesses is not sufficient for
this probe, likely because the issue is in the path to the memory
controller rather than in the core itself.

Read the written address back before checking the alias address. This
appears to force the write to become observable at the probe address
before using the alias read to decide whether the tested address range
exists.

If the readback does not match the written pattern, restore the saved
value and continue with the next check. This keeps the probe robust for
addresses that do not reliably retain the test pattern.

Upstream-Status: Submitted [https://lore.kernel.org/u-boot/20260429100400.2860345-1-ghidoliemanuele@gmail.com/]
Fixes: 0977448b45e2 ("common: memsize: add RAM size probe based on alias detection")
Signed-off-by: Emanuele Ghidoli <emanuele.ghidoli@toradex.com>

1 files changed, 18 insertions, 2 deletions

diff --git a/common/memsize.c b/common/memsize.c
index 4c8e34e2b09..f82696c72c3 100644
--- a/common/memsize.c
+++ b/common/memsize.c
@@ -134,6 +134,7 @@ long get_ram_size(long *base, long maxsize)
 long probe_ram_size_by_alias(const struct ram_alias_check *checks)
 {
 	long save[2];
+	long pat;
 	int dcache_en = dcache_status();
 	long ret = 0;
 
@@ -147,12 +148,27 @@ long probe_ram_size_by_alias(const struct ram_alias_check *checks)
 		if (dcache_en)
 			dcache_flush_invalidate(s);
 
-		*d = ~save[0];
+		pat = ~save[0];
+		*d = pat;
 		sync();
 		if (dcache_en)
 			dcache_flush_invalidate(d);
 
-		if (*s != ~save[0])
+		/*
+		 * Make sure the test pattern is observable at the probe
+		 * address before checking whether it is also visible through
+		 * the alias address.
+		 */
+		if (*d != pat) {
+			*d = save[1];
+			sync();
+			if (dcache_en)
+				dcache_flush_invalidate(d);
+			checks++;
+			continue;
+		}
+
+		if (*s != pat)
 			ret = checks->size;
 
 		/* Restore content */