Re: [Qemu-devel] [PATCH] docker: add sanitizers back to clang build

2019-09-19 Thread John Snow
t;> >>> Hi, >>> >>> This series seems to have some coding style problems. See output below for >>> more information: >>> >>> Subject: [Qemu-devel] [PATCH] docker: add sanitizers back to clang build >>> Message-id: 20190912014442.5757-1

Re: [Qemu-devel] [PATCH] docker: add sanitizers back to clang build

2019-09-17 Thread Fam Zheng
some coding style problems. See output below for > > more information: > > > > Subject: [Qemu-devel] [PATCH] docker: add sanitizers back to clang build > > Message-id: 20190912014442.5757-1-js...@redhat.com > > Type: series > > > > === TEST S

Re: [Qemu-devel] [PATCH] docker: add sanitizers back to clang build

2019-09-12 Thread John Snow
On 9/11/19 9:52 PM, no-re...@patchew.org wrote: > Patchew URL: https://patchew.org/QEMU/20190912014442.5757-1-js...@redhat.com/ > > > > Hi, > > This series seems to have some coding style problems. See output below for > more information: > > Subject: [Q

Re: [Qemu-devel] [PATCH] docker: add sanitizers back to clang build

2019-09-11 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20190912014442.5757-1-js...@redhat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [PATCH] docker: add sanitizers back to clang build Message-id: 20190912014442.5757-1-js

[Qemu-devel] [PATCH] docker: add sanitizers back to clang build

2019-09-11 Thread John Snow
Fedora23 is but a distant twinkle. The sanitizer works again, and even if not, we have --enable-sanitizers now. --- The problem is that I do see some ASAN warnings: TESTcheck-qtest-aarch64: tests/device-introspect-test Direct leak of 2272 byte(s) in 48 object(s) allocated from: #0