<div dir="ltr">see <a href="http://lists.openembedded.org/pipermail/openembedded-core/2018-May/150654.html">http://lists.openembedded.org/pipermail/openembedded-core/2018-May/150654.html</a></div><br><div class="gmail_quote"><div dir="ltr">On Fri, May 11, 2018 at 12:38 AM Andre McCurdy <<a href="mailto:armccurdy@gmail.com">armccurdy@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, May 10, 2018 at 3:07 PM, Martin Jansa <<a href="mailto:martin.jansa@gmail.com" target="_blank">martin.jansa@gmail.com</a>> wrote:<br>
> On Thu, May 10, 2018 at 11:43:25PM +0200, Martin Jansa wrote:<br>
>> On Thu, May 10, 2018 at 12:27:50PM -0700, Andre McCurdy wrote:<br>
>> > On Thu, May 10, 2018 at 12:11 PM, Martin Jansa <<a href="mailto:martin.jansa@gmail.com" target="_blank">martin.jansa@gmail.com</a>> wrote:<br>
>> > > On Thu, May 10, 2018 at 11:50:02AM -0700, Khem Raj wrote:<br>
>> > >> Hi Martin<br>
>> > >><br>
>> > >> Thanks for testing and reporting back<br>
>> > >><br>
>> > >> On 5/9/18 2:38 AM, Martin Jansa wrote:<br>
>> > >> > My initial tests show couple issues, but usually caused by other changes<br>
>> > >> > in that branch, not the gcc-8 itself.<br>
>> > >> ><br>
>> > >> > 1) strace-4.22 from<br>
>> > >> > <a href="http://git.openembedded.org/openembedded-core-contrib/commit/?h=kraj/gcc-8&id=af33a8b721cc9caebd3f5226b4c5903f666ab654" rel="noreferrer" target="_blank">http://git.openembedded.org/openembedded-core-contrib/commit/?h=kraj/gcc-8&id=af33a8b721cc9caebd3f5226b4c5903f666ab654</a><br>
>> > >> > fails to build with ptest enabled (it builds with 4.20 version if I<br>
>> > >> > revert this change)<br>
>> > >> > ../../strace-4.22/tests/inject-nf.c: In function 'main':<br>
>> > >> > ../../strace-4.22/tests/inject-nf.c:86:1: error: r7 cannot be used in<br>
>> > >> > asm here<br>
>> > >> > }<br>
>> > >> > ^<br>
>> > >><br>
>> > >> are you targeting thumb1 ? how can I reproduce it ?<br>
>> > ><br>
>> > > I'm trying to find out what's different in the builds where it was<br>
>> > > failing, will provide more info later.<br>
>> ><br>
>> > This is probably due to making an inline syscall from Thumb (doesn't a<br>
>> > matter Thumb1 or Thumb2) with frame pointers enabled.<br>
>> ><br>
>> > Does adding -fomit-frame-pointer to CFLAGS fix it?<br>
>><br>
>> It was with raspberrypi3, thumb (Thumb2) enabled in DISTRO, there is<br>
>> already -fno-omit-frame-pointer in the default command line for it,<br>
>> adding -fomit-frame-pointer at the end fixes it:<br>
>><br>
>> docker-lge @ ~/build/build-webos-master/BUILD/work/raspberrypi3-webos-linux-gnueabi/strace/4.22-r0/build/tests $ arm-webos-linux-gnueabi-gcc -march=armv7ve -mthumb -mfpu=neon-vfpv4 -mfloat-abi=hard -mcpu=cortex-a7 --sysroot=/OE/webos/build/build-webos-master/BUILD/work/raspberrypi3-webos-linux-gnueabi/strace/4.22-r0/recipe-sysroot -DHAVE_CONFIG_H -I. -I../linux/arm -I../../strace-4.22/linux/arm -I../linux -I../../strace-4.22/linux -I.. -I../../strace-4.22 -DTESTS_SIZEOF_KERNEL_LONG_T=4 -DTESTS_SIZEOF_LONG=4 -Wall -Wempty-body -Wformat-security -Wignored-qualifiers -Wimplicit-fallthrough=5 -Winit-self -Wlogical-op -Wmissing-parameter-type -Wnested-externs -Wold-style-declaration -Wold-style-definition -Wsign-compare -Wtype-limits -Wwrite-strings -O -fno-omit-frame-pointer -g -feliminate-unused-debug-types -fdebug-prefix-map=/OE/webos/build/build-webos-master/BUILD/work/raspberrypi3-webos-linux-gnueabi/strace/4.22-r0=/usr/src/debug/strace/4.22-r0 -fdebug-prefix-map=/OE/webos/build/build-webos-master/BUILD/work/raspberrypi3-webos-linux-gnueabi/strace/4.22-r0/recipe-sysroot= -fdebug-prefix-map=/OE/webos/build/build-webos-master/BUILD/work/raspberrypi3-webos-linux-gnueabi/strace/4.22-r0/recipe-sysroot-native= -pipe -c -o inject-nf.o ../../strace-4.22/tests/inject-nf.c -fomit-frame-pointer<br>
>><br>
>> This might come from:<br>
>> meta/conf/bitbake.conf:DEBUG_OPTIMIZATION = "-O -fno-omit-frame-pointer ${DEBUG_FLAGS} -pipe"<br>
>> because in this build I had DEBUG_BUILD enabled.<br>
>><br>
>> Will retest with 4.20 version + DEBUG_BUILD to see if it fails there as well now.<br>
><br>
> 4.20 doesn't fail with DEBUG_BUILD, because inject-nf.c test is new in<br>
> 4.22:<br>
><br>
> <a href="https://github.com/strace/strace/commit/58598cd7f6e23e531d71bfe5a4e35f898a4f3b2d#diff-dc01d20c1e55f8adf7536cb46d4481e1" rel="noreferrer" target="_blank">https://github.com/strace/strace/commit/58598cd7f6e23e531d71bfe5a4e35f898a4f3b2d#diff-dc01d20c1e55f8adf7536cb46d4481e1</a><br>
><br>
> What is preferred fix for this? Is adding -fomit-frame-pointer to CFLAGS<br>
> when ptest is in DISTRO_FEATURES acceptable solution?<br>
<br>
I would say unconditionally adding -fomit-frame-pointer to CFLAGS for<br>
all builds is OK (strace is a debug tool - not something that<br>
generally needs to be debugged and frame pointers aren't essential for<br>
debugging anyway).<br>
</blockquote></div>