mbox series

[v3,00/19] testing & plugin updates

Message ID 20200225124710.14152-1-alex.bennee@linaro.org
Headers show
Series testing & plugin updates | expand

Message

Alex Bennée Feb. 25, 2020, 12:46 p.m. UTC 
Hi,

Nothing over v2 apart from signoffs. It would be nice to get the last
one reviewed before I roll the PR.

 - tests/iotests: be a little more forgiving on the size test

Alex Bennée (13):
  tests/tcg: include a skip runner for pauth3 with plugins
  tests/rcutorture: update usage hint
  tests/rcutorture: better document locking of stats
  tests/rcutorture: mild documenting refactor of update thread
  travis.yml: single-thread build-tcg stages
  tests/iotests: be a little more forgiving on the size test
  tracing: only allow -trace to override -D if set
  docs/devel: document query handle lifetimes
  target/riscv: progressively load the instruction during decode
  tests/plugins: make howvec clean-up after itself.
  tests/tcg: give debug builds a little bit longer
  tests/tcg: fix typo in configure.sh test for v8.3
  tests/tcg: take into account expected clashes pauth-4

Chen Qun (1):
  tests/plugin: prevent uninitialized warning

Emilio G. Cota (1):
  plugins/core: add missing break in cb_to_tcg_flags

Richard Henderson (1):
  tcg: save vaddr temp for plugin usage

Thomas Huth (1):
  travis.yml: Test the s390-ccw build, too

Wainer dos Santos Moschetta (1):
  travis.yml: Fix Travis YAML configuration warnings

Yoshinori Sato (1):
  qemu/bitops.h: Add extract8 and extract16

 docs/devel/tcg-plugins.rst                | 13 +++-
 include/qemu/bitops.h                     | 38 ++++++++++++
 target/riscv/instmap.h                    |  8 +--
 plugins/core.c                            |  1 +
 target/riscv/translate.c                  | 40 ++++++------
 tcg/tcg-op.c                              | 23 ++++++-
 tests/plugin/bb.c                         |  6 +-
 tests/plugin/howvec.c                     | 26 +++++---
 tests/plugin/insn.c                       |  3 +-
 tests/rcutorture.c                        | 74 ++++++++++++++++-------
 tests/tcg/aarch64/pauth-4.c               | 54 +++++++++++------
 trace/control.c                           | 11 +++-
 .travis.yml                               | 23 ++++---
 tests/qemu-iotests/214                    |  4 +-
 tests/tcg/Makefile.target                 |  4 +-
 tests/tcg/aarch64/Makefile.softmmu-target |  2 +
 tests/tcg/configure.sh                    |  2 +-
 17 files changed, 235 insertions(+), 97 deletions(-)

Comments

no-reply@patchew.org Feb. 26, 2020, 1:36 p.m. UTC | #1 
Patchew URL: https://patchew.org/QEMU/20200225124710.14152-1-alex.bennee@linaro.org/



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: 20200225124710.14152-1-alex.bennee@linaro.org
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/158271897341.44994.2741557659975232894.stgit@lep8c.aus.stglabs.ibm.com -> patchew/158271897341.44994.2741557659975232894.stgit@lep8c.aus.stglabs.ibm.com
 - [tag update]      patchew/20200205112041.6003-1-vsementsov@virtuozzo.com -> patchew/20200205112041.6003-1-vsementsov@virtuozzo.com
 - [tag update]      patchew/20200226094357.25061-1-jasowang@redhat.com -> patchew/20200226094357.25061-1-jasowang@redhat.com
 - [tag update]      patchew/20200226102549.12158-1-eric.auger@redhat.com -> patchew/20200226102549.12158-1-eric.auger@redhat.com
 * [new tag]         patchew/20200226122054.366b9cda@table.localdomain -> patchew/20200226122054.366b9cda@table.localdomain
 * [new tag]         patchew/20200226125424.481840-1-eblake@redhat.com -> patchew/20200226125424.481840-1-eblake@redhat.com
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/20200225124710.14152-1-alex.bennee@linaro.org/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com