Patchew URL: https://patchew.org/QEMU/20190308202858.26636-1-js...@redhat.com/
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20190308202858.26636-1-js...@redhat.com Subject: [Qemu-devel] [PULL 00/17] Bitmaps 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 === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 From https://github.com/patchew-project/qemu t [tag update] patchew/20190308202858.26636-1-js...@redhat.com -> patchew/20190308202858.26636-1-js...@redhat.com Switched to a new branch 'test' d67d39e43b block/dirty-bitmaps: implement inconsistent bit cc50ffdaaa block/dirty-bitmaps: disallow busy bitmaps as merge source fe93ecd8b1 block/dirty-bitmaps: prohibit removing readonly bitmaps 1d82e65c16 block/dirty-bitmaps: prohibit readonly bitmaps for backups 61c55a9de7 block/dirty-bitmaps: add block_dirty_bitmap_check function 4f1ceecd95 block/dirty-bitmap: add inconsistent status 9d5d76cbd6 block/dirty-bitmaps: add inconsistent bit bdc998d9d5 iotests: add busy/recording bit test to 124 22c03a2fd5 blockdev: remove unused paio parameter documentation d74e8a7202 block/dirty-bitmaps: move comment block 8ea8b7a875 block/dirty-bitmaps: unify qmp_locked and user_locked calls ae989ad6ff block/dirty-bitmap: explicitly lock bitmaps with successors 6c71258b44 nbd: change error checking order for bitmaps cf861d0168 block/dirty-bitmap: change semantics of enabled predicate 5ba70dfbcf block/dirty-bitmap: remove set/reset assertions against enabled bit 83886ed7db block/dirty-bitmaps: rename frozen predicate helper 0e38e3ff4f block/dirty-bitmap: add recording and busy properties === OUTPUT BEGIN === 1/17 Checking commit 0e38e3ff4ff4 (block/dirty-bitmap: add recording and busy properties) 2/17 Checking commit 83886ed7db44 (block/dirty-bitmaps: rename frozen predicate helper) WARNING: line over 80 characters #87: FILE: block/dirty-bitmap.c:248: + error_setg(errp, "Cannot create a successor for a bitmap that is in-use " total: 0 errors, 1 warnings, 124 lines checked Patch 2/17 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 3/17 Checking commit 5ba70dfbcf19 (block/dirty-bitmap: remove set/reset assertions against enabled bit) 4/17 Checking commit cf861d01687f (block/dirty-bitmap: change semantics of enabled predicate) 5/17 Checking commit 6c71258b445a (nbd: change error checking order for bitmaps) 6/17 Checking commit ae989ad6ffc7 (block/dirty-bitmap: explicitly lock bitmaps with successors) 7/17 Checking commit 8ea8b7a8757e (block/dirty-bitmaps: unify qmp_locked and user_locked calls) ERROR: open brace '{' following function declarations go on the next line #39: FILE: block/dirty-bitmap.c:190: +bool bdrv_dirty_bitmap_busy(BdrvDirtyBitmap *bitmap) { total: 1 errors, 0 warnings, 271 lines checked Patch 7/17 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 8/17 Checking commit d74e8a720254 (block/dirty-bitmaps: move comment block) 9/17 Checking commit 22c03a2fd589 (blockdev: remove unused paio parameter documentation) 10/17 Checking commit bdc998d9d5ed (iotests: add busy/recording bit test to 124) 11/17 Checking commit 9d5d76cbd6ca (block/dirty-bitmaps: add inconsistent bit) WARNING: Block comments use a leading /* on a separate line #26: FILE: block/dirty-bitmap.c:49: + bool inconsistent; /* bitmap is persistent, but inconsistent. WARNING: Block comments use a trailing */ on a separate line #28: FILE: block/dirty-bitmap.c:51: + * a QMP user can remove it. */ total: 0 errors, 2 warnings, 82 lines checked Patch 11/17 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 12/17 Checking commit 4f1ceecd9528 (block/dirty-bitmap: add inconsistent status) 13/17 Checking commit 61c55a9de7e4 (block/dirty-bitmaps: add block_dirty_bitmap_check function) ERROR: open brace '{' following function declarations go on the next line #37: FILE: block/dirty-bitmap.c:177: +static bool bdrv_dirty_bitmap_busy(const BdrvDirtyBitmap *bitmap) { WARNING: line over 80 characters #284: FILE: migration/block-dirty-bitmap.c:305: + if (bdrv_dirty_bitmap_check(bitmap, BDRV_BITMAP_DEFAULT, &local_err)) { total: 1 errors, 1 warnings, 236 lines checked Patch 13/17 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 14/17 Checking commit 1d82e65c16d6 (block/dirty-bitmaps: prohibit readonly bitmaps for backups) 15/17 Checking commit fe93ecd8b1cd (block/dirty-bitmaps: prohibit removing readonly bitmaps) 16/17 Checking commit cc50ffdaaa38 (block/dirty-bitmaps: disallow busy bitmaps as merge source) 17/17 Checking commit d67d39e43b84 (block/dirty-bitmaps: implement inconsistent bit) === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20190308202858.26636-1-js...@redhat.com/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [http://patchew.org/]. Please send your feedback to patchew-de...@redhat.com
