[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 00/38] Linux user for 5.0 patches
From: |
no-reply |
Subject: |
Re: [PULL 00/38] Linux user for 5.0 patches |
Date: |
Mon, 16 Mar 2020 19:01:48 -0700 (PDT) |
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: [PULL 00/38] Linux user for 5.0 patches
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
Switched to a new branch 'test'
e0e5c4d41 linux-user, openrisc: sync syscall numbers with kernel v5.5
2910e84 linux-user, nios2: sync syscall numbers with kernel v5.5
8c86b7e linux-user, aarch64: sync syscall numbers with kernel v5.5
6e455d8 scripts: add a script to generate syscall_nr.h
0e2de33 linux-user,mips: update syscall-args-o32.c.inc
d4f3bcb linux-user,mips: move content of mips_syscall_args
480aff5 linux-user: update syscall.tbl from linux 0bf999f9c5e7
3fc8443 linux-user, scripts: add a script to update syscall.tbl
0948216 linux-user, mips64: add syscall table generation support
b9ae2f7 linux-user, mips: add syscall table generation support
a047c7e linux-user, x86_64: add syscall table generation support
9585b9f linux-user, i386: add syscall table generation support
95211fc linux-user, x86_64, i386: cleanup TARGET_NR_arch_prctl
f1b32e0 linux-user, sparc, sparc64: add syscall table generation support
54f036b linux-user, s390x: add syscall table generation support
6a847de linux-user, s390x: remove syscall definitions for !TARGET_S390X
9e0df32 linux-user, ppc: add syscall table generation support
ed29cb5 linux-user, arm: add syscall table generation support
50fd494 linux-user, microblaze: add syscall table generation support
bfe6334 linux-user, sh4: add syscall table generation support
976cf65 linux-user, xtensa: add syscall table generation support
e541b96 linux-user, m68k: add syscall table generation support
192f551 linux-user, hppa: add syscall table generation support
feeb587 linux-user, alpha: add syscall table generation support
9eab8db0 linux-user: introduce parameters to generate syscall_nr.h
c360ebe linux-user/riscv: Update the syscall_nr's to the 5.5 kernel
dd8dcb9 linux-user: Support futex_time64
b3a71ee linux-user/syscall: Add support for clock_gettime64/clock_settime64
e16637d linux-user: Protect more syscalls
8c0e24b linux-user: Update TASK_UNMAPPED_BASE for aarch64
8076da7 linux-user: fix socket() strace
a8ead8b linux-user: do prlimit selectively
53c5852 linux-user: Add AT_EXECFN auxval
8071d57 linux-user: Flush out implementation of gettimeofday
79ab97a linux-user: Add x86_64 vsyscall page to /proc/self/maps
1623ac6 linux-user/i386: Emulate x86_64 vsyscalls
0c9b392 linux-user/i386: Split out gen_signal
6b005bd target/i386: Renumber EXCP_SYSCALL
=== OUTPUT BEGIN ===
1/38 Checking commit 6b005bde41b2 (target/i386: Renumber EXCP_SYSCALL)
2/38 Checking commit 0c9b3925c43e (linux-user/i386: Split out gen_signal)
3/38 Checking commit 1623ac6f2365 (linux-user/i386: Emulate x86_64 vsyscalls)
4/38 Checking commit 79ab97a59627 (linux-user: Add x86_64 vsyscall page to
/proc/self/maps)
5/38 Checking commit 8071d573d622 (linux-user: Flush out implementation of
gettimeofday)
6/38 Checking commit 53c58520ac4a (linux-user: Add AT_EXECFN auxval)
7/38 Checking commit a8ead8b1c548 (linux-user: do prlimit selectively)
8/38 Checking commit 8076da785c55 (linux-user: fix socket() strace)
9/38 Checking commit 8c0e24b13310 (linux-user: Update TASK_UNMAPPED_BASE for
aarch64)
10/38 Checking commit e16637d2e34d (linux-user: Protect more syscalls)
11/38 Checking commit b3a71ee542ba (linux-user/syscall: Add support for
clock_gettime64/clock_settime64)
12/38 Checking commit dd8dcb9f6218 (linux-user: Support futex_time64)
WARNING: architecture specific defines should be avoided
#26: FILE: linux-user/syscall.c:248:
+#if defined(__NR_futex)
WARNING: architecture specific defines should be avoided
#29: FILE: linux-user/syscall.c:251:
+#if defined(__NR_futex_time64)
WARNING: architecture specific defines should be avoided
#40: FILE: linux-user/syscall.c:303:
+#if (defined(TARGET_NR_futex) && defined(__NR_futex)) || \
WARNING: architecture specific defines should be avoided
#46: FILE: linux-user/syscall.c:309:
+#if (defined(TARGET_NR_futex_time64) && defined(__NR_futex_teim64))
ERROR: space required after that ',' (ctx:VxV)
#47: FILE: linux-user/syscall.c:310:
+_syscall6(int,sys_futex_time64,int *,uaddr,int,op,int,val,
^
ERROR: space required after that ',' (ctx:VxV)
#47: FILE: linux-user/syscall.c:310:
+_syscall6(int,sys_futex_time64,int *,uaddr,int,op,int,val,
^
ERROR: space required after that ',' (ctx:OxV)
#47: FILE: linux-user/syscall.c:310:
+_syscall6(int,sys_futex_time64,int *,uaddr,int,op,int,val,
^
ERROR: space required after that ',' (ctx:VxV)
#47: FILE: linux-user/syscall.c:310:
+_syscall6(int,sys_futex_time64,int *,uaddr,int,op,int,val,
^
ERROR: space required after that ',' (ctx:VxV)
#47: FILE: linux-user/syscall.c:310:
+_syscall6(int,sys_futex_time64,int *,uaddr,int,op,int,val,
^
ERROR: space required after that ',' (ctx:VxV)
#47: FILE: linux-user/syscall.c:310:
+_syscall6(int,sys_futex_time64,int *,uaddr,int,op,int,val,
^
ERROR: space required after that ',' (ctx:VxV)
#47: FILE: linux-user/syscall.c:310:
+_syscall6(int,sys_futex_time64,int *,uaddr,int,op,int,val,
^
ERROR: space required after that ',' (ctx:OxV)
#48: FILE: linux-user/syscall.c:311:
+ const struct timespec *,timeout,int *,uaddr2,int,val3)
^
ERROR: space required after that ',' (ctx:VxV)
#48: FILE: linux-user/syscall.c:311:
+ const struct timespec *,timeout,int *,uaddr2,int,val3)
^
ERROR: space required after that ',' (ctx:OxV)
#48: FILE: linux-user/syscall.c:311:
+ const struct timespec *,timeout,int *,uaddr2,int,val3)
^
ERROR: space required after that ',' (ctx:VxV)
#48: FILE: linux-user/syscall.c:311:
+ const struct timespec *,timeout,int *,uaddr2,int,val3)
^
ERROR: space required after that ',' (ctx:VxV)
#48: FILE: linux-user/syscall.c:311:
+ const struct timespec *,timeout,int *,uaddr2,int,val3)
^
WARNING: architecture specific defines should be avoided
#58: FILE: linux-user/syscall.c:776:
+#if defined(__NR_futex)
WARNING: architecture specific defines should be avoided
#62: FILE: linux-user/syscall.c:780:
+#if defined(__NR_futex_time64)
ERROR: space required after that ',' (ctx:VxV)
#63: FILE: linux-user/syscall.c:781:
+safe_syscall6(int,futex_time64,int *,uaddr,int,op,int,val, \
^
ERROR: space required after that ',' (ctx:VxV)
#63: FILE: linux-user/syscall.c:781:
+safe_syscall6(int,futex_time64,int *,uaddr,int,op,int,val, \
^
ERROR: space required after that ',' (ctx:OxV)
#63: FILE: linux-user/syscall.c:781:
+safe_syscall6(int,futex_time64,int *,uaddr,int,op,int,val, \
^
ERROR: space required after that ',' (ctx:VxV)
#63: FILE: linux-user/syscall.c:781:
+safe_syscall6(int,futex_time64,int *,uaddr,int,op,int,val, \
^
ERROR: space required after that ',' (ctx:VxV)
#63: FILE: linux-user/syscall.c:781:
+safe_syscall6(int,futex_time64,int *,uaddr,int,op,int,val, \
^
ERROR: space required after that ',' (ctx:VxV)
#63: FILE: linux-user/syscall.c:781:
+safe_syscall6(int,futex_time64,int *,uaddr,int,op,int,val, \
^
ERROR: space required after that ',' (ctx:VxV)
#63: FILE: linux-user/syscall.c:781:
+safe_syscall6(int,futex_time64,int *,uaddr,int,op,int,val, \
^
ERROR: space required after that ',' (ctx:OxV)
#64: FILE: linux-user/syscall.c:782:
+ const struct timespec *,timeout,int *,uaddr2,int,val3)
^
ERROR: space required after that ',' (ctx:VxV)
#64: FILE: linux-user/syscall.c:782:
+ const struct timespec *,timeout,int *,uaddr2,int,val3)
^
ERROR: space required after that ',' (ctx:OxV)
#64: FILE: linux-user/syscall.c:782:
+ const struct timespec *,timeout,int *,uaddr2,int,val3)
^
ERROR: space required after that ',' (ctx:VxV)
#64: FILE: linux-user/syscall.c:782:
+ const struct timespec *,timeout,int *,uaddr2,int,val3)
^
ERROR: space required after that ',' (ctx:VxV)
#64: FILE: linux-user/syscall.c:782:
+ const struct timespec *,timeout,int *,uaddr2,int,val3)
^
WARNING: architecture specific defines should be avoided
#87: FILE: linux-user/syscall.c:6930:
+#if defined(__NR_futex)
WARNING: architecture specific defines should be avoided
#93: FILE: linux-user/syscall.c:6936:
+#if defined(__NR_futex_time64)
WARNING: architecture specific defines should be avoided
#99: FILE: linux-user/syscall.c:6942:
+#if defined(__NR_futex)
WARNING: architecture specific defines should be avoided
#112: FILE: linux-user/syscall.c:6955:
+#if defined(__NR_futex)
WARNING: architecture specific defines should be avoided
#117: FILE: linux-user/syscall.c:6960:
+#if defined(__NR_futex_time64)
WARNING: architecture specific defines should be avoided
#124: FILE: linux-user/syscall.c:6967:
+#if defined(__NR_futex)
WARNING: line over 80 characters
#171: FILE: linux-user/syscall.c:7030:
+static int do_futex_time64(target_ulong uaddr, int op, int val, target_ulong
timeout,
WARNING: Block comments use a leading /* on a separate line
#177: FILE: linux-user/syscall.c:7036:
+ /* ??? We assume FUTEX_* constants are the same on both host
WARNING: Block comments use * on subsequent lines
#178: FILE: linux-user/syscall.c:7037:
+ /* ??? We assume FUTEX_* constants are the same on both host
+ and target. */
WARNING: Block comments use a trailing */ on a separate line
#178: FILE: linux-user/syscall.c:7037:
+ and target. */
WARNING: Block comments use a leading /* on a separate line
#202: FILE: linux-user/syscall.c:7061:
+ /* For FUTEX_REQUEUE, FUTEX_CMP_REQUEUE, and FUTEX_WAKE_OP, the
WARNING: Block comments use * on subsequent lines
#203: FILE: linux-user/syscall.c:7062:
+ /* For FUTEX_REQUEUE, FUTEX_CMP_REQUEUE, and FUTEX_WAKE_OP, the
+ TIMEOUT parameter is interpreted as a uint32_t by the kernel.
WARNING: Block comments use a trailing */ on a separate line
#206: FILE: linux-user/syscall.c:7065:
+ since it's not compared to guest memory. */
total: 24 errors, 19 warnings, 212 lines checked
Patch 12/38 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
13/38 Checking commit c360ebe7de7c (linux-user/riscv: Update the syscall_nr's
to the 5.5 kernel)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#14:
new file mode 100644
total: 0 errors, 1 warnings, 898 lines checked
Patch 13/38 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
14/38 Checking commit 9eab8db04760 (linux-user: introduce parameters to
generate syscall_nr.h)
15/38 Checking commit feeb58754c64 (linux-user, alpha: add syscall table
generation support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#49:
new file mode 100644
total: 0 errors, 1 warnings, 536 lines checked
Patch 15/38 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
16/38 Checking commit 192f5517224c (linux-user, hppa: add syscall table
generation support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#48:
new file mode 100644
total: 0 errors, 1 warnings, 491 lines checked
Patch 16/38 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
17/38 Checking commit e541b963923a (linux-user, m68k: add syscall table
generation support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#47:
new file mode 100644
total: 0 errors, 1 warnings, 493 lines checked
Patch 17/38 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
18/38 Checking commit 976cf651bda4 (linux-user, xtensa: add syscall table
generation support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#47:
new file mode 100644
total: 0 errors, 1 warnings, 464 lines checked
Patch 18/38 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
19/38 Checking commit bfe633493e3f (linux-user, sh4: add syscall table
generation support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#48:
new file mode 100644
total: 0 errors, 1 warnings, 497 lines checked
Patch 19/38 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
20/38 Checking commit 50fd494f1955 (linux-user, microblaze: add syscall table
generation support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#49:
new file mode 100644
total: 0 errors, 1 warnings, 501 lines checked
Patch 20/38 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
21/38 Checking commit ed29cb5bbefd (linux-user, arm: add syscall table
generation support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#62:
new file mode 100644
total: 0 errors, 1 warnings, 531 lines checked
Patch 21/38 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
22/38 Checking commit 9e0df327c4ad (linux-user, ppc: add syscall table
generation support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#74:
new file mode 100644
total: 0 errors, 1 warnings, 608 lines checked
Patch 22/38 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
23/38 Checking commit 6a847de9f011 (linux-user, s390x: remove syscall
definitions for !TARGET_S390X)
24/38 Checking commit 54f036b32733 (linux-user, s390x: add syscall table
generation support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#49:
new file mode 100644
total: 0 errors, 1 warnings, 498 lines checked
Patch 24/38 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
25/38 Checking commit f1b32e0612eb (linux-user, sparc, sparc64: add syscall
table generation support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#60:
new file mode 100644
total: 0 errors, 1 warnings, 1072 lines checked
Patch 25/38 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
26/38 Checking commit 95211fc87250 (linux-user, x86_64, i386: cleanup
TARGET_NR_arch_prctl)
27/38 Checking commit 9585b9f23dee (linux-user, i386: add syscall table
generation support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#50:
new file mode 100644
total: 0 errors, 1 warnings, 497 lines checked
Patch 27/38 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
28/38 Checking commit a047c7eb4e5d (linux-user, x86_64: add syscall table
generation support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#48:
new file mode 100644
total: 0 errors, 1 warnings, 455 lines checked
Patch 28/38 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
29/38 Checking commit b9ae2f7b8c15 (linux-user, mips: add syscall table
generation support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#53:
new file mode 100644
total: 0 errors, 1 warnings, 488 lines checked
Patch 29/38 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
30/38 Checking commit 0948216dd07f (linux-user, mips64: add syscall table
generation support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#61:
new file mode 100644
total: 0 errors, 1 warnings, 802 lines checked
Patch 30/38 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
31/38 Checking commit 3fc8443a1f92 (linux-user, scripts: add a script to update
syscall.tbl)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#30:
new file mode 100755
total: 0 errors, 1 warnings, 56 lines checked
Patch 31/38 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
32/38 Checking commit 480aff5b8af6 (linux-user: update syscall.tbl from linux
0bf999f9c5e7)
33/38 Checking commit d4f3bcb3817a (linux-user,mips: move content of
mips_syscall_args)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#470:
new file mode 100644
total: 0 errors, 1 warnings, 886 lines checked
Patch 33/38 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
34/38 Checking commit 0e2de33421e5 (linux-user,mips: update
syscall-args-o32.c.inc)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#908:
new file mode 100755
total: 0 errors, 1 warnings, 938 lines checked
Patch 34/38 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
35/38 Checking commit 6e455d83bfa0 (scripts: add a script to generate
syscall_nr.h)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#30:
new file mode 100755
total: 0 errors, 1 warnings, 109 lines checked
Patch 35/38 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
36/38 Checking commit 8c86b7e22eaf (linux-user, aarch64: sync syscall numbers
with kernel v5.5)
37/38 Checking commit 2910e845696d (linux-user, nios2: sync syscall numbers
with kernel v5.5)
38/38 Checking commit e0e5c4d41f17 (linux-user, openrisc: sync syscall numbers
with kernel v5.5)
=== 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
- [PULL 26/38] linux-user, x86_64, i386: cleanup TARGET_NR_arch_prctl, (continued)
- [PULL 26/38] linux-user, x86_64, i386: cleanup TARGET_NR_arch_prctl, Laurent Vivier, 2020/03/16
- [PULL 34/38] linux-user,mips: update syscall-args-o32.c.inc, Laurent Vivier, 2020/03/16
- [PULL 28/38] linux-user, x86_64: add syscall table generation support, Laurent Vivier, 2020/03/16
- [PULL 24/38] linux-user, s390x: add syscall table generation support, Laurent Vivier, 2020/03/16
- [PULL 38/38] linux-user, openrisc: sync syscall numbers with kernel v5.5, Laurent Vivier, 2020/03/16
- Re: [PULL 00/38] Linux user for 5.0 patches, Peter Maydell, 2020/03/16
- Re: [PULL 00/38] Linux user for 5.0 patches, no-reply, 2020/03/16
- Re: [PULL 00/38] Linux user for 5.0 patches, no-reply, 2020/03/16
- Re: [PULL 00/38] Linux user for 5.0 patches, no-reply, 2020/03/16
- Re: [PULL 00/38] Linux user for 5.0 patches,
no-reply <=
- Re: [PULL 00/38] Linux user for 5.0 patches, no-reply, 2020/03/16