Hi Keerthy,

On Wed, Aug 14, 2019 at 01:53:50PM +0530, Keerthy wrote:
> Eugeniu,
> 
> Could you kindly help me with the steps to reproduce the issue?

Sure. My pleasure. See [0].

> I am sending out a fix patch to convert all the size instances to fdt_size_t
> type.
> 
> I have posted: https://patchwork.ozlabs.org/patch/1146830/
> 
> I believe the above should fix the size mismatch.

The patch appears to fix the problem. Thanks!

[0] Steps to reproduce the issue reported in
    https://patchwork.ozlabs.org/patch/1090094/#2212555

* Apply patch [1] to u-boot/master
* u-boot (master) gcc --version
gcc (Ubuntu 7.4.0-1ubuntu1~16.04~ppa1) 7.4.0
* u-boot (master) make defconfig all -j12
* u-boot (master) ./u-boot -d arch/sandbox/dts/test.dtb

[1] Dirty patch to enable ASAN in sandbox U-Boot (v2019.10-rc2):

diff --git a/Makefile b/Makefile
index 3b0864ae8eaa..b8ef4f267ed5 100644
--- a/Makefile
+++ b/Makefile
@@ -391,7 +391,7 @@ KBUILD_CPPFLAGS := -D__KERNEL__ -D__UBOOT__
 
 KBUILD_CFLAGS   := -Wall -Wstrict-prototypes \
                   -Wno-format-security \
-                  -fno-builtin -ffreestanding $(CSTD_FLAG)
+                  -fno-builtin -ffreestanding $(CSTD_FLAG) -fsanitize=address
 KBUILD_CFLAGS  += -fshort-wchar -fno-strict-aliasing
 KBUILD_AFLAGS   := -D__ASSEMBLY__
 
@@ -768,6 +768,7 @@ PLATFORM_LIBGCC = arch/$(ARCH)/lib/lib.a
 else
 PLATFORM_LIBGCC := -L $(shell dirname `$(CC) $(c_flags) 
-print-libgcc-file-name`) -lgcc
 endif
+PLATFORM_LIBS := -lasan $(PLATFORM_LIBS)
 PLATFORM_LIBS += $(PLATFORM_LIBGCC)
 
 ifdef CONFIG_CC_COVERAGE

-- 
Best Regards,
Eugeniu.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to