Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/69231 )
Change subject: arch/x86/memmove.c: Add temporary fix for 64bit memmove ......................................................................
arch/x86/memmove.c: Add temporary fix for 64bit memmove
Change-Id: Ib43ec19df97194d6b1c18bfacb5fe8211ba0ffe5 Signed-off-by: Arthur Heymans arthur@aheymans.xyz --- M src/arch/x86/memmove.c 1 file changed, 15 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/31/69231/1
diff --git a/src/arch/x86/memmove.c b/src/arch/x86/memmove.c index 3ec50b2..b05f03b 100644 --- a/src/arch/x86/memmove.c +++ b/src/arch/x86/memmove.c @@ -190,9 +190,11 @@ "11:" : "=&c" (d0), "=&S" (d1), "=&D" (d2), "=r" (d3), "=r" (d4), "=r"(d5) - : "0" (n), - "1" (src), - "2" (dest) + /* TODO: port proper memmove from Linux for 64bit. + For now it is ok as all coreboot core remains below 4G */ + : "0" ((uint32_t)(uintptr_t)n), + "1" ((uint32_t)(uintptr_t)src), + "2" ((uint32_t)(uintptr_t)dest) : "memory");
return ret;