[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 00/13] Add Thread Sanitizer support to QEMU
From: |
Alex Bennée |
Subject: |
Re: [PATCH v2 00/13] Add Thread Sanitizer support to QEMU |
Date: |
Mon, 08 Jun 2020 16:42:01 +0100 |
User-agent: |
mu4e 1.5.2; emacs 28.0.50 |
Robert Foley <robert.foley@linaro.org> writes:
> v1: https://lists.gnu.org/archive/html/qemu-devel/2020-05/msg08302.html
>
> Changes in v2:
> - Fixed make check under TSan. With the below fixes, make check
> under TSan completes successfully, albeit with TSan warnings.
> - We found that several unit tests and the qtests hit an issue in TSan,
> which results in a hung test. This is a known issue:
> https://github.com/google/sanitizers/issues/1116
> - Under TSan, disable the 3 unit tests that hit this above issue.
> - Under TSan, disable the qtests since they hit this issue too.
> - Split out the docker testing for tsan into its own test (test-tsan).
> - configure: Error out if tsan and other sanitizers are used together.
> - configure: Cleaned up warnings during tsan build caused by tsan libraries.
<snip>
I've complete my pass. I think we are looking pretty good and once the
tracepoints and function pointer stuff is dealt with I think we are
ready to merge.
--
Alex Bennée
- Re: [PATCH v2 09/13] tests/docker: Added docker build support for TSan., (continued)
- [PATCH v2 10/13] include/qemu: Added tsan.h for annotations., Robert Foley, 2020/06/05
- [PATCH v2 11/13] util: Added tsan annotate for thread name., Robert Foley, 2020/06/05
- [PATCH v2 12/13] docs: Added details on TSan to testing.rst, Robert Foley, 2020/06/05
- [PATCH v2 13/13] tests: Disable select tests under TSan, which hit TSan issue., Robert Foley, 2020/06/05
- Re: [PATCH v2 00/13] Add Thread Sanitizer support to QEMU, no-reply, 2020/06/05
- Re: [PATCH v2 00/13] Add Thread Sanitizer support to QEMU,
Alex Bennée <=