Emilio G Cota writes: > On Mon, Sep 25, 2017 at 21:07:45 +0300, Lluís Vilanova wrote: >> You're doing it right, and I've checked that the branch is properly pushed. >> Can >> you compile with V=1 to show me the failing cmdline?
> $ make V=1 > (cd /data/src/qemu2; printf '#define QEMU_PKGVERSION '; if test -n ""; then > printf '""\n'; else if test -d .git; then printf '" ('; git describe --match > 'v*' 2>/dev/null | tr -d '\n'; if ! git diff-index --quiet HEAD &>/dev/null; > then printf -- '-dirty'; fi; printf ')"\n'; else printf '""\n'; fi; fi) > > qemu-version.h.tmp > if ! cmp -s qemu-version.h qemu-version.h.tmp; then mv qemu-version.h.tmp > qemu-version.h; else rm qemu-version.h.tmp; fi > make -I/data/src/qemu2/dtc VPATH=/data/src/qemu2/dtc -C dtc V="1" > LIBFDT_srcdir=/data/src/qemu2/dtc/libfdt CPPFLAGS="-I/data/src/qemu2/build/dtc > -I/data/src/qemu2/dtc -I/data/src/qemu2/dtc/libfdt" CFLAGS="-O2 > -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -g -fvisibility=hidden > -I/usr/include/pixman-1 -I/data/src/qemu2/dtc/libfdt -Werror -pthread > -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -fPIE > -DPIE > -m64 -mcx16 -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE > -Wstrict-prototypes -Wredundant-decls -Wall -Wundef -Wwrite-strings > -Wmissing-prototypes -fno-strict-aliasing -fno-common -fwrapv -Wendif-labels > -Wno-missing-include-dirs -Wempty-body -Wnested-externs -Wformat-security > -Wformat-y2k -Winit-self -Wignored-qualifiers -Wold-style-declaration > -Wold-style-definition -Wtype-limits -fstack-protector-strong > -I/usr/include/libpng12 -I/data/src/qemu2/tests" LDFLAGS="-rdynamic > -Wl,--warn-common -Wl,-z,relro -Wl,-z,now -pie -m64 -g " ARFLAGS="rv" CC="cc" > AR="ar" LD="ld" BUILD_DIR=/data/src/qemu2/build libfdt/libfdt.a > make[1]: Entering directory '/data/src/qemu2/build/dtc' > make[1]: 'libfdt/libfdt.a' is up to date. > make[1]: Leaving directory '/data/src/qemu2/build/dtc' > make BUILD_DIR=/data/src/qemu2/build -C x86_64-linux-user V="1" > TARGET_DIR="x86_64-linux-user/" all > make[1]: Entering directory '/data/src/qemu2/build/x86_64-linux-user' > cc -I/data/src/qemu2/build/. -I. -I/data/src/qemu2/tcg > -I/data/src/qemu2/tcg/i386 -I/data/src/qemu2/instrument > -I/data/src/qemu2/linux-headers -I/data/src/qemu2/build/linux-headers > -I. -I/data/src/qemu2 -I/data/src/qemu2/accel/tcg -I/data/src/qemu2/include > -fvisibility=hidden -I/usr/include/pixman-1 -I/data/src/qemu2/dtc/libfdt > -Werror > -pthread -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include > -fPIE -DPIE -m64 -mcx16 -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 > -D_LARGEFILE_SOURCE > -Wstrict-prototypes -Wredundant-decls -Wall -Wundef -Wwrite-strings > -Wmissing-prototypes -fno-strict-aliasing -fno-common -fwrapv -Wendif-labels > -Wno-missing-include-dirs -Wempty-body -Wnested-externs -Wformat-security > -Wformat-y2k -Winit-self -Wignored-qualifiers -Wold-style-declaration > -Wold-style-definition -Wtype-limits -fstack-protector-strong > -I/usr/include/libpng12 -I../linux-headers -I.. -I/data/src/qemu2/target/i386 > -DNEED_CPU_H -I/data/src/qemu2/include -I/data/src/qemu2/linux-user/x86_64 > -I/data/src/qemu2/linux-user/host/x86_64 -I/data/src/qemu2/linux-user -MMD -MP > -MT gdbstub-xml.o -MF ./gdbstub-xml.d -O2 -U_FORTIFY_SOURCE > -D_FORTIFY_SOURCE=2 > -g -DQEMU_TARGET_BUILD=1 -c -o gdbstub-xml.o gdbstub-xml.c > cc -I/data/src/qemu2/build/instrument -Iinstrument -I/data/src/qemu2/tcg > -I/data/src/qemu2/tcg/i386 -I/data/src/qemu2/instrument > -I/data/src/qemu2/linux-headers -I/data/src/qemu2/build/linux-headers > -I. -I/data/src/qemu2 -I/data/src/qemu2/accel/tcg -I/data/src/qemu2/include > -fvisibility=hidden -I/usr/include/pixman-1 -I/data/src/qemu2/dtc/libfdt > -Werror > -pthread -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include > -fPIE -DPIE -m64 -mcx16 -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 > -D_LARGEFILE_SOURCE > -Wstrict-prototypes -Wredundant-decls -Wall -Wundef -Wwrite-strings > -Wmissing-prototypes -fno-strict-aliasing -fno-common -fwrapv -Wendif-labels > -Wno-missing-include-dirs -Wempty-body -Wnested-externs -Wformat-security > -Wformat-y2k -Winit-self -Wignored-qualifiers -Wold-style-declaration > -Wold-style-definition -Wtype-limits -fstack-protector-strong > -I/usr/include/libpng12 -I../linux-headers -I.. -I/data/src/qemu2/target/i386 > -DNEED_CPU_H -I/data/src/qemu2/include -I/data/src/qemu2/linux-user/x86_64 > -I/data/src/qemu2/linux-user/host/x86_64 -I/data/src/qemu2/linux-user -MMD -MP > -MT instrument/trace.o -MF instrument/trace.d -O2 -U_FORTIFY_SOURCE > -D_FORTIFY_SOURCE=2 -g -DQEMU_TARGET_BUILD=1 -c -o instrument/trace.o > /data/src/qemu2/instrument/trace.c > /data/src/qemu2/instrument/trace.c:13:30: fatal error: qemu-instr/trace.h: No > such file or directory > compilation terminated. > /data/src/qemu2/rules.mak:66: recipe for target 'instrument/trace.o' failed > make[1]: *** [instrument/trace.o] Error 1 > make[1]: Leaving directory '/data/src/qemu2/build/x86_64-linux-user' > Makefile:326: recipe for target 'subdir-x86_64-linux-user' failed > make: *** [subdir-x86_64-linux-user] Error 2 The includes look fine to me... (there's "-I/data/src/qemu2/instrument"). Can you paste the result of "find /data/src/qemu2/instrument"? You should have a "/data/src/qemu2/instrument/qemu-instr/trace.h" file there. In any case, I was planning to drop the tracing API for the next series. Thanks, Lluis