Patchew URL: https://patchew.org/QEMU/20190711104412.31233-1-quintela@xxxxxxxxxx/ Hi, This series seems to have some coding style problems. See output below for more information: Message-id: 20190711104412.31233-1-quintela@xxxxxxxxxx Type: series Subject: [Qemu-devel] [PULL 00/19] Migration patches === 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 === >From https://github.com/patchew-project/qemu * [new tag] patchew/20190711104412.31233-1-quintela@xxxxxxxxxx -> patchew/20190711104412.31233-1-quintela@xxxxxxxxxx Auto packing the repository in background for optimum performance. See "git help gc" for manual housekeeping. Switched to a new branch 'test' 209c2845db migration: allow private destination ram with x-ignore-shared c008de3513 migration: Split log_clear() into smaller chunks 1b98b966ec kvm: Support KVM_CLEAR_DIRTY_LOG 6850c34958 kvm: Introduce slots lock for memory listener 787a3e91db kvm: Persistent per kvmslot dirty bitmap 18fb631382 kvm: Update comments for sync_dirty_bitmap 86d95e6ce0 memory: Introduce memory listener hook log_clear() d1063f1461 memory: Pass mr into snapshot_and_clear_dirty 3fafbda5af bitmap: Add bitmap_copy_with_{src|dst}_offset() 78c643f8c7 memory: Don't set migration bitmap when without migration c5bac6deb3 migration: No need to take rcu during sync_dirty_bitmap ea445e2533 migratioin/ram.c: reset complete_round when we gets a queued page e8e5e91626 migration/multifd: sync packet_num after all thread are done e5c62ba89d cutils: remove one unnecessary pointer operation 5914c83527 migration/xbzrle: update cache and current_data in one place 404656a2b0 migration/multifd: call multifd_send_sync_main when sending RAM_SAVE_FLAG_EOS fd522f622b migration-test: Add migration multifd test 410a42b077 migration-test: rename parameter to parameter_int c4c921ae43 migration: fix multifd_recv event typo === OUTPUT BEGIN === 1/19 Checking commit c4c921ae438d (migration: fix multifd_recv event typo) 2/19 Checking commit 410a42b0779f (migration-test: rename parameter to parameter_int) 3/19 Checking commit fd522f622bf7 (migration-test: Add migration multifd test) 4/19 Checking commit 404656a2b029 (migration/multifd: call multifd_send_sync_main when sending RAM_SAVE_FLAG_EOS) 5/19 Checking commit 5914c835270c (migration/xbzrle: update cache and current_data in one place) 6/19 Checking commit e5c62ba89d60 (cutils: remove one unnecessary pointer operation) 7/19 Checking commit e8e5e91626d2 (migration/multifd: sync packet_num after all thread are done) 8/19 Checking commit ea445e253364 (migratioin/ram.c: reset complete_round when we gets a queued page) 9/19 Checking commit c5bac6deb363 (migration: No need to take rcu during sync_dirty_bitmap) 10/19 Checking commit 78c643f8c785 (memory: Don't set migration bitmap when without migration) 11/19 Checking commit 3fafbda5afa5 (bitmap: Add bitmap_copy_with_{src|dst}_offset()) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #67: new file mode 100644 total: 0 errors, 1 warnings, 193 lines checked Patch 11/19 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 12/19 Checking commit d1063f1461e2 (memory: Pass mr into snapshot_and_clear_dirty) 13/19 Checking commit 86d95e6ce000 (memory: Introduce memory listener hook log_clear()) 14/19 Checking commit 18fb63138295 (kvm: Update comments for sync_dirty_bitmap) 15/19 Checking commit 787a3e91dbe7 (kvm: Persistent per kvmslot dirty bitmap) 16/19 Checking commit 6850c3495802 (kvm: Introduce slots lock for memory listener) 17/19 Checking commit 1b98b966ec31 (kvm: Support KVM_CLEAR_DIRTY_LOG) WARNING: line over 80 characters #201: FILE: accel/kvm/kvm-all.c:1054: + KVMMemoryListener *kml = container_of(listener, KVMMemoryListener, listener); total: 0 errors, 1 warnings, 217 lines checked Patch 17/19 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 18/19 Checking commit c008de3513ac (migration: Split log_clear() into smaller chunks) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #38: new file mode 100644 ERROR: trailing whitespace #48: FILE: include/exec/memory.h.rej:6: + $ ERROR: trailing whitespace #670: FILE: memory.c.rej:6: + $ ERROR: trailing whitespace #5765: FILE: migration/ram.c.rej:4: + $ ERROR: trailing whitespace #5772: FILE: migration/ram.c.rej:11: + $ total: 4 errors, 1 warnings, 6000 lines checked Patch 18/19 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 19/19 Checking commit 209c2845db95 (migration: allow private destination ram with x-ignore-shared) === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20190711104412.31233-1-quintela@xxxxxxxxxx/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-devel@xxxxxxxxxx
