[PATCH for 5.0-rc3 v1 00/11] more random fixes

Alex Bennée posted 11 patches 4 years ago
Test docker-mingw@fedora passed
Test docker-quick@centos7 passed
Test checkpatch failed
Test FreeBSD passed
Test asan passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200409211529.5269-1-alex.bennee@linaro.org
configure                                  |   3 +-
include/exec/cpu-all.h                     |  25 +-
linux-user/qemu.h                          |  31 +-
target/alpha/cpu-param.h                   |  15 +-
accel/tcg/translate-all.c                  |  15 +-
bsd-user/main.c                            |   4 +-
linux-user/elfload.c                       | 503 ++++++++++-----------
linux-user/flatload.c                      |   6 +
linux-user/main.c                          |  27 +-
linux-user/ppc/signal.c                    |  69 ++-
linux-user/syscall.c                       |  43 +-
target/i386/gdbstub.c                      |   2 +-
target/m68k/helper.c                       |   4 +-
.travis.yml                                |   4 +-
tests/docker/dockerfiles/debian10.docker   |   2 +
tests/docker/dockerfiles/debian9.docker    |   2 -
tests/docker/dockerfiles/fedora.docker     |   2 +-
tests/docker/dockerfiles/travis.docker     |   2 +-
tests/docker/dockerfiles/ubuntu.docker     |   2 +-
tests/docker/dockerfiles/ubuntu1804.docker |   2 +-
tests/docker/test-misc                     |   2 +
21 files changed, 375 insertions(+), 390 deletions(-)
[PATCH for 5.0-rc3 v1 00/11] more random fixes
Posted by Alex Bennée 4 years ago
Hi,

Here are some more random fixes for the tree. In no particular order
we have:

  - A couple of bugs found in the gdbstub GByteArray conversion
  - A trivial fix to /proc/self/stat output
  - An attempt to fix broken PIE builds for Windows (please test!)
  - Some fixes to get "make docker-all-tests" running again
  - Some travis MacOSX tweaks

I've also included the guest base re-factoring patches as it makes it
easier for me to soak test the tree with the sanitiser although those
actual fixes won't go into 5.0 at this late stage.

The following patches need review:

 - linux-user: fix /proc/self/stat handling
 - configure: disable PIE for Windows builds
 - configure: redirect sphinx-build check to config.log
 - tests/docker: add docs FEATURE flag and use for test-misc
 - linux-user: completely re-write init_guest_space

Alex Bennée (5):
  linux-user: completely re-write init_guest_space
  tests/docker: add docs FEATURE flag and use for test-misc
  configure: redirect sphinx-build check to config.log
  configure: disable PIE for Windows builds
  linux-user: fix /proc/self/stat handling

Peter Xu (1):
  gdbstub: i386: Fix gdb_get_reg16() parameter to unbreak gdb

Philippe Mathieu-Daudé (2):
  target/m68k/helper: Fix m68k_fpu_gdb_get_reg() use of GByteArray
  .travis.yml: Build OSX 10.14 with Xcode 10.0

Richard Henderson (3):
  exec/cpu-all: Use bool for have_guest_base
  accel/tcg: Relax va restrictions on 64-bit guests
  linux-user/ppc: Fix padding in mcontext_t for ppc64

 configure                                  |   3 +-
 include/exec/cpu-all.h                     |  25 +-
 linux-user/qemu.h                          |  31 +-
 target/alpha/cpu-param.h                   |  15 +-
 accel/tcg/translate-all.c                  |  15 +-
 bsd-user/main.c                            |   4 +-
 linux-user/elfload.c                       | 503 ++++++++++-----------
 linux-user/flatload.c                      |   6 +
 linux-user/main.c                          |  27 +-
 linux-user/ppc/signal.c                    |  69 ++-
 linux-user/syscall.c                       |  43 +-
 target/i386/gdbstub.c                      |   2 +-
 target/m68k/helper.c                       |   4 +-
 .travis.yml                                |   4 +-
 tests/docker/dockerfiles/debian10.docker   |   2 +
 tests/docker/dockerfiles/debian9.docker    |   2 -
 tests/docker/dockerfiles/fedora.docker     |   2 +-
 tests/docker/dockerfiles/travis.docker     |   2 +-
 tests/docker/dockerfiles/ubuntu.docker     |   2 +-
 tests/docker/dockerfiles/ubuntu1804.docker |   2 +-
 tests/docker/test-misc                     |   2 +
 21 files changed, 375 insertions(+), 390 deletions(-)

-- 
2.20.1


Re: [PATCH for 5.0-rc3 v1 00/11] more random fixes
Posted by no-reply@patchew.org 4 years ago
Patchew URL: https://patchew.org/QEMU/20200409211529.5269-1-alex.bennee@linaro.org/



Hi,

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

Subject: [PATCH for 5.0-rc3 v1 00/11] more random fixes
Message-id: 20200409211529.5269-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 ===

Switched to a new branch 'test'
2779bae .travis.yml: Build OSX 10.14 with Xcode 10.0
ffa69ee linux-user: fix /proc/self/stat handling
da73496 gdbstub: i386: Fix gdb_get_reg16() parameter to unbreak gdb
a7392db target/m68k/helper: Fix m68k_fpu_gdb_get_reg() use of GByteArray
6cdbe4b configure: disable PIE for Windows builds
e70b41a configure: redirect sphinx-build check to config.log
84b0026 tests/docker: add docs FEATURE flag and use for test-misc
e991630 linux-user/ppc: Fix padding in mcontext_t for ppc64
3fe0015 accel/tcg: Relax va restrictions on 64-bit guests
3a50414 exec/cpu-all: Use bool for have_guest_base
8327d01 linux-user: completely re-write init_guest_space

=== OUTPUT BEGIN ===
1/11 Checking commit 8327d0183d84 (linux-user: completely re-write init_guest_space)
ERROR: trailing whitespace
#715: FILE: linux-user/qemu.h:229:
+ * $

total: 1 errors, 0 warnings, 681 lines checked

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

2/11 Checking commit 3a504143c770 (exec/cpu-all: Use bool for have_guest_base)
3/11 Checking commit 3fe00151c0e3 (accel/tcg: Relax va restrictions on 64-bit guests)
ERROR: Macros with complex values should be enclosed in parenthesis
#91: FILE: include/exec/cpu-all.h:182:
+# define GUEST_ADDR_MAX_  ~0ul

total: 1 errors, 0 warnings, 88 lines checked

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

4/11 Checking commit e99163099022 (linux-user/ppc: Fix padding in mcontext_t for ppc64)
5/11 Checking commit 84b002680ba0 (tests/docker: add docs FEATURE flag and use for test-misc)
6/11 Checking commit e70b41a4ac1e (configure: redirect sphinx-build check to config.log)
7/11 Checking commit 6cdbe4bcdb6b (configure: disable PIE for Windows builds)
8/11 Checking commit a7392db3e091 (target/m68k/helper: Fix m68k_fpu_gdb_get_reg() use of GByteArray)
9/11 Checking commit da73496e5afd (gdbstub: i386: Fix gdb_get_reg16() parameter to unbreak gdb)
10/11 Checking commit ffa69ee03e63 (linux-user: fix /proc/self/stat handling)
11/11 Checking commit 2779baef4f14 (.travis.yml: Build OSX 10.14 with Xcode 10.0)
=== OUTPUT END ===

Test command exited with code: 1


The full log is available at
http://patchew.org/logs/20200409211529.5269-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