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
 * [new tag]         patchew/address@hidden -> patchew/address@hidden
Switched to a new branch 'test'
2cc4ca7 tests/tcg: target/s390x: Test MVC
e9e71c5 tests/tcg: target/s390x: Test MVO
96ecdc7 s390x/tcg: MVO: Fault-safe handling
90370c5 s390x/tcg: MVST: Fault-safe handling
0d7a739 s390x/tcg: MVZ: Fault-safe handling
69e0b44 s390x/tcg: MVN: Fault-safe handling
752f7d7 s390x/tcg: MVCIN: Fault-safe handling
d13fb76 s390x/tcg: NC: Fault-safe handling
35f5eac s390x/tcg: XC: Fault-safe handling
9e18c09 s390x/tcg: OC: Fault-safe handling
c00d7eb s390x/tcg: MVCLU: Fault-safe handling
c4d0f6f s390x/tcg: MVC: Fault-safe handling on destructive overlaps
a237750 s390x/tcg: MVCS/MVCP: Use access_memmove()
1a52654 s390x/tcg: Fault-safe memmove
23a79fe s390x/tcg: Fault-safe memset
0cf4557 s390x/tcg: Always use MMU_USER_IDX for CONFIG_USER_ONLY
b49d8da s390x/tcg: MVST: Fix storing back the addresses to registers
0ffe998 s390x/tcg: MVST: Check for specification exceptions
fa839d7 s390x/tcg: MVCS/MVCP: Properly wrap the length
064b0d8 s390x/tcg: MVCOS: Lengths are 32 bit in 24/31-bit mode
9f7778f s390x/tcg: MVCS/MVCP: Check for special operation exceptions
e39e5aa s390x/tcg: MVCLU/MVCLE: Process max 4k bytes at a time
a46823b s390x/tcg: MVPG: Properly wrap the addresses
b666147 s390x/tcg: MVPG: Check for specification exceptions
bb38f52 s390x/tcg: MVC: Use is_destructive_overlap()
2ec6eb1 s390x/tcg: MVC: Increment the length once
0240a92 s390x/tcg: MVCL: Process max 4k bytes at a time
ae46582 s390x/tcg: MVCL: Detect destructive overlaps
1f4217f s390x/tcg: MVCL: Zero out unused bits of address
145e04f s390x/tcg: Reset exception_index to -1 instead of 0

=== OUTPUT BEGIN ===
1/30 Checking commit 145e04fd5b46 (s390x/tcg: Reset exception_index to -1 
instead of 0)
2/30 Checking commit 1f4217fe98bf (s390x/tcg: MVCL: Zero out unused bits of 
address)
3/30 Checking commit ae46582ae9fc (s390x/tcg: MVCL: Detect destructive overlaps)
4/30 Checking commit 0240a922a5d3 (s390x/tcg: MVCL: Process max 4k bytes at a 
time)
5/30 Checking commit 2ec6eb1f4616 (s390x/tcg: MVC: Increment the length once)
6/30 Checking commit bb38f5238f20 (s390x/tcg: MVC: Use is_destructive_overlap())
7/30 Checking commit b66614710291 (s390x/tcg: MVPG: Check for specification 
exceptions)
8/30 Checking commit a46823ba5527 (s390x/tcg: MVPG: Properly wrap the addresses)
9/30 Checking commit e39e5aa1fb29 (s390x/tcg: MVCLU/MVCLE: Process max 4k bytes 
at a time)
10/30 Checking commit 9f7778fd0154 (s390x/tcg: MVCS/MVCP: Check for special 
operation exceptions)
11/30 Checking commit 064b0d8840dc (s390x/tcg: MVCOS: Lengths are 32 bit in 
24/31-bit mode)
12/30 Checking commit fa839d764a74 (s390x/tcg: MVCS/MVCP: Properly wrap the 
length)
13/30 Checking commit 0ffe9989a152 (s390x/tcg: MVST: Check for specification 
exceptions)
14/30 Checking commit b49d8da94774 (s390x/tcg: MVST: Fix storing back the 
addresses to registers)
15/30 Checking commit 0cf4557440b3 (s390x/tcg: Always use MMU_USER_IDX for 
CONFIG_USER_ONLY)
16/30 Checking commit 23a79fe382e8 (s390x/tcg: Fault-safe memset)
17/30 Checking commit 1a52654156d7 (s390x/tcg: Fault-safe memmove)
18/30 Checking commit a237750a2aa5 (s390x/tcg: MVCS/MVCP: Use access_memmove())
19/30 Checking commit c4d0f6fa363c (s390x/tcg: MVC: Fault-safe handling on 
destructive overlaps)
20/30 Checking commit c00d7ebea2e0 (s390x/tcg: MVCLU: Fault-safe handling)
21/30 Checking commit 9e18c0998a89 (s390x/tcg: OC: Fault-safe handling)
22/30 Checking commit 35f5eacbeb12 (s390x/tcg: XC: Fault-safe handling)
23/30 Checking commit d13fb76627a8 (s390x/tcg: NC: Fault-safe handling)
24/30 Checking commit 752f7d75ad3d (s390x/tcg: MVCIN: Fault-safe handling)
25/30 Checking commit 69e0b44feb4d (s390x/tcg: MVN: Fault-safe handling)
26/30 Checking commit 0d7a739637de (s390x/tcg: MVZ: Fault-safe handling)
27/30 Checking commit 90370c5f7e92 (s390x/tcg: MVST: Fault-safe handling)
28/30 Checking commit 96ecdc71a9a1 (s390x/tcg: MVO: Fault-safe handling)
29/30 Checking commit e9e71c5682ab (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 2cc4ca7e8d71 (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