Re: [Qemu-devel] [PATCH v7 00/54] fix building of tests/tcg

qemu-devel
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 00/54] fix building of tests/tcg - last chanc

From:	no-reply
Subject:	Re: [Qemu-devel] [PATCH v7 00/54] fix building of tests/tcg - last chance to review!
Date:	Fri, 15 Jun 2018 15:29:07 -0700 (PDT)
Hi,

This series seems to have some coding style problems. See output below for
more information:

Type: series
Message-id: address@hidden
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/address@hidden -> patchew/address@hidden
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 address@hidden
[Prev in Thread]
Current Thread
[Next in Thread]
[Qemu-devel] [PATCH v7 13/54] tests/tcg/multiarch: move most output to stdout, (continued)
- [Qemu-devel] [PATCH v7 13/54] tests/tcg/multiarch: move most output to stdout, Alex Bennée, 2018/06/15
  - Re: [Qemu-devel] [PATCH v7 13/54] tests/tcg/multiarch: move most output to stdout, Thomas Huth, 2018/06/16
- [Qemu-devel] [PATCH v7 50/54] docker: docker.py add check sub-command, Alex Bennée, 2018/06/15
  - Re: [Qemu-devel] [PATCH v7 50/54] docker: docker.py add check sub-command, Philippe Mathieu-Daudé, 2018/06/18
- [Qemu-devel] [PATCH v7 36/54] tests/tcg: enable building for m68k, Alex Bennée, 2018/06/15
- [Qemu-devel] [PATCH v7 27/54] tests/tcg: move MIPS specific tests into subdir, Alex Bennée, 2018/06/15
- [Qemu-devel] [PATCH v7 14/54] tests/tcg: move i386 specific tests into subdir, Alex Bennée, 2018/06/15
- [Qemu-devel] [PATCH v7 38/54] tests/tcg: enable building for sparc64, Alex Bennée, 2018/06/15
- [Qemu-devel] [PATCH v7 52/54] docker: docker.py adding age check command, Alex Bennée, 2018/06/15
  - Re: [Qemu-devel] [PATCH v7 52/54] docker: docker.py adding age check command, Philippe Mathieu-Daudé, 2018/06/18
- Re: [Qemu-devel] [PATCH v7 00/54] fix building of tests/tcg - last chance to review!, no-reply <=
Prev by Date: [Qemu-devel] [PATCH v3 1/2] kvm: support -dedicated cpu-pm=on|off
Next by Date: Re: [Qemu-devel] [PATCH 6/6] linux-user: Use *at functions to implement interp_prefix
Previous by thread: Re: [Qemu-devel] [PATCH v7 52/54] docker: docker.py adding age check command
Next by thread: [Qemu-devel] [PATCH v3 0/2] Fix compilation with python-3 if en_US.UTF-8 is unavailable
Index(es):
- Date
- Thread