Patchew URL: https://patchew.org/QEMU/20200522160755.886-1-robert.fo...@linaro.org/
Hi, This series seems to have some coding style problems. See output below for more information: Message-id: 20200522160755.886-1-robert.fo...@linaro.org Subject: [PATCH 00/19] Add Thread Sanitizer support to QEMU Type: series === TEST SCRIPT BEGIN === #!/bin/bash git rev-parse base > /dev/null || exit 0 git config --local diff.renamelimit 0 git config --local diff.renames True git config --local diff.algorithm histogram ./scripts/checkpatch.pl --mailback base.. === TEST SCRIPT END === Switched to a new branch 'test' 1737663 docs: Added details on TSan to testing.rst 82aa460 target/arm: Fix tsan warning in cpu.c 4a3bd5a util: Added tsan annotate for thread name. f2614bb util: fixed tsan warnings in thread_pool.c 56529c7 qht: Fix tsan warnings. 128f63c util/async: Fixed tsan warnings f86c38c accel/tcg: Fixed tsan warnings. 0d7ee16 configure: added tsan support for blacklist. cfb2d34 accel/tcg: Fixed tsan warnings related to parallel_cpus 1ef1ed2 include/qemu: Added tsan.h for annotations. bd287e9 tests/docker: Added docker build support for TSan. 6baf0d3 thread: add tsan annotations to QemuSpin bbf88d9 translate-all: call qemu_spin_destroy for PageDesc 5f0a213 tcg: call qemu_spin_destroy for tb->jmp_lock fb19649 qht: call qemu_spin_destroy for head buckets 688ca64 cputlb: destroy CPUTLB with tlb_destroy be8d1f8 thread: add qemu_spin_destroy 2a326b6 cpu: convert queued work to a QSIMPLEQ 7fb7830 configure: add --enable-tsan flag + fiber annotations for coroutine-ucontext === OUTPUT BEGIN === 1/19 Checking commit 7fb7830797be (configure: add --enable-tsan flag + fiber annotations for coroutine-ucontext) 2/19 Checking commit 2a326b6f7215 (cpu: convert queued work to a QSIMPLEQ) 3/19 Checking commit be8d1f8ff517 (thread: add qemu_spin_destroy) 4/19 Checking commit 688ca64764bf (cputlb: destroy CPUTLB with tlb_destroy) 5/19 Checking commit fb19649f7025 (qht: call qemu_spin_destroy for head buckets) 6/19 Checking commit 5f0a21365e6e (tcg: call qemu_spin_destroy for tb->jmp_lock) 7/19 Checking commit bbf88d92575a (translate-all: call qemu_spin_destroy for PageDesc) 8/19 Checking commit 6baf0d37cfdf (thread: add tsan annotations to QemuSpin) 9/19 Checking commit bd287e96cf4a (tests/docker: Added docker build support for TSan.) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #74: new file mode 100644 total: 0 errors, 1 warnings, 118 lines checked Patch 9/19 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 10/19 Checking commit 1ef1ed22be4b (include/qemu: Added tsan.h for annotations.) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #18: new file mode 100644 total: 0 errors, 1 warnings, 48 lines checked Patch 10/19 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 11/19 Checking commit cfb2d343b6ed (accel/tcg: Fixed tsan warnings related to parallel_cpus) 12/19 Checking commit 0d7ee16ffe83 (configure: added tsan support for blacklist.) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #28: new file mode 100644 total: 0 errors, 1 warnings, 14 lines checked Patch 12/19 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 13/19 Checking commit f86c38c6aa87 (accel/tcg: Fixed tsan warnings.) 14/19 Checking commit 128f63c37b76 (util/async: Fixed tsan warnings) 15/19 Checking commit 56529c7ff837 (qht: Fix tsan warnings.) 16/19 Checking commit f2614bbafdb4 (util: fixed tsan warnings in thread_pool.c) 17/19 Checking commit 4a3bd5a64414 (util: Added tsan annotate for thread name.) 18/19 Checking commit 82aa460722b3 (target/arm: Fix tsan warning in cpu.c) 19/19 Checking commit 17376630a146 (docs: Added details on TSan to testing.rst) ERROR: trailing whitespace #34: FILE: docs/devel/testing.rst:413: + $ ERROR: trailing whitespace #40: FILE: docs/devel/testing.rst:419: +the files with TSan warnings. $ total: 2 errors, 0 warnings, 78 lines checked Patch 19/19 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20200522160755.886-1-robert.fo...@linaro.org/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-de...@redhat.com