diff mbox series

memblock tests: Fix testing with 32-bit physical addresses

Message ID 20220304125249.253578-1-karolinadrobnik@gmail.com (mailing list archive)
State New
Headers show
Series memblock tests: Fix testing with 32-bit physical addresses | expand

Commit Message

Karolina Drobnik March 4, 2022, 12:52 p.m. UTC 
Building memblock simulator on x86_64 with 32BIT_PHYS_ADDR_T=1
produces "cast to pointer from integer of different size" warnings.
Fix them by building the binary in 32-bit environment when using
32-bit physical addresses.

Signed-off-by: Karolina Drobnik <karolinadrobnik@gmail.com>
---

This patch is on the top of memblock/for-next.

 tools/testing/memblock/scripts/Makefile.include | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

--
2.30.2

Comments

Mike Rapoport March 6, 2022, 5:04 a.m. UTC | #1 
On Fri, 4 Mar 2022 13:52:49 +0100, Karolina Drobnik wrote:
> Building memblock simulator on x86_64 with 32BIT_PHYS_ADDR_T=1
> produces "cast to pointer from integer of different size" warnings.
> Fix them by building the binary in 32-bit environment when using
> 32-bit physical addresses.
> 
> 

Applied, thanks!

[1/1] memblock tests: Fix testing with 32-bit physical addresses
      commit 2a7ceac9e58167fadc3496c5f694543d4bbe03ef

Best regards,
diff mbox series

Patch

diff --git a/tools/testing/memblock/scripts/Makefile.include b/tools/testing/memblock/scripts/Makefile.include
index 699b0d6cda07..393655310324 100644
--- a/tools/testing/memblock/scripts/Makefile.include
+++ b/tools/testing/memblock/scripts/Makefile.include
@@ -11,7 +11,9 @@  ifeq ($(MOVABLE_NODE), 1)
 	CFLAGS += -D MOVABLE_NODE
 endif

-# Use 32 bit physical addresses
+# Use 32 bit physical addresses.
+# Remember to install 32-bit version of dependencies.
 ifeq ($(32BIT_PHYS_ADDR_T), 1)
-	CFLAGS += -U CONFIG_PHYS_ADDR_T_64BIT
+	CFLAGS += -m32 -U CONFIG_PHYS_ADDR_T_64BIT
+	LDFLAGS += -m32
 endif