Re: [PATCH v3 0/7] dwc-hsotg (aka dwc2) USB host controller emulation

[Paul Zimmerman <pauldzim@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 11123 bytes --]

Just FYI I think I have the fix for this (missing a
DEFINE_PROP_END_OF_LIST()).

Thanks,
Paul


On Mon, Apr 20, 2020 at 8:15 PM <no-reply@patchew.org> wrote:

> Patchew URL:
> https://patchew.org/QEMU/20200421014551.10426-1-pauldzim@gmail.com/
>
>
>
> 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
> export ARCH=x86_64
> make docker-image-fedora V=1 NETWORK=1
> time make docker-test-debug@fedora TARGET_LIST=x86_64-softmmu J=14
> NETWORK=1
> === TEST SCRIPT END ===
>
> PASS 2 check-qlit /qlit/qobject_from_qlit
> MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
> tests/test-qobject-output-visitor -m=quick -k --tap < /dev/null | ./scripts/
> tap-driver.pl --test-name="test-qobject-output-visitor"
> =================================================================
> ==6149==ERROR: AddressSanitizer: global-buffer-overflow on address
> 0x56224c1caf88 at pc 0x5622493e6bb5 bp 0x7ffe5c060dc0 sp 0x7ffe5c060db8
> READ of size 8 at 0x56224c1caf88 thread T0
> PASS 1 test-qobject-output-visitor /visitor/output/int
> PASS 2 test-qobject-output-visitor /visitor/output/bool
> ---
> PASS 32 test-opts-visitor /visitor/opts/range/beyond
> PASS 33 test-opts-visitor /visitor/opts/dict/unvisited
> MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
> tests/test-coroutine -m=quick -k --tap < /dev/null | ./scripts/
> tap-driver.pl --test-name="test-coroutine"
> ==6260==WARNING: ASan doesn't fully support makecontext/swapcontext
> functions and may produce false positives in some cases!
> PASS 1 test-coroutine /basic/no-dangling-access
> PASS 2 test-coroutine /basic/lifecycle
> ==6260==WARNING: ASan is ignoring requested __asan_handle_no_return: stack
> top: 0x7ffd4f2ef000; bottom 0x7f4998ecb000; size: 0x00b3b6424000
> (771856941056)
> False positive error reports may follow
> For details see https://github.com/google/sanitizers/issues/189
> PASS 3 test-coroutine /basic/yield
> ---
> PASS 12 test-aio /aio/event/flush
> PASS 13 test-aio /aio/event/wait/no-flush-cb
> PASS 14 test-aio /aio/timer/schedule
> ==6275==WARNING: ASan doesn't fully support makecontext/swapcontext
> functions and may produce false positives in some cases!
> PASS 15 test-aio /aio/coroutine/queue-chaining
> PASS 16 test-aio /aio-gsource/flush
> PASS 17 test-aio /aio-gsource/bh/schedule
> ---
> PASS 28 test-aio /aio-gsource/timer/schedule
> MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
> tests/test-aio-multithread -m=quick -k --tap < /dev/null | ./scripts/
> tap-driver.pl --test-name="test-aio-multithread"
> PASS 1 test-aio-multithread /aio/multi/lifecycle
> ==6280==WARNING: ASan doesn't fully support makecontext/swapcontext
> functions and may produce false positives in some cases!
> PASS 2 test-aio-multithread /aio/multi/schedule
> PASS 3 test-aio-multithread /aio/multi/mutex/contended
> PASS 4 test-aio-multithread /aio/multi/mutex/handoff
> PASS 5 test-aio-multithread /aio/multi/mutex/mcs
> PASS 6 test-aio-multithread /aio/multi/mutex/pthread
> MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
> tests/test-throttle -m=quick -k --tap < /dev/null | ./scripts/
> tap-driver.pl --test-name="test-throttle"
> ==6314==WARNING: ASan doesn't fully support makecontext/swapcontext
> functions and may produce false positives in some cases!
> PASS 1 test-throttle /throttle/leak_bucket
> PASS 2 test-throttle /throttle/compute_wait
> PASS 3 test-throttle /throttle/init
> ---
> PASS 14 test-throttle /throttle/config/max
> PASS 15 test-throttle /throttle/config/iops_size
> MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
> tests/test-thread-pool -m=quick -k --tap < /dev/null | ./scripts/
> tap-driver.pl --test-name="test-thread-pool"
> ==6318==WARNING: ASan doesn't fully support makecontext/swapcontext
> functions and may produce false positives in some cases!
> PASS 1 test-thread-pool /thread-pool/submit
> PASS 2 test-thread-pool /thread-pool/submit-aio
> PASS 3 test-thread-pool /thread-pool/submit-co
> ---
> PASS 39 test-hbitmap /hbitmap/next_dirty_area/next_dirty_area_4
> PASS 40 test-hbitmap
> /hbitmap/next_dirty_area/next_dirty_area_after_truncate
> MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
> tests/test-bdrv-drain -m=quick -k --tap < /dev/null | ./scripts/
> tap-driver.pl --test-name="test-bdrv-drain"
> ==6390==WARNING: ASan doesn't fully support makecontext/swapcontext
> functions and may produce false positives in some cases!
> PASS 1 test-bdrv-drain /bdrv-drain/nested
> PASS 2 test-bdrv-drain /bdrv-drain/multiparent
> PASS 3 test-bdrv-drain /bdrv-drain/set_aio_context
> ---
> PASS 41 test-bdrv-drain /bdrv-drain/bdrv_drop_intermediate/poll
> PASS 42 test-bdrv-drain /bdrv-drain/replace_child/mid-drain
> MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
> tests/test-bdrv-graph-mod -m=quick -k --tap < /dev/null | ./scripts/
> tap-driver.pl --test-name="test-bdrv-graph-mod"
> ==6429==WARNING: ASan doesn't fully support makecontext/swapcontext
> functions and may produce false positives in some cases!
> PASS 1 test-bdrv-graph-mod /bdrv-graph-mod/update-perm-tree
> PASS 2 test-bdrv-graph-mod /bdrv-graph-mod/should-update-child
> MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
> tests/test-blockjob -m=quick -k --tap < /dev/null | ./scripts/
> tap-driver.pl --test-name="test-blockjob"
> ==6433==WARNING: ASan doesn't fully support makecontext/swapcontext
> functions and may produce false positives in some cases!
> PASS 1 test-blockjob /blockjob/ids
> PASS 2 test-blockjob /blockjob/cancel/created
> PASS 3 test-blockjob /blockjob/cancel/running
> ---
> PASS 7 test-blockjob /blockjob/cancel/pending
> PASS 8 test-blockjob /blockjob/cancel/concluded
> MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
> tests/test-blockjob-txn -m=quick -k --tap < /dev/null | ./scripts/
> tap-driver.pl --test-name="test-blockjob-txn"
> ==6437==WARNING: ASan doesn't fully support makecontext/swapcontext
> functions and may produce false positives in some cases!
> PASS 1 test-blockjob-txn /single/success
> PASS 2 test-blockjob-txn /single/failure
> PASS 3 test-blockjob-txn /single/cancel
> ---
> PASS 6 test-blockjob-txn /pair/cancel
> PASS 7 test-blockjob-txn /pair/fail-cancel-race
> MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
> tests/test-block-backend -m=quick -k --tap < /dev/null | ./scripts/
> tap-driver.pl --test-name="test-block-backend"
> ==6441==WARNING: ASan doesn't fully support makecontext/swapcontext
> functions and may produce false positives in some cases!
> PASS 1 test-block-backend /block-backend/drain_aio_error
> PASS 2 test-block-backend /block-backend/drain_all_aio_error
> MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
> tests/test-block-iothread -m=quick -k --tap < /dev/null | ./scripts/
> tap-driver.pl --test-name="test-block-iothread"
> ==6445==WARNING: ASan doesn't fully support makecontext/swapcontext
> functions and may produce false positives in some cases!
> PASS 1 test-block-iothread /sync-op/pread
> PASS 2 test-block-iothread /sync-op/pwrite
> PASS 3 test-block-iothread /sync-op/load_vmstate
> ---
> PASS 15 test-block-iothread /propagate/diamond
> PASS 16 test-block-iothread /propagate/mirror
> MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
> tests/test-image-locking -m=quick -k --tap < /dev/null | ./scripts/
> tap-driver.pl --test-name="test-image-locking"
> ==6465==WARNING: ASan doesn't fully support makecontext/swapcontext
> functions and may produce false positives in some cases!
> PASS 1 test-image-locking /image-locking/basic
> PASS 2 test-image-locking /image-locking/set-perm-abort
> MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
> tests/test-x86-cpuid -m=quick -k --tap < /dev/null | ./scripts/
> tap-driver.pl --test-name="test-x86-cpuid"
> ---
> PASS 1 test-rcu-tailq /rcu/qtailq/single-threaded
> socket_accept failed: Resource temporarily unavailable
> **
> ERROR:/tmp/qemu-test/src/tests/qtest/libqtest.c:301:qtest_init_without_qmp_handshake:
> assertion failed: (s->fd >= 0 && s->qmp_fd >= 0)
> /tmp/qemu-test/src/tests/qtest/libqtest.c:166: kill_qemu() tried to
> terminate QEMU process but encountered exit status 1 (expected 0)
> ERROR - Bail out!
> ERROR:/tmp/qemu-test/src/tests/qtest/libqtest.c:301:qtest_init_without_qmp_handshake:
> assertion failed: (s->fd >= 0 && s->qmp_fd >= 0)
> make: *** [/tmp/qemu-test/src/tests/Makefile.include:636:
> check-qtest-x86_64] Error 1
> make: *** Waiting for unfinished jobs....
> PASS 2 test-rcu-tailq /rcu/qtailq/short-few
> PASS 3 test-rcu-tailq /rcu/qtailq/long-many
> ---
> PASS 3 test-logging /logging/logfile_write_path
> PASS 4 test-logging /logging/logfile_lock_path
> MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
> tests/test-replication -m=quick -k --tap < /dev/null | ./scripts/
> tap-driver.pl --test-name="test-replication"
> ==6882==WARNING: ASan doesn't fully support makecontext/swapcontext
> functions and may produce false positives in some cases!
> PASS 1 test-replication /replication/primary/read
> PASS 2 test-replication /replication/primary/write
> PASS 3 test-replication /replication/primary/start
> ---
> PASS 6 test-replication /replication/primary/get_error_all
> PASS 7 test-replication /replication/secondary/read
> PASS 8 test-replication /replication/secondary/write
> ==6882==WARNING: ASan is ignoring requested __asan_handle_no_return: stack
> top: 0x7fffab8b6000; bottom 0x7f38b34b0000; size: 0x00c6f8406000
> (854568493056)
> False positive error reports may follow
> For details see https://github.com/google/sanitizers/issues/189
> PASS 9 test-replication /replication/secondary/start
> ---
>     raise CalledProcessError(retcode, cmd)
> subprocess.CalledProcessError: Command '['sudo', '-n', 'docker', 'run',
> '--label', 'com.qemu.instance.uuid=45f88c4b9a644d6bb7f191a23649e083', '-u',
> '1003', '--security-opt', 'seccomp=unconfined', '--rm', '-e',
> 'TARGET_LIST=x86_64-softmmu', '-e', 'EXTRA_CONFIGURE_OPTS=', '-e', 'V=',
> '-e', 'J=14', '-e', 'DEBUG=', '-e', 'SHOW_ENV=', '-e',
> 'CCACHE_DIR=/var/tmp/ccache', '-v',
> '/home/patchew2/.cache/qemu-docker-ccache:/var/tmp/ccache:z', '-v',
> '/var/tmp/patchew-tester-tmp-v1o9n7mt/src/docker-src.2020-04-20-22.43.23.2115:/var/tmp/qemu:z,ro',
> 'qemu:fedora', '/var/tmp/qemu/run', 'test-debug']' returned non-zero exit
> status 2.
>
> filter=--filter=label=com.qemu.instance.uuid=45f88c4b9a644d6bb7f191a23649e083
> make[1]: *** [docker-run] Error 1
> make[1]: Leaving directory `/var/tmp/patchew-tester-tmp-v1o9n7mt/src'
> make: *** [docker-run-test-debug@fedora] Error 2
>
> real    32m12.420s
> user    0m8.663s
>
>
> The full log is available at
>
> http://patchew.org/logs/20200421014551.10426-1-pauldzim@gmail.com/testing.asan/?type=message
> .
> ---
> Email generated automatically by Patchew [https://patchew.org/].
> Please send your feedback to patchew-devel@redhat.com

[-- Attachment #2: Type: text/html, Size: 14295 bytes --]