[PULL 00/12] Linux user for 5.1 patches

Laurent Vivier posted 12 patches 3 years, 8 months ago
Test FreeBSD passed
Test docker-quick@centos7 passed
Test checkpatch failed
Test docker-mingw@fedora passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200704162545.311133-1-laurent@vivier.eu
Maintainers: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>, Riku Voipio <riku.voipio@iki.fi>, Laurent Vivier <laurent@vivier.eu>, Artyom Tarasenko <atar4qemu@gmail.com>
There is a newer version of this series
MAINTAINERS                 |   3 +-
bsd-user/main.c             |   6 +-
configure                   |  10 +
include/exec/user/thunk.h   |   1 +
linux-user/ioctls.h         |  17 +-
linux-user/qemu.h           |  40 +++-
linux-user/sparc/cpu_loop.c |   6 +-
linux-user/strace.c         | 435 +++++++++++++++++++++++++++++++-----
linux-user/strace.list      |  40 ++--
linux-user/syscall.c        | 136 ++++++++---
linux-user/syscall_defs.h   |  15 ++
linux-user/syscall_types.h  |  33 +++
target/sparc/translate.c    |   2 +
thunk.c                     | 154 +++++++++++++
14 files changed, 782 insertions(+), 116 deletions(-)
[PULL 00/12] Linux user for 5.1 patches
Posted by Laurent Vivier 3 years, 8 months ago
The following changes since commit e7651153a8801dad6805d450ea8bef9b46c1adf5:

  Merge remote-tracking branch 'remotes/amarkovic/tags/mips-gsoc-queue-jun-27-2020' into staging (2020-06-27 22:57:36 +0100)

are available in the Git repository at:

  git://github.com/vivier/qemu.git tags/linux-user-for-5.1-pull-request

for you to fetch changes up to 8f902c540eccd72086e8f67d977e30e05659783f:

  MAINTAINERS: update linux-user maintainer (2020-07-04 18:09:29 +0200)

----------------------------------------------------------------
linux-user pull request 2020-07-02

Update linux-user maintainer
Improve strace output for some syscalls
Display contents of ioctl() parameters
Fix sparc64 flushw operation

----------------------------------------------------------------

Chen Gang (1):
  linux-user: syscall: ioctls: support DRM_IOCTL_VERSION

Filip Bozuta (8):
  linux-user: Extend strace support to enable argument printing after
    syscall execution
  linux-user: Add strace support for a group of syscalls
  linux-user: Add strace support for printing argument of syscalls used
    for extended attributes
  linux-user: Add strace support for printing arguments of lseek()
  linux-user: Add strace support for printing arguments of
    chown()/lchown()
  linux-user: Add strace support for printing arguments of fallocate()
  linux-user: Add thunk argument types for SIOCGSTAMP and SIOCGSTAMPNS
  linux-user: Add strace support for printing arguments of ioctl()

Giuseppe Musacchio (2):
  target/sparc: Translate flushw opcode
  linux-user/sparc64: Fix the handling of window spill trap

Riku Voipio (1):
  MAINTAINERS: update linux-user maintainer

 MAINTAINERS                 |   3 +-
 bsd-user/main.c             |   6 +-
 configure                   |  10 +
 include/exec/user/thunk.h   |   1 +
 linux-user/ioctls.h         |  17 +-
 linux-user/qemu.h           |  40 +++-
 linux-user/sparc/cpu_loop.c |   6 +-
 linux-user/strace.c         | 435 +++++++++++++++++++++++++++++++-----
 linux-user/strace.list      |  40 ++--
 linux-user/syscall.c        | 136 ++++++++---
 linux-user/syscall_defs.h   |  15 ++
 linux-user/syscall_types.h  |  33 +++
 target/sparc/translate.c    |   2 +
 thunk.c                     | 154 +++++++++++++
 14 files changed, 782 insertions(+), 116 deletions(-)

-- 
2.26.2


Re: [PULL 00/12] Linux user for 5.1 patches
Posted by no-reply@patchew.org 3 years, 8 months ago
Patchew URL: https://patchew.org/QEMU/20200704162545.311133-1-laurent@vivier.eu/



