Re: [PULL 00/30] s390x/tcg update

[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:

Message-id: address@hidden
Subject: [PULL 00/30] s390x/tcg update
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 ===

From https://github.com/patchew-project/qemu
 - [tag update]      patchew/address@hidden -> patchew/address@hidden
Switched to a new branch 'test'
8444594 tests/tcg: target/s390x: Test MVC
ee2ed6d tests/tcg: target/s390x: Test MVO
7b0e80e s390x/tcg: MVO: Fault-safe handling
9790bed s390x/tcg: MVST: Fault-safe handling
12658d2 s390x/tcg: MVZ: Fault-safe handling
371cae1 s390x/tcg: MVN: Fault-safe handling
87a7cbd s390x/tcg: MVCIN: Fault-safe handling
200a39a s390x/tcg: NC: Fault-safe handling
08ce1db s390x/tcg: XC: Fault-safe handling
ef2eb0b s390x/tcg: OC: Fault-safe handling
1a01d78 s390x/tcg: MVCLU: Fault-safe handling
25baafe s390x/tcg: MVC: Fault-safe handling on destructive overlaps
8848832 s390x/tcg: MVCS/MVCP: Use access_memmove()
fae34f8 s390x/tcg: Fault-safe memmove
4bb7e2d s390x/tcg: Fault-safe memset
d965ec2 s390x/tcg: Always use MMU_USER_IDX for CONFIG_USER_ONLY
5a5dcf2 s390x/tcg: MVST: Fix storing back the addresses to registers
57c4578 s390x/tcg: MVST: Check for specification exceptions
b45d82e s390x/tcg: MVCS/MVCP: Properly wrap the length
96a5992 s390x/tcg: MVCOS: Lengths are 32 bit in 24/31-bit mode
97a8db7 s390x/tcg: MVCS/MVCP: Check for special operation exceptions
ba37bf0 s390x/tcg: MVCLU/MVCLE: Process max 4k bytes at a time
f2d36b3 s390x/tcg: MVPG: Properly wrap the addresses
2b5d295 s390x/tcg: MVPG: Check for specification exceptions
e8cc891 s390x/tcg: MVC: Use is_destructive_overlap()
afc7b33 s390x/tcg: MVC: Increment the length once
9720059 s390x/tcg: MVCL: Process max 4k bytes at a time
6f05a52 s390x/tcg: MVCL: Detect destructive overlaps
cdeec53 s390x/tcg: MVCL: Zero out unused bits of address
c4938ed s390x/tcg: Reset exception_index to -1 instead of 0

=== OUTPUT BEGIN ===
1/30 Checking commit c4938edb13bc (s390x/tcg: Reset exception_index to -1 
instead of 0)
2/30 Checking commit cdeec532ad35 (s390x/tcg: MVCL: Zero out unused bits of 
address)
3/30 Checking commit 6f05a524b299 (s390x/tcg: MVCL: Detect destructive overlaps)
4/30 Checking commit 97200592f682 (s390x/tcg: MVCL: Process max 4k bytes at a 
time)
5/30 Checking commit afc7b33a5e55 (s390x/tcg: MVC: Increment the length once)
6/30 Checking commit e8cc891074c3 (s390x/tcg: MVC: Use is_destructive_overlap())
7/30 Checking commit 2b5d29504cd0 (s390x/tcg: MVPG: Check for specification 
exceptions)
8/30 Checking commit f2d36b3cc65e (s390x/tcg: MVPG: Properly wrap the addresses)
9/30 Checking commit ba37bf02af24 (s390x/tcg: MVCLU/MVCLE: Process max 4k bytes 
at a time)
10/30 Checking commit 97a8db757a04 (s390x/tcg: MVCS/MVCP: Check for special 
operation exceptions)
11/30 Checking commit 96a599293f36 (s390x/tcg: MVCOS: Lengths are 32 bit in 
24/31-bit mode)
12/30 Checking commit b45d82e2df86 (s390x/tcg: MVCS/MVCP: Properly wrap the 
length)
13/30 Checking commit 57c45783106b (s390x/tcg: MVST: Check for specification 
exceptions)
14/30 Checking commit 5a5dcf274a03 (s390x/tcg: MVST: Fix storing back the 
addresses to registers)
15/30 Checking commit d965ec28dd7c (s390x/tcg: Always use MMU_USER_IDX for 
CONFIG_USER_ONLY)
16/30 Checking commit 4bb7e2d79eb8 (s390x/tcg: Fault-safe memset)
17/30 Checking commit fae34f841740 (s390x/tcg: Fault-safe memmove)
18/30 Checking commit 88488321fd07 (s390x/tcg: MVCS/MVCP: Use access_memmove())
19/30 Checking commit 25baafe3dc44 (s390x/tcg: MVC: Fault-safe handling on 
destructive overlaps)
20/30 Checking commit 1a01d784e526 (s390x/tcg: MVCLU: Fault-safe handling)
21/30 Checking commit ef2eb0b81437 (s390x/tcg: OC: Fault-safe handling)
22/30 Checking commit 08ce1db165f6 (s390x/tcg: XC: Fault-safe handling)
23/30 Checking commit 200a39a6646c (s390x/tcg: NC: Fault-safe handling)
24/30 Checking commit 87a7cbd6ecb8 (s390x/tcg: MVCIN: Fault-safe handling)
25/30 Checking commit 371cae1d272f (s390x/tcg: MVN: Fault-safe handling)
26/30 Checking commit 12658d24f74c (s390x/tcg: MVZ: Fault-safe handling)
27/30 Checking commit 9790bedceafa (s390x/tcg: MVST: Fault-safe handling)
28/30 Checking commit 7b0e80e77ad1 (s390x/tcg: MVO: Fault-safe handling)
29/30 Checking commit ee2ed6d1c8e1 (tests/tcg: target/s390x: Test MVO)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#24: 
new file mode 100644

total: 0 errors, 1 warnings, 29 lines checked

Patch 29/30 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
30/30 Checking commit 8444594d1eba (tests/tcg: target/s390x: Test MVC)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#30: 
new file mode 100644

ERROR: use sigaction to establish signal handlers; signal is not portable
#68: FILE: tests/tcg/s390x/mvc.c:34:
+    if (signal(SIGSEGV, handle_sigsegv) == SIG_ERR) {

total: 1 errors, 1 warnings, 113 lines checked

Patch 30/30 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