Hi On Fri, Feb 23, 2018 at 12:37 PM, Paolo Bonzini <pbonz...@redhat.com> wrote: > On 15/02/2018 22:25, Marc-André Lureau wrote: >> Hi, >> >> The following patches fix a regression introduced in commit >> 218bb57dd79d that prevent ASAN from being detected & used. There is >> also a works around for a GCC ASAN optimization bug. A few test leaks >> are fixed, and a few patches reenable vhost-user memfd test fixing the >> recent race bug that was identified in the previous version. >> >> Marc-André Lureau (6): >> build-sys: fix -fsanitize=address check >> lockable: workaround GCC link issue with ASAN >> vhost-user-test: add back memfd check >> vhost-user-test: do not hang if chardev creation failed >> ahci-test: fix opts leak of skip tests >> sdhci-test: fix leaks >> >> include/qemu/lockable.h | 2 +- >> tests/ahci-test.c | 1 + >> tests/sdhci-test.c | 2 ++ >> tests/vhost-user-test.c | 94 >> +++++++++++++++++++++++++++++++++++-------------- >> configure | 22 ++++++------ >> 5 files changed, 83 insertions(+), 38 deletions(-) >> > > Queued 1-2-5-6, not really confident enough in what's going on with > vhost-user-test. :)
It hangs when chardev creation fails. Which is apparent when running with --debug-log. That's also why I added "vhost-user-test: do not hang if chardev creation failed". The problem comes from reusing socket location without waiting for the idle cleanup. Using different socket path solves it (vhost-user-test: add back memfd check) Maxime, could you help review those patches? thanks -- Marc-André Lureau
