Re: [qemu-s390x] [Qemu-devel] [PULL 00/10] s390x 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: [Qemu-devel] [PULL 00/10] s390x update
Type: series

=== TEST SCRIPT BEGIN ===
#!/bin/bash
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
 t [tag update]            patchew/address@hidden -> patchew/address@hidden
Switched to a new branch 'test'
30bd10a753 s390x: upgrade status of KVM cores to "supported"
d0c8053e37 s390x/kvm: add tracepoint to ioeventfd interface
58912e7c70 s390x/cpumodel: add z14 GA2 model
f0a763165c s390x/cpumodel: default enable mepoch for z14 and later
b57cf83254 s390x/cpumodel: mepochptff: warn when no mepoch and re-align group 
init
4036f9a67e s390x: add zPCI feature to "qemu" CPU model
625c89c556 target/s390x: Implement STCK et al for CONFIG_USER_ONLY
2a49e616b7 target/s390x: Split out s390-tod.h
1eaa7f0eda s390x: always provide pci support
602fe9bdb0 s390x: Fix the confusing contributions-after-2012 license statements

=== OUTPUT BEGIN ===
1/10 Checking commit 602fe9bdb008 (s390x: Fix the confusing 
contributions-after-2012 license statements)
2/10 Checking commit 1eaa7f0eda15 (s390x: always provide pci support)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#40: 
deleted file mode 100644

total: 0 errors, 1 warnings, 47 lines checked

Patch 2/10 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
3/10 Checking commit 2a49e616b722 (target/s390x: Split out s390-tod.h)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#52: 
new file mode 100644

total: 0 errors, 1 warnings, 57 lines checked

Patch 3/10 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
4/10 Checking commit 625c89c556d5 (target/s390x: Implement STCK et al for 
CONFIG_USER_ONLY)
5/10 Checking commit 4036f9a67eee (s390x: add zPCI feature to "qemu" CPU model)
6/10 Checking commit b57cf832547f (s390x/cpumodel: mepochptff: warn when no 
mepoch and re-align group init)
ERROR: line over 90 characters
#38: FILE: target/s390x/cpu_features.c:468:
+    FEAT_GROUP_INIT("mepochptff", MULTIPLE_EPOCH_PTFF, "PTFF enhancements 
introduced with Multiple-epoch facility"),

total: 1 errors, 0 warnings, 24 lines checked

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

7/10 Checking commit f0a763165c94 (s390x/cpumodel: default enable mepoch for 
z14 and later)
WARNING: line over 80 characters
#31: FILE: hw/s390x/s390-virtio-ccw.c:663:
+    s390_cpudef_group_featoff_greater(14, 1, 
S390_FEAT_GROUP_MULTIPLE_EPOCH_PTFF);

total: 0 errors, 1 warnings, 55 lines checked

Patch 7/10 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
8/10 Checking commit 58912e7c7085 (s390x/cpumodel: add z14 GA2 model)
9/10 Checking commit d0c8053e3756 (s390x/kvm: add tracepoint to ioeventfd 
interface)
10/10 Checking commit 30bd10a75380 (s390x: upgrade status of KVM cores to 
"supported")
=== 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 [http://patchew.org/].
Please send your feedback to address@hidden