Hi,

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

Subject: [PULL 00/12] Linux user for 5.1 patches
Type: series
Message-id: 20200704162545.311133-1-laurent@vivier.eu

=== 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 ===

From https://github.com/patchew-project/qemu
 * [new tag]         patchew/20200704162545.311133-1-laurent@vivier.eu -> patchew/20200704162545.311133-1-laurent@vivier.eu
Switched to a new branch 'test'
cb3a4fc MAINTAINERS: update linux-user maintainer
7b3f6cd linux-user: Add strace support for printing arguments of ioctl()
9d17bbc linux-user: Add thunk argument types for SIOCGSTAMP and SIOCGSTAMPNS
58d847d linux-user: Add strace support for printing arguments of fallocate()
03bc028 linux-user: Add strace support for printing arguments of chown()/lchown()
53ad262 linux-user: Add strace support for printing arguments of lseek()
3f67ba4 linux-user: Add strace support for printing argument of syscalls used for extended attributes
161ae19 linux-user: Add strace support for a group of syscalls
db4c17d linux-user: Extend strace support to enable argument printing after syscall execution
22bc812 linux-user: syscall: ioctls: support DRM_IOCTL_VERSION
e97195e linux-user/sparc64: Fix the handling of window spill trap
575e0ad target/sparc: Translate flushw opcode

=== OUTPUT BEGIN ===
1/12 Checking commit 575e0adf7d7e (target/sparc: Translate flushw opcode)
2/12 Checking commit e97195ec31d7 (linux-user/sparc64: Fix the handling of window spill trap)
3/12 Checking commit 22bc81224268 (linux-user: syscall: ioctls: support DRM_IOCTL_VERSION)
4/12 Checking commit db4c17d404f2 (linux-user: Extend strace support to enable argument printing after syscall execution)
5/12 Checking commit 161ae192cebd (linux-user: Add strace support for a group of syscalls)
6/12 Checking commit 3f67ba4ddae0 (linux-user: Add strace support for printing argument of syscalls used for extended attributes)
7/12 Checking commit 53ad262300f4 (linux-user: Add strace support for printing arguments of lseek())
8/12 Checking commit 03bc02818e5f (linux-user: Add strace support for printing arguments of chown()/lchown())
9/12 Checking commit 58d847d0e8eb (linux-user: Add strace support for printing arguments of fallocate())
ERROR: storage class should be at the beginning of the declaration
#72: FILE: linux-user/strace.c:1138:
+UNUSED static struct flags falloc_flags[] = {

total: 1 errors, 0 warnings, 104 lines checked

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

10/12 Checking commit 9d17bbca2cab (linux-user: Add thunk argument types for SIOCGSTAMP and SIOCGSTAMPNS)
11/12 Checking commit 7b3f6cd1f53d (linux-user: Add strace support for printing arguments of ioctl())
12/12 Checking commit cb3a4fcab63a (MAINTAINERS: update linux-user maintainer)
=== OUTPUT END ===

Test command exited with code: 1


The full log is available at
http://patchew.org/logs/20200704162545.311133-1-laurent@vivier.eu/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
Re: [PULL 00/12] Linux user for 5.1 patches
Posted by Peter Maydell 3 years, 8 months ago
On Sat, 4 Jul 2020 at 17:27, Laurent Vivier <laurent@vivier.eu> wrote:
>
> The following changes since commit e7651153a8801dad6805d450ea8bef9b46c1adf5:
>
>   Merge remote-tracking branch 'remotes/amarkovic/tags/mips-gsoc-queue-jun-27-2020' into staging (2020-06-27 22:57:36 +0100)
>
> are available in the Git repository at:
>
>   git://github.com/vivier/qemu.git tags/linux-user-for-5.1-pull-request
>
> for you to fetch changes up to 8f902c540eccd72086e8f67d977e30e05659783f:
>
>   MAINTAINERS: update linux-user maintainer (2020-07-04 18:09:29 +0200)
>
> ----------------------------------------------------------------
> linux-user pull request 2020-07-02
>
> Update linux-user maintainer
> Improve strace output for some syscalls
> Display contents of ioctl() parameters
> Fix sparc64 flushw operation


Applied, thanks.

Please update the changelog at https://wiki.qemu.org/ChangeLog/5.1
for any user-visible changes.

-- PMM

Re: [PULL 00/12] Linux user for 5.1 patches
Posted by no-reply@patchew.org 3 years, 8 months ago
Patchew URL: https://patchew.org/QEMU/20200704162545.311133-1-laurent@vivier.eu/



Hi,

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

Subject: [PULL 00/12] Linux user for 5.1 patches
Type: series
Message-id: 20200704162545.311133-1-laurent@vivier.eu

=== 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
 - [tag update]      patchew/20200703090816.3295-1-vsementsov@virtuozzo.com -> patchew/20200703090816.3295-1-vsementsov@virtuozzo.com
 - [tag update]      patchew/20200704144943.18292-1-f4bug@amsat.org -> patchew/20200704144943.18292-1-f4bug@amsat.org
 - [tag update]      patchew/20200704162545.311133-1-laurent@vivier.eu -> patchew/20200704162545.311133-1-laurent@vivier.eu
 * [new tag]         patchew/20200704163927.28188-1-philmd@redhat.com -> patchew/20200704163927.28188-1-philmd@redhat.com
Switched to a new branch 'test'
694f568 MAINTAINERS: update linux-user maintainer
171c990 linux-user: Add strace support for printing arguments of ioctl()
a424bb6 linux-user: Add thunk argument types for SIOCGSTAMP and SIOCGSTAMPNS
27bf310 linux-user: Add strace support for printing arguments of fallocate()
f1b1206 linux-user: Add strace support for printing arguments of chown()/lchown()
9396f03 linux-user: Add strace support for printing arguments of lseek()
5c704cf linux-user: Add strace support for printing argument of syscalls used for extended attributes
7762a3d linux-user: Add strace support for a group of syscalls
e90d86c linux-user: Extend strace support to enable argument printing after syscall execution
7b88c97 linux-user: syscall: ioctls: support DRM_IOCTL_VERSION
8f4ede9 linux-user/sparc64: Fix the handling of window spill trap
fbf7216 target/sparc: Translate flushw opcode

=== OUTPUT BEGIN ===
1/12 Checking commit fbf72163152b (target/sparc: Translate flushw opcode)
2/12 Checking commit 8f4ede92f5cb (linux-user/sparc64: Fix the handling of window spill trap)
3/12 Checking commit 7b88c97552f7 (linux-user: syscall: ioctls: support DRM_IOCTL_VERSION)
4/12 Checking commit e90d86c96d96 (linux-user: Extend strace support to enable argument printing after syscall execution)
5/12 Checking commit 7762a3d9062f (linux-user: Add strace support for a group of syscalls)
6/12 Checking commit 5c704cf44345 (linux-user: Add strace support for printing argument of syscalls used for extended attributes)
7/12 Checking commit 9396f0375162 (linux-user: Add strace support for printing arguments of lseek())
8/12 Checking commit f1b120653fe1 (linux-user: Add strace support for printing arguments of chown()/lchown())
9/12 Checking commit 27bf310f6f8f (linux-user: Add strace support for printing arguments of fallocate())
ERROR: storage class should be at the beginning of the declaration
#72: FILE: linux-user/strace.c:1138:
+UNUSED static struct flags falloc_flags[] = {

total: 1 errors, 0 warnings, 104 lines checked

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

10/12 Checking commit a424bb625a68 (linux-user: Add thunk argument types for SIOCGSTAMP and SIOCGSTAMPNS)
11/12 Checking commit 171c990f6863 (linux-user: Add strace support for printing arguments of ioctl())
12/12 Checking commit 694f568cab30 (MAINTAINERS: update linux-user maintainer)
=== OUTPUT END ===

Test command exited with code: 1


The full log is available at
http://patchew.org/logs/20200704162545.311133-1-laurent@vivier.eu/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com