Hi, This series seems to have some coding style problems. See output below for more information:
Type: series Message-id: 20180615194705.28019-1-alex.ben...@linaro.org Subject: [Qemu-devel] [PATCH v7 00/54] fix building of tests/tcg - last chance to review! === 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 From https://github.com/patchew-project/qemu * [new tag] patchew/20180615194705.28019-1-alex.ben...@linaro.org -> patchew/20180615194705.28019-1-alex.ben...@linaro.org Switched to a new branch 'test' ea7a4dbd0b .travis.yml: add check-tcg test 652c13f1eb tests/docker/Makefile.include: only force SID to NOCACHE if old b70bc02b5f docker: docker.py adding age check command fbdbc02b19 tests/Makefile: call sub-makes with SKIP_DOCKER_BUILD=1 5fc8f8b693 docker: docker.py add check sub-command 77bed125f5 docker: docker.py don't conflate checksums for extra_files 148852d075 tests: add top-level make dependency for docker builds af6ee2ab56 tests/tcg: override runners for broken tests 7d4195d3d0 tests/tcg: add run, diff, and skip helper macros 432f648e13 tests/Makefile.include: add [build|clean|check]-tcg targets 08e57f8079 Makefile.target: add (clean-/build-)guest-tests targets fccd475614 tests/tcg/Makefile: update to be called from Makefile.target bd2f60e170 tests/tcg: enable building for PowerPC b3ac2e1ba8 docker: move debian-powerpc-cross to sid based build 5cd0a65cbe tests/tcg: enable building for RISCV64 572ec17e38 tests/tcg: enable building for mips64 10fcf0b447 tests/tcg: enable building for sparc64 4a77005f2c tests/tcg: enable building for sh4 13f7f0d965 tests/tcg: enable building for m68k 3d20ff21ce tests/tcg: enable building for HPPA ea2fcf674f tests/tcg/alpha: add Alpha specific tests dd92f8fdfa tests/tcg: enable building for Alpha a2d10caaf4 tests/tcg: enable building for ppc64 0e6da3e2c3 tests/tcg: enable building for s390x f7c8cdc696 tests/tcg/mips: include common mips hello-mips 97518355bd tests/docker/Makefile.include: fix mipsel-cross dependancy e359ed8d61 tests/tcg: enable building for MIPS 97ec8f3d2c tests/tcg: move MIPS specific tests into subdir 61086dc3e8 tests/tcg/arm: add fcvt test cases for AArch32/64 6cbb7a06c0 tests/tcg: enable building for AArch64 9a6f991ee2 tests/tcg/arm: fix up test-arm-iwmmxt test 32c5010c20 tests/tcg: enable building for ARM 2146fdbe57 tests/tcg: move ARM specific tests into subdir 3091d9d510 tests/tcg/i386/test-i386: fix printf format b82c887f2a tests/tcg/i386/test-i386: use modern vector_size attributes 37aaf79acf tests/tcg/x86_64: add Makefile.target b5c4e6bf40 tests/tcg/i386: add runner for test-i386-fprem 246d97eacf tests/tcg/i386: fix test-i386 8671ac9d73 tests/tcg/i386: Build fix for hello-i386 8edd5f2136 tests/tcg: enable building for i386 67af1a0edc tests/tcg: move i386 specific tests into subdir de3b73900a tests/tcg/multiarch: move most output to stdout 3504d45b45 tests/tcg/multiarch: don't hard code paths/ports for linux-test 127f1a9428 tests/tcg/multiarch: enable additional linux-test tests 3e1df9f60f tests/tcg/multiarch: Build fix for linux-test edc1f17df2 tests/tcg: move architecture independent tests into subdir 7e3c3cc8cd docker: Makefile.include introduce DOCKER_SCRIPT 4a9acb8a80 docker: allow "cc" command to run in user context e578387528 docker: extend "cc" command to accept compiler aecae6731d docker: Add "cc" subcommand c9235deb15 configure: set cross_cc_FOO for host compiler 505c30b5c9 configure: allow user to specify --cross-cc-cflags-foo= 58d739bb59 configure: move i386_cc to cross_cc_i386 9944156f78 configure: add support for --cross-cc-FOO === OUTPUT BEGIN === Checking PATCH 1/54: configure: add support for --cross-cc-FOO... Checking PATCH 2/54: configure: move i386_cc to cross_cc_i386... Checking PATCH 3/54: configure: allow user to specify --cross-cc-cflags-foo=... Checking PATCH 4/54: configure: set cross_cc_FOO for host compiler... Checking PATCH 5/54: docker: Add "cc" subcommand... Checking PATCH 6/54: docker: extend "cc" command to accept compiler... Checking PATCH 7/54: docker: allow "cc" command to run in user context... Checking PATCH 8/54: docker: Makefile.include introduce DOCKER_SCRIPT... Checking PATCH 9/54: tests/tcg: move architecture independent tests into subdir... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #138: new file mode 100644 total: 0 errors, 1 warnings, 136 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 10/54: tests/tcg/multiarch: Build fix for linux-test... ERROR: if this code is redundant consider removing it #156: FILE: tests/tcg/multiarch/linux-test.c:323: +#if 0 total: 1 errors, 0 warnings, 185 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 11/54: tests/tcg/multiarch: enable additional linux-test tests... Checking PATCH 12/54: tests/tcg/multiarch: don't hard code paths/ports for linux-test... Checking PATCH 13/54: tests/tcg/multiarch: move most output to stdout... ERROR: code indent should never use tabs #37: FILE: tests/tcg/multiarch/test-mmap.c:60: +^Ifprintf(stdout, "%s", __func__);$ ERROR: code indent should never use tabs #46: FILE: tests/tcg/multiarch/test-mmap.c:109: +^Ifprintf(stdout, " passed\n");$ ERROR: code indent should never use tabs #55: FILE: tests/tcg/multiarch/test-mmap.c:118: +^Ifprintf(stdout, "%s", __func__);$ ERROR: code indent should never use tabs #64: FILE: tests/tcg/multiarch/test-mmap.c:133: +^Ifprintf(stdout, " passed\n");$ ERROR: code indent should never use tabs #73: FILE: tests/tcg/multiarch/test-mmap.c:144: +^Ifprintf(stdout, "%s", __func__);$ ERROR: code indent should never use tabs #82: FILE: tests/tcg/multiarch/test-mmap.c:183: +^Ifprintf(stdout, " passed\n");$ ERROR: code indent should never use tabs #91: FILE: tests/tcg/multiarch/test-mmap.c:197: +^Ifprintf(stdout, "%s addr=%p", __func__, addr);$ ERROR: code indent should never use tabs #100: FILE: tests/tcg/multiarch/test-mmap.c:215: +^Ifprintf(stdout, " passed\n");$ ERROR: code indent should never use tabs #110: FILE: tests/tcg/multiarch/test-mmap.c:228: +^Ifprintf(stdout, "%s addr=%p", __func__, addr);$ ERROR: code indent should never use tabs #111: FILE: tests/tcg/multiarch/test-mmap.c:229: +^Ifprintf(stdout, "FIXME: QEMU fails to track pages used by the host.");$ ERROR: code indent should never use tabs #120: FILE: tests/tcg/multiarch/test-mmap.c:246: +^Ifprintf(stdout, " passed\n");$ ERROR: code indent should never use tabs #129: FILE: tests/tcg/multiarch/test-mmap.c:255: +^Ifprintf(stdout, "%s", __func__);$ ERROR: code indent should never use tabs #138: FILE: tests/tcg/multiarch/test-mmap.c:297: +^Ifprintf(stdout, " passed\n");$ ERROR: code indent should never use tabs #147: FILE: tests/tcg/multiarch/test-mmap.c:307: +^Ifprintf(stdout, "%s", __func__);$ ERROR: code indent should never use tabs #156: FILE: tests/tcg/multiarch/test-mmap.c:330: +^Ifprintf(stdout, " passed\n");$ ERROR: code indent should never use tabs #165: FILE: tests/tcg/multiarch/test-mmap.c:346: +^Ifprintf(stdout, "%s addr=%p", __func__, (void *)addr);$ ERROR: code indent should never use tabs #174: FILE: tests/tcg/multiarch/test-mmap.c:374: +^Ifprintf(stdout, " passed\n");$ ERROR: code indent should never use tabs #183: FILE: tests/tcg/multiarch/test-mmap.c:387: +^Ifprintf(stdout, "%s addr=%p", __func__, (void *)addr);$ ERROR: code indent should never use tabs #192: FILE: tests/tcg/multiarch/test-mmap.c:429: +^Ifprintf(stdout, " passed\n");$ total: 19 errors, 0 warnings, 154 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 14/54: tests/tcg: move i386 specific tests into subdir... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #84: new file mode 100644 total: 0 errors, 1 warnings, 120 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 15/54: tests/tcg: enable building for i386... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #21: new file mode 100644 total: 0 errors, 1 warnings, 19 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 16/54: tests/tcg/i386: Build fix for hello-i386... ERROR: externs should be avoided in .c files #22: FILE: tests/tcg/i386/hello-i386.c:23: +void _start(void); total: 1 errors, 0 warnings, 7 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 17/54: tests/tcg/i386: fix test-i386... Checking PATCH 18/54: tests/tcg/i386: add runner for test-i386-fprem... Checking PATCH 19/54: tests/tcg/x86_64: add Makefile.target... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #49: new file mode 100644 total: 0 errors, 1 warnings, 33 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/54: tests/tcg/i386/test-i386: use modern vector_size attributes... Checking PATCH 21/54: tests/tcg/i386/test-i386: fix printf format... Checking PATCH 22/54: tests/tcg: move ARM specific tests into subdir... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #52: new file mode 100644 total: 0 errors, 1 warnings, 46 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/54: tests/tcg: enable building for ARM... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #18: new file mode 100644 total: 0 errors, 1 warnings, 21 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 24/54: tests/tcg/arm: fix up test-arm-iwmmxt test... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #42: rename from tests/tcg/arm/test-arm-iwmmxt.s total: 0 errors, 1 warnings, 14 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 25/54: tests/tcg: enable building for AArch64... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #30: new file mode 100644 total: 0 errors, 1 warnings, 15 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 26/54: tests/tcg/arm: add fcvt test cases for AArch32/64... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #19: new file mode 100644 WARNING: architecture specific defines should be avoided #3430: FILE: tests/tcg/arm/fcvt.c:76: +# if __GNUC_PREREQ(3, 3) ERROR: space prohibited between function name and open parenthesis '(' #3431: FILE: tests/tcg/arm/fcvt.c:77: +# define SNANF (__builtin_nansf ("")) ERROR: space prohibited between function name and open parenthesis '(' #3432: FILE: tests/tcg/arm/fcvt.c:78: +# define SNAN (__builtin_nans ("")) ERROR: space prohibited between function name and open parenthesis '(' #3433: FILE: tests/tcg/arm/fcvt.c:79: +# define SNANL (__builtin_nansl ("")) ERROR: spaces required around that '+' (ctx:VxV) #3441: FILE: tests/tcg/arm/fcvt.c:87: + -1.111E+31, ^ ERROR: spaces required around that '+' (ctx:VxV) #3442: FILE: tests/tcg/arm/fcvt.c:88: + -1.111E+30, ^ ERROR: spaces required around that '-' (ctx:VxV) #3443: FILE: tests/tcg/arm/fcvt.c:89: + -1.08700982e-12, ^ ERROR: spaces required around that '-' (ctx:VxV) #3444: FILE: tests/tcg/arm/fcvt.c:90: + -1.78051176e-20, ^ ERROR: spaces required around that '-' (ctx:VxV) #3448: FILE: tests/tcg/arm/fcvt.c:94: + 2.98023224e-08, ^ ERROR: spaces required around that '-' (ctx:VxV) #3449: FILE: tests/tcg/arm/fcvt.c:95: + 5.96046E-8, /* min positive FP16 subnormal */ ^ ERROR: spaces required around that '-' (ctx:VxV) #3450: FILE: tests/tcg/arm/fcvt.c:96: + 6.09756E-5, /* max subnormal FP16 */ ^ ERROR: spaces required around that '-' (ctx:VxV) #3451: FILE: tests/tcg/arm/fcvt.c:97: + 6.10352E-5, /* min positive normal FP16 */ ^ ERROR: spaces required around that '+' (ctx:VxV) #3462: FILE: tests/tcg/arm/fcvt.c:108: + 1.111E+30, ^ WARNING: architecture specific defines should be avoided #3480: FILE: tests/tcg/arm/fcvt.c:126: +#if defined(__arm__) WARNING: architecture specific defines should be avoided #3505: FILE: tests/tcg/arm/fcvt.c:151: +#if defined(__arm__) WARNING: architecture specific defines should be avoided #3527: FILE: tests/tcg/arm/fcvt.c:173: +#if defined(__arm__) ERROR: spaces required around that '-' (ctx:VxV) #3548: FILE: tests/tcg/arm/fcvt.c:194: + {-FLT_MAX-1.0}, ^ ERROR: spaces required around that '+' (ctx:VxV) #3550: FILE: tests/tcg/arm/fcvt.c:196: + {-1.111E+31}, ^ ERROR: spaces required around that '+' (ctx:VxV) #3551: FILE: tests/tcg/arm/fcvt.c:197: + {-1.111E+30}, /* half prec */ ^ ERROR: spaces required around that '-' (ctx:VxV) #3557: FILE: tests/tcg/arm/fcvt.c:203: + {2.98023224e-08}, ^ ERROR: spaces required around that '-' (ctx:VxV) #3558: FILE: tests/tcg/arm/fcvt.c:204: + {5.96046E-8}, /* min positive FP16 subnormal */ ^ ERROR: spaces required around that '-' (ctx:VxV) #3559: FILE: tests/tcg/arm/fcvt.c:205: + {6.09756E-5}, /* max subnormal FP16 */ ^ ERROR: spaces required around that '-' (ctx:VxV) #3560: FILE: tests/tcg/arm/fcvt.c:206: + {6.10352E-5}, /* min positive normal FP16 */ ^ ERROR: spaces required around that '-' (ctx:VxV) #3564: FILE: tests/tcg/arm/fcvt.c:210: + {1.3789972848607228e-308}, ^ ERROR: spaces required around that '-' (ctx:VxV) #3565: FILE: tests/tcg/arm/fcvt.c:211: + {1.4914738736681624e-308}, ^ WARNING: architecture specific defines should be avoided #3599: FILE: tests/tcg/arm/fcvt.c:245: +#if defined(__arm__) WARNING: architecture specific defines should be avoided #3623: FILE: tests/tcg/arm/fcvt.c:269: +#if defined(__arm__) WARNING: architecture specific defines should be avoided #3646: FILE: tests/tcg/arm/fcvt.c:292: +#if defined(__arm__) WARNING: architecture specific defines should be avoided #3690: FILE: tests/tcg/arm/fcvt.c:336: +#if defined(__arm__) WARNING: architecture specific defines should be avoided #3713: FILE: tests/tcg/arm/fcvt.c:359: +#if defined(__arm__) WARNING: architecture specific defines should be avoided #3735: FILE: tests/tcg/arm/fcvt.c:381: +#if defined(__arm__) WARNING: architecture specific defines should be avoided #3780: FILE: tests/tcg/arm/fcvt.c:426: +#if defined(__arm__) ERROR: space prohibited before open square bracket '[' #3785: FILE: tests/tcg/arm/fcvt.c:431: + : /* no output */ : [flags] "n" (1 << 26) : "r1" ); ERROR: space prohibited before that close parenthesis ')' #3785: FILE: tests/tcg/arm/fcvt.c:431: + : /* no output */ : [flags] "n" (1 << 26) : "r1" ); ERROR: space prohibited before open square bracket '[' #3790: FILE: tests/tcg/arm/fcvt.c:436: + : /* no output */ : [flags] "n" (1 << 26) : "x1" ); ERROR: space prohibited before that close parenthesis ')' #3790: FILE: tests/tcg/arm/fcvt.c:436: + : /* no output */ : [flags] "n" (1 << 26) : "x1" ); total: 25 errors, 12 warnings, 7039 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 27/54: tests/tcg: move MIPS specific tests into subdir... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #39: new file mode 100644 total: 0 errors, 1 warnings, 24 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 28/54: tests/tcg: enable building for MIPS... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #19: new file mode 100644 total: 0 errors, 1 warnings, 17 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 29/54: tests/docker/Makefile.include: fix mipsel-cross dependancy... Checking PATCH 30/54: tests/tcg/mips: include common mips hello-mips... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #15: new file mode 100644 total: 0 errors, 1 warnings, 19 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 31/54: tests/tcg: enable building for s390x... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #20: new file mode 100644 total: 0 errors, 1 warnings, 2 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 32/54: tests/tcg: enable building for ppc64... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #17: new file mode 100644 total: 0 errors, 1 warnings, 2 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 33/54: tests/tcg: enable building for Alpha... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #38: new file mode 100644 total: 0 errors, 1 warnings, 65 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 34/54: tests/tcg/alpha: add Alpha specific tests... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #20: deleted file mode 100644 total: 0 errors, 1 warnings, 31 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 35/54: tests/tcg: enable building for HPPA... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #28: new file mode 100644 total: 0 errors, 1 warnings, 27 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 36/54: tests/tcg: enable building for m68k... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #30: new file mode 100644 total: 0 errors, 1 warnings, 28 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 37/54: tests/tcg: enable building for sh4... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #32: new file mode 100644 total: 0 errors, 1 warnings, 30 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 38/54: tests/tcg: enable building for sparc64... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #29: new file mode 100644 total: 0 errors, 1 warnings, 27 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 39/54: tests/tcg: enable building for mips64... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #29: new file mode 100644 total: 0 errors, 1 warnings, 28 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 40/54: tests/tcg: enable building for RISCV64... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #29: new file mode 100644 total: 0 errors, 1 warnings, 29 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 41/54: docker: move debian-powerpc-cross to sid based build... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #51: deleted file mode 100755 total: 0 errors, 1 warnings, 71 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 42/54: tests/tcg: enable building for PowerPC... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #23: new file mode 100644 total: 0 errors, 1 warnings, 19 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 43/54: tests/tcg/Makefile: update to be called from Makefile.target... Checking PATCH 44/54: Makefile.target: add (clean-/build-)guest-tests targets... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #49: new file mode 100644 total: 0 errors, 1 warnings, 106 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 45/54: tests/Makefile.include: add [build|clean|check]-tcg targets... Checking PATCH 46/54: tests/tcg: add run, diff, and skip helper macros... Checking PATCH 47/54: tests/tcg: override runners for broken tests... Checking PATCH 48/54: tests: add top-level make dependency for docker builds... WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #57: new file mode 100644 total: 0 errors, 1 warnings, 54 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 49/54: docker: docker.py don't conflate checksums for extra_files... Checking PATCH 50/54: docker: docker.py add check sub-command... Checking PATCH 51/54: tests/Makefile: call sub-makes with SKIP_DOCKER_BUILD=1... Checking PATCH 52/54: docker: docker.py adding age check command... Checking PATCH 53/54: tests/docker/Makefile.include: only force SID to NOCACHE if old... Checking PATCH 54/54: .travis.yml: add check-tcg test... === 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