Hi, This series seems to have some coding style problems. See output below for more information:
Type: series Message-id: 20180307110010.2205-1-quint...@redhat.com Subject: [Qemu-devel] [RFC v10 00/24] Multifd === 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' 0d4c2b3560 migration: Send pages through the multifd channels dfd20c6d98 migration: Create pages structure for reception 24414a65b0 migration: Create ram_multifd_page 2d73c8a690 migration: Transmit initial package through the multifd channels cfe7de3af1 migration: Delay start of migration main routines c753eaf6af migration: Create multifd channels 87eeee0345 migration: Add multifd traces for start/end thread 17e961cc77 migration: Export functions to create send channels 94f824225d migration: Synchronize recv threads 7f34d667a9 migration: Synchronize send threads 4a4b00a54d migration: Be sure all recv channels are created c97d843216 migration: Introduce multifd_recv_new_channel() 8a13787f66 migration: Reference counting recv channels correctly 577fbe0895 migration: terminate_* can be called for other threads d4f0966d50 migration: In case of error just end the migration c9443cf0c7 migration: Set error state in case of error caea778e74 migration: Add multifd test eab612ab0a tests: Add migration compress threads tests 849a3b52d2 tests: Add basic migration precopy tcp test 5ce68931c2 tests: Migration ppc now inlines its program 5f046ff880 migration: Set the migration tcp port 4a55e7ead3 migration: Create tcp_port parameter 0daf31aeec tests: Add migration xbzrle test 2d9ce85e32 tests: Add migration precopy test === OUTPUT BEGIN === Checking PATCH 1/24: tests: Add migration precopy test... Checking PATCH 2/24: tests: Add migration xbzrle test... Checking PATCH 3/24: migration: Create tcp_port parameter... Checking PATCH 4/24: migration: Set the migration tcp port... Checking PATCH 5/24: tests: Migration ppc now inlines its program... Checking PATCH 6/24: tests: Add basic migration precopy tcp test... Checking PATCH 7/24: tests: Add migration compress threads tests... Checking PATCH 8/24: migration: Add multifd test... Checking PATCH 9/24: migration: Set error state in case of error... Checking PATCH 10/24: migration: In case of error just end the migration... Checking PATCH 11/24: migration: terminate_* can be called for other threads... Checking PATCH 12/24: migration: Reference counting recv channels correctly... Checking PATCH 13/24: migration: Introduce multifd_recv_new_channel()... Checking PATCH 14/24: migration: Be sure all recv channels are created... Checking PATCH 15/24: migration: Synchronize send threads... Checking PATCH 16/24: migration: Synchronize recv threads... Checking PATCH 17/24: migration: Export functions to create send channels... Checking PATCH 18/24: migration: Add multifd traces for start/end thread... Checking PATCH 19/24: migration: Create multifd channels... ERROR: do not initialise statics to 0 or NULL #109: FILE: migration/ram.c:735: + static int i = 0; total: 1 errors, 0 warnings, 100 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 20/24: migration: Delay start of migration main routines... Checking PATCH 21/24: migration: Transmit initial package through the multifd channels... Checking PATCH 22/24: migration: Create ram_multifd_page... ERROR: do not initialise statics to 0 or NULL #184: FILE: migration/ram.c:660: + static int next_channel = 0; ERROR: space prohibited before that close parenthesis ')' #204: FILE: migration/ram.c:680: + for (i = next_channel;; i = (i + 1) % migrate_multifd_channels() ) { total: 2 errors, 0 warnings, 250 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 23/24: migration: Create pages structure for reception... Checking PATCH 24/24: migration: Send pages through the multifd channels... === OUTPUT END === Test command exited with code: 1 --- Email generated automatically by Patchew [http://patchew.org/]. Please send your feedback to patchew-de...@freelists.org