Re: [PATCH v3 00/19] testing & plugin updates

[no-reply]

Patchew URL: https://patchew.org/QEMU/address@hidden/



Hi,

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

Subject: [PATCH  v3 00/19] testing & plugin updates
Message-id: address@hidden
Type: series

=== 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
 * [new tag]         patchew/address@hidden -> patchew/address@hidden
 - [tag update]      patchew/address@hidden -> patchew/address@hidden
 - [tag update]      patchew/address@hidden -> patchew/address@hidden
 - [tag update]      patchew/address@hidden -> patchew/address@hidden
 * [new tag]         patchew/20200226122054.366b9cda@table.localdomain -> 
patchew/20200226122054.366b9cda@table.localdomain
 * [new tag]         patchew/address@hidden -> patchew/address@hidden
Auto packing the repository for optimum performance. You may also
run "git gc" manually. See "git help gc" for more information.
Switched to a new branch 'test'
f5a36bd tests/tcg: take into account expected clashes pauth-4
a912120 tests/tcg: fix typo in configure.sh test for v8.3
460922a tcg: save vaddr temp for plugin usage
765efaa tests/tcg: give debug builds a little bit longer
3ef8a10 tests/plugins: make howvec clean-up after itself.
22744b6 target/riscv: progressively load the instruction during decode
8ae738a qemu/bitops.h: Add extract8 and extract16
d2a1203 tests/plugin: prevent uninitialized warning
b1a5387 plugins/core: add missing break in cb_to_tcg_flags
34893a2 docs/devel: document query handle lifetimes
a15f458 tracing: only allow -trace to override -D if set
a4963d5 tests/iotests: be a little more forgiving on the size test
c88b071 travis.yml: single-thread build-tcg stages
145aad2 travis.yml: Fix Travis YAML configuration warnings
2944b87 travis.yml: Test the s390-ccw build, too
901bfdf tests/rcutorture: mild documenting refactor of update thread
38d1f0b tests/rcutorture: better document locking of stats
549cbb3 tests/rcutorture: update usage hint
39535f9 tests/tcg: include a skip runner for pauth3 with plugins

=== OUTPUT BEGIN ===
1/19 Checking commit 39535f9b7a09 (tests/tcg: include a skip runner for pauth3 
with plugins)
2/19 Checking commit 549cbb301602 (tests/rcutorture: update usage hint)
3/19 Checking commit 38d1f0b4951e (tests/rcutorture: better document locking of 
stats)
4/19 Checking commit 901bfdfaa514 (tests/rcutorture: mild documenting refactor 
of update thread)
5/19 Checking commit 2944b8798439 (travis.yml: Test the s390-ccw build, too)
6/19 Checking commit 145aad29e004 (travis.yml: Fix Travis YAML configuration 
warnings)
7/19 Checking commit c88b0718ccdf (travis.yml: single-thread build-tcg stages)
8/19 Checking commit a4963d59b0a5 (tests/iotests: be a little more forgiving on 
the size test)
9/19 Checking commit a15f458976b5 (tracing: only allow -trace to override -D if 
set)
10/19 Checking commit 34893a28f7fc (docs/devel: document query handle lifetimes)
11/19 Checking commit b1a5387c45a5 (plugins/core: add missing break in 
cb_to_tcg_flags)
12/19 Checking commit d2a1203bbd09 (tests/plugin: prevent uninitialized warning)
13/19 Checking commit 8ae738a574f7 (qemu/bitops.h: Add extract8 and extract16)
14/19 Checking commit 22744b62fd12 (target/riscv: progressively load the 
instruction during decode)
15/19 Checking commit 3ef8a10a32b7 (tests/plugins: make howvec clean-up after 
itself.)
16/19 Checking commit 765efaae0c80 (tests/tcg: give debug builds a little bit 
longer)
17/19 Checking commit 460922a55240 (tcg: save vaddr temp for plugin usage)
18/19 Checking commit a9121200ef14 (tests/tcg: fix typo in configure.sh test 
for v8.3)
19/19 Checking commit f5a36bd9ce81 (tests/tcg: take into account expected 
clashes pauth-4)
WARNING: Block comments use a leading /* on a separate line
#62: FILE: tests/tcg/aarch64/pauth-4.c:25:
+            : /* out */ "=r"(x), "=r"(y)

WARNING: Block comments use a leading /* on a separate line
#63: FILE: tests/tcg/aarch64/pauth-4.c:26:
+            : /* in */ [in] "r" (in)

ERROR: space prohibited before open square bracket '['
#63: FILE: tests/tcg/aarch64/pauth-4.c:26:
+            : /* in */ [in] "r" (in)

WARNING: Block comments use a leading /* on a separate line
#64: FILE: tests/tcg/aarch64/pauth-4.c:27:
+            : /* clobbers */);

total: 1 errors, 3 warnings, 62 lines checked

Patch 19/19 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

=== OUTPUT END ===

Test command exited with code: 1


The full log is available at
http://patchew.org/logs/address@hidden/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to address@hidden