On 14/08/19 3:17 PM, Eugeniu Rosca wrote:
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!
Thanks for testing it. If all your tests have passed please post your Tested-by on that patch.
[0] Steps to reproduce the issue reported in https://patchwork.ozlabs.org/patch/1090094/#2212555
Thanks for the instructions. I will make sure i test this before posting any patch related to drivers/core/of
Best Regards, Keerthy
- 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@@ -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