Hi, This series seems to have some coding style problems. See output below for more information:
Type: series Message-id: 20181106110548.4209-1-luc.mic...@greensocs.com Subject: [Qemu-devel] [PATCH v4 00/16] gdbstub: support for the multiprocess extension === TEST SCRIPT BEGIN === #!/bin/bash BASE=base n=1 total=$(git log --oneline $BASE.. | wc -l) failed=0 git config --local diff.renamelimit 0 git config --local diff.renames True git config --local diff.algorithm histogram commits="$(git log --format=%H --reverse $BASE..)" for c in $commits; do echo "Checking PATCH $n/$total: $(git log -n 1 --format=%s $c)..." if ! git show $c --format=email | ./scripts/checkpatch.pl --mailback -; then failed=1 echo fi n=$((n+1)) done exit $failed === TEST SCRIPT END === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 Switched to a new branch 'test' 1e2778d39e arm/xlnx-zynqmp: put APUs and RPUs in separate CPU clusters eed16df098 gdbstub: add multiprocess extension support e88cd44bd3 gdbstub: gdb_set_stop_cpu: ignore request when process is not attached aa2fa0a3fa gdbstub: processes initialization on new peer connection 4afe5a152f gdbstub: add support for vAttach packets a7fd6581c5 gdbstub: add support for extended mode packet 5998191123 gdbstub: add multiprocess support to 'D' packets 244b52d66c gdbstub: add multiprocess support to gdb_vm_state_change() 1422ee16bb gdbstub: add multiprocess support to Xfer:features:read: be15097ae2 gdbstub: add multiprocess support to (f|s)ThreadInfo and ThreadExtraInfo c0410a7b7d gdbstub: add multiprocess support to 'sC' packets 5811d90600 gdbstub: add multiprocess support to vCont packets cc51ac6f1b gdbstub: add multiprocess support to 'H' and 'T' packets fc28bafab2 gdbstub: add multiprocess support to '?' packets 863423637d gdbstub: introduce GDB processes 1bf8b223d1 hw/cpu: introduce CPU clusters === OUTPUT BEGIN === Checking PATCH 1/16: hw/cpu: introduce CPU clusters... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #26: new file mode 100644 ERROR: do not initialise statics to 0 or NULL #58: FILE: hw/cpu/cluster.c:28: + static uint64_t cluster_id_auto_increment = 0; total: 1 errors, 1 warnings, 102 lines checked Your patch has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. Checking PATCH 2/16: gdbstub: introduce GDB processes... Checking PATCH 3/16: gdbstub: add multiprocess support to '?' packets... Checking PATCH 4/16: gdbstub: add multiprocess support to 'H' and 'T' packets... Checking PATCH 5/16: gdbstub: add multiprocess support to vCont packets... Checking PATCH 6/16: gdbstub: add multiprocess support to 'sC' packets... Checking PATCH 7/16: gdbstub: add multiprocess support to (f|s)ThreadInfo and ThreadExtraInfo... Checking PATCH 8/16: gdbstub: add multiprocess support to Xfer:features:read:... Checking PATCH 9/16: gdbstub: add multiprocess support to gdb_vm_state_change()... Checking PATCH 10/16: gdbstub: add multiprocess support to 'D' packets... Checking PATCH 11/16: gdbstub: add support for extended mode packet... Checking PATCH 12/16: gdbstub: add support for vAttach packets... Checking PATCH 13/16: gdbstub: processes initialization on new peer connection... Checking PATCH 14/16: gdbstub: gdb_set_stop_cpu: ignore request when process is not attached... Checking PATCH 15/16: gdbstub: add multiprocess extension support... Checking PATCH 16/16: arm/xlnx-zynqmp: put APUs and RPUs in separate CPU clusters... === OUTPUT END === Test command exited with code: 1 --- Email generated automatically by Patchew [http://patchew.org/]. Please send your feedback to patchew-de...@redhat.com