Re: [Qemu-devel] [PATCH v6 00/49] fix building of tests/tcg

qemu-devel
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 00/49] fix building of tests/tcg

From:	no-reply
Subject:	Re: [Qemu-devel] [PATCH v6 00/49] fix building of tests/tcg
Date:	Fri, 8 Jun 2018 07:23:48 -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 v6 00/49] fix building of tests/tcg

=== 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'
a3b42cec1c .travis.yml: add check-tcg test
9311c8f413 tests/docker: prevent sub-makes re-building debian-sid
09ca6d8e3b tests: add top-level make dependency for docker builds
2e879d658a target/sh4: Fix translator.c assertion failure for gUSA
0af702f272 tests/tcg: override runners for broken tests
7aaea5aa91 tests/tcg: add run, diff, and skip helper macros
e0eaf5b9a7 tests/Makefile.include: add [build|clean|check]-tcg targets
14fd002c16 Makefile.target: add (clean-/build-)guest-tests targets
3d0926618b tests/tcg/Makefile: update to be called from Makefile.target
c88f115914 tests/tcg: enable building for PowerPC
b563a10aac docker: move debian-powerpc-cross to sid based build
734d1f7ede tests/tcg: enable building for RISCV64
22bf732b43 tests/tcg: enable building for mips64
9b84cd6979 tests/tcg: enable building for sparc64
fefb20a640 tests/tcg: enable building for sh4
54483dadbf tests/tcg: enable building for m68k
605c4d05e0 tests/tcg: enable building for HPPA
39c366d478 tests/tcg/alpha: add Alpha specific tests
d007a1df09 tests/tcg: enable building for Alpha
da85acf3b9 tests/tcg: enable building for ppc64
275041e496 tests/tcg: enable building for s390x
76b1b57115 tests/tcg/mips: include common mips hello-mips
3001eff269 tests/tcg: enable building for MIPS
fc2f8e7d15 tests/tcg: move MIPS specific tests into subdir
68015cff55 tests/tcg/arm: add fcvt test cases for AArch32/64
6cf0b6d54f tests/tcg: enable building for AArch64
82a59304e2 tests/tcg/arm: fix up test-arm-iwmmxt test
2d3d6eb162 tests/tcg: enable building for ARM
1010d901f6 tests/tcg: move ARM specific tests into subdir
03414050fb tests/tcg/i386/test-i386: fix printf format
18dc0dd9c2 tests/tcg/i386/test-i386: use modern vector_size attributes
b1060c7415 tests/tcg/x86_64: add Makefile.target
f23ed58cdd tests/tcg/i386: add runner for test-i386-fprem
0d5ea512bb tests/tcg/i386: fix test-i386
6e69457c08 tests/tcg/i386: Build fix for hello-i386
c50f3d900e tests/tcg: enable building for i386
aab757d75e tests/tcg: move i386 specific tests into subdir
d76905d84c tests/tcg/multiarch: move most output to stdout
9138f45c68 tests/tcg/multiarch: enable additional linux-test tests
ced507e550 tests/tcg/multiarch: Build fix for linux-test
02861f0e01 tests/tcg: move architecture independent tests into subdir
2e91417605 docker: Makefile.include introduce DOCKER_SCRIPT
95e4639b49 docker: allow "cc" command to run in user context
638a3973fc docker: extend "cc" command to accept compiler
814faf606c docker: Add "cc" subcommand
cb7522263d configure: set cross_cc_FOO for host compiler
750b722573 configure: allow user to specify --cross-cc-cflags-foo=
00ce60af25 configure: move i386_cc to cross_cc_i386
badb0ec98b configure: add support for --cross-cc-FOO

=== OUTPUT BEGIN ===
Checking PATCH 1/49: configure: add support for --cross-cc-FOO...
Checking PATCH 2/49: configure: move i386_cc to cross_cc_i386...
Checking PATCH 3/49: configure: allow user to specify --cross-cc-cflags-foo=...
Checking PATCH 4/49: configure: set cross_cc_FOO for host compiler...
Checking PATCH 5/49: docker: Add "cc" subcommand...
Checking PATCH 6/49: docker: extend "cc" command to accept compiler...
Checking PATCH 7/49: docker: allow "cc" command to run in user context...
Checking PATCH 8/49: docker: Makefile.include introduce DOCKER_SCRIPT...
Checking PATCH 9/49: 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/49: 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/49: tests/tcg/multiarch: enable additional linux-test tests...
Checking PATCH 12/49: 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 13/49: 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 14/49: 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 15/49: 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 16/49: tests/tcg/i386: fix test-i386...
Checking PATCH 17/49: tests/tcg/i386: add runner for test-i386-fprem...
Checking PATCH 18/49: 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 19/49: tests/tcg/i386/test-i386: use modern vector_size 
attributes...
Checking PATCH 20/49: tests/tcg/i386/test-i386: fix printf format...
Checking PATCH 21/49: 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 22/49: 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 23/49: 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 24/49: 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 25/49: 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 26/49: 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 27/49: 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 28/49: 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 29/49: 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 30/49: 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 31/49: 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 32/49: 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 33/49: 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 34/49: 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 35/49: 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 36/49: 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 37/49: 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 38/49: 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 39/49: docker: move debian-powerpc-cross to sid based build...
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#49: 
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 40/49: tests/tcg: enable building for PowerPC...
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#22: 
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 41/49: tests/tcg/Makefile: update to be called from 
Makefile.target...
Checking PATCH 42/49: Makefile.target: add (clean-/build-)guest-tests targets...
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#48: 
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 43/49: tests/Makefile.include: add [build|clean|check]-tcg 
targets...
Checking PATCH 44/49: tests/tcg: add run, diff, and skip helper macros...
Checking PATCH 45/49: tests/tcg: override runners for broken tests...
Checking PATCH 46/49: target/sh4: Fix translator.c assertion failure for gUSA...
Checking PATCH 47/49: tests: add top-level make dependency for docker builds...
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#52: 
new file mode 100644

total: 0 errors, 1 warnings, 51 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 48/49: tests/docker: prevent sub-makes re-building debian-sid...
Checking PATCH 49/49: .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]
Re: [Qemu-devel] [PATCH v6 47/49] tests: add top-level make dependency for docker builds, (continued)
- [Qemu-devel] [PATCH v6 43/49] tests/Makefile.include: add [build|clean|check]-tcg targets, Alex Bennée, 2018/06/08
- [Qemu-devel] [PATCH v6 18/49] tests/tcg/x86_64: add Makefile.target, Alex Bennée, 2018/06/08
- [Qemu-devel] [PATCH v6 31/49] tests/tcg: enable building for Alpha, Alex Bennée, 2018/06/08
- [Qemu-devel] [PATCH v6 25/49] tests/tcg/arm: add fcvt test cases for AArch32/64, Alex Bennée, 2018/06/08
- [Qemu-devel] [PATCH v6 42/49] Makefile.target: add (clean-/build-)guest-tests targets, Alex Bennée, 2018/06/08
  - Re: [Qemu-devel] [PATCH v6 42/49] Makefile.target: add (clean-/build-)guest-tests targets, Philippe Mathieu-Daudé, 2018/06/10
- Re: [Qemu-devel] [PATCH v6 00/49] fix building of tests/tcg, no-reply <=
- Re: [Qemu-devel] [PATCH v6 00/49] fix building of tests/tcg, Philippe Mathieu-Daudé, 2018/06/10
  - Re: [Qemu-devel] [PATCH v6 00/49] fix building of tests/tcg, Alex Bennée, 2018/06/11
    - Re: [Qemu-devel] [PATCH v6 00/49] fix building of tests/tcg, Philippe Mathieu-Daudé, 2018/06/11
- Re: [Qemu-devel] [PATCH v6 00/49] fix building of tests/tcg, Philippe Mathieu-Daudé, 2018/06/10
Prev by Date: Re: [Qemu-devel] [RFC untested PATCH] i386/cpu: make -cpu host support monitor/mwait
Next by Date: [Qemu-devel] [PATCH v13 2/5] i386: Introduce auto_topoext bit to manage topoext
Previous by thread: Re: [Qemu-devel] [PATCH v6 42/49] Makefile.target: add (clean-/build-)guest-tests targets
Next by thread: Re: [Qemu-devel] [PATCH v6 00/49] fix building of tests/tcg
Index(es):
- Date
- Thread