Re: [Qemu-devel] [PATCH v13 for-4.1 00/11] qemu_thread_create: propagate the error to callers to handle

[Fei Li]

Hi all,

Sorry for mynegligence.. I will do the same update for the
qemu_thread_create() in test-char.c in next version.

Have a nice day, thanks
Fei

在 2019/3/25 下午10:17, address@hidden 写道:
> Patchew URL: https://patchew.org/QEMU/address@hidden/
>
>
>
> Hi,
>
> This series failed the asan build test. Please find the testing commands and
> their output below. If you have Docker installed, you can probably reproduce it
> locally.
>
> === TEST SCRIPT BEGIN ===
> #!/bin/bash
> time make address@hidden TARGET_LIST=x86_64-softmmu J=14 NETWORK=1
> === TEST SCRIPT END ===
>
> clang -iquote /tmp/qemu-test/build/tests -iquote tests -iquote 
> /tmp/qemu-test/src/tcg -iquote /tmp/qemu-test/src/tcg/i386 
> -I/tmp/qemu-test/src/linux-headers -I/tmp/qemu-test/build/linux-headers -iquote 
> . -iquote /tmp/qemu-test/src -iquote /tmp/qemu-test/src/accel/tcg -iquote 
> /tmp/qemu-test/src/include -I/usr/include/pixman-1  
> -I/tmp/qemu-test/src/dtc/libfdt -Werror -DHAS_LIBSSH2_SFTP_FSYNC  -pthread 
> -I/usr/include/glib-2.0 -I/usr/lib64/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 -std=gnu99  -Wno-string-plus-int 
> -Wno-initializer-overrides -Wexpansion-to-defined -Wendif-labels 
> -Wno-shift-negative-value -Wno-missing-include-dirs -Wempty-body 
> -Wnested-externs -Wformat-security -Wformat-y2k -Winit-self 
> -Wignored-qualifiers -Wold-style-definition -Wtype-limits 
> -fstack-protector-strong  -I/usr/include/p11-kit-1    -I/usr/include/libpng16  
> -I/usr/include/spice-1 -I/usr/include/spice-server -I/usr/include/cacard 
> -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -I/usr/include/nss3 
> -I/usr/include/nspr4 -pthread -I/usr/include/libmount -I/usr/include/blkid 
> -I/usr/include/uuid -I/usr/include/pixman-1  -I/tmp/qemu-test/src/tests -MMD 
> -MP -MT tests/test-blockjob-txn.o -MF tests/test-blockjob-txn.d 
> -fsanitize=undefined -fsanitize=address -g   -c -o tests/test-blockjob-txn.o 
> /tmp/qemu-test/src/tests/test-blockjob-txn.c
> clang -iquote /tmp/qemu-test/build/tests -iquote tests -iquote 
> /tmp/qemu-test/src/tcg -iquote /tmp/qemu-test/src/tcg/i386 
> -I/tmp/qemu-test/src/linux-headers -I/tmp/qemu-test/build/linux-headers -iquote 
> . -iquote /tmp/qemu-test/src -iquote /tmp/qemu-test/src/accel/tcg -iquote 
> /tmp/qemu-test/src/include -I/usr/include/pixman-1  
> -I/tmp/qemu-test/src/dtc/libfdt -Werror -DHAS_LIBSSH2_SFTP_FSYNC  -pthread 
> -I/usr/include/glib-2.0 -I/usr/lib64/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 -std=gnu99  -Wno-string-plus-int 
> -Wno-initializer-overrides -Wexpansion-to-defined -Wendif-labels 
> -Wno-shift-negative-value -Wno-missing-include-dirs -Wempty-body 
> -Wnested-externs -Wformat-security -Wformat-y2k -Winit-self 
> -Wignored-qualifiers -Wold-style-definition -Wtype-limits 
> -fstack-protector-strong  -I/usr/include/p11-kit-1    -I/usr/include/libpng16  
> -I/usr/include/spice-1 -I/usr/include/spice-server -I/usr/include/cacard 
> -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -I/usr/include/nss3 
> -I/usr/include/nspr4 -pthread -I/usr/include/libmount -I/usr/include/blkid 
> -I/usr/include/uuid -I/usr/include/pixman-1  -I/tmp/qemu-test/src/tests -MMD 
> -MP -MT tests/test-block-backend.o -MF tests/test-block-backend.d 
> -fsanitize=undefined -fsanitize=address -g   -c -o tests/test-block-backend.o 
> /tmp/qemu-test/src/tests/test-block-backend.c
> clang -iquote /tmp/qemu-test/build/tests -iquote tests -iquote 
> /tmp/qemu-test/src/tcg -iquote /tmp/qemu-test/src/tcg/i386 
> -I/tmp/qemu-test/src/linux-headers -I/tmp/qemu-test/build/linux-headers -iquote 
> . -iquote /tmp/qemu-test/src -iquote /tmp/qemu-test/src/accel/tcg -iquote 
> /tmp/qemu-test/src/include -I/usr/include/pixman-1  
> -I/tmp/qemu-test/src/dtc/libfdt -Werror -DHAS_LIBSSH2_SFTP_FSYNC  -pthread 
> -I/usr/include/glib-2.0 -I/usr/lib64/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 -std=gnu99  -Wno-string-plus-int 
> -Wno-initializer-overrides -Wexpansion-to-defined -Wendif-labels 
> -Wno-shift-negative-value -Wno-missing-include-dirs -Wempty-body 
> -Wnested-externs -Wformat-security -Wformat-y2k -Winit-self 
> -Wignored-qualifiers -Wold-style-definition -Wtype-limits 
> -fstack-protector-strong  -I/usr/include/p11-kit-1    -I/usr/include/libpng16  
> -I/usr/include/spice-1 -I/usr/include/spice-server -I/usr/include/cacard 
> -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -I/usr/include/nss3 
> -I/usr/include/nspr4 -pthread -I/usr/include/libmount -I/usr/include/blkid 
> -I/usr/include/uuid -I/usr/include/pixman-1  -I/tmp/qemu-test/src/tests -MMD 
> -MP -MT tests/test-block-iothread.o -MF tests/test-block-iothread.d 
> -fsanitize=undefined -fsanitize=address -g   -c -o tests/test-block-iothread.o 
> /tmp/qemu-test/src/tests/test-block-iothread.c
> /tmp/qemu-test/src/tests/test-char.c:795:50: error: too few arguments to 
> function call, expected 6, have 5
>                         addr, QEMU_THREAD_JOINABLE);
>                                                   ^
> /tmp/qemu-test/src/include/qemu/thread.h:155:1: note: 'qemu_thread_create' 
> declared here
> int qemu_thread_create(QemuThread *thread, const char *name,
> ^
> /tmp/qemu-test/src/tests/test-char.c:903:49: error: too few arguments to 
> function call, expected 6, have 5
>                         ioc, QEMU_THREAD_JOINABLE);
>                                                  ^
> /tmp/qemu-test/src/include/qemu/thread.h:155:1: note: 'qemu_thread_create' 
> declared here
> int qemu_thread_create(QemuThread *thread, const char *name,
> ^
> /tmp/qemu-test/src/tests/test-char.c:996:53: error: too few arguments to 
> function call, expected 6, have 5
>                             ioc, QEMU_THREAD_JOINABLE);
>                                                      ^
> /tmp/qemu-test/src/include/qemu/thread.h:155:1: note: 'qemu_thread_create' 
> declared here
>
>
> The full log is available at
> http://patchew.org/logs/address@hidden/testing.asan/?type=message.
> ---
> Email generated automatically by Patchew [https://patchew.org/].
> Please send your feedback to address@hidden