Re: [Qemu-devel] [PULL 00/28] pc, pci, virtio, vhost: fixes, features

[no-reply]

Hi,

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

Type: series
Message-id: address@hidden
Subject: [Qemu-devel] [PULL 00/28] pc, pci, virtio, vhost: fixes, features

=== TEST SCRIPT BEGIN ===
#!/bin/bash

BASE=base
n=1
total=$(git log --oneline $BASE.. | wc -l)
failed=0

git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram

commits="$(git log --format=%H --reverse $BASE..)"
for c in $commits; do
    echo "Checking PATCH $n/$total: $(git log -n 1 --format=%s $c)..."
    if ! git show $c --format=email | ./scripts/checkpatch.pl --mailback -; then
        failed=1
        echo
    fi
    n=$((n+1))
done

exit $failed
=== TEST SCRIPT END ===

Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
From https://github.com/patchew-project/qemu
 * [new tag]               patchew/address@hidden -> patchew/address@hidden
 t [tag update]            patchew/address@hidden -> patchew/address@hidden
Switched to a new branch 'test'
4c9a6a2ffe intel-iommu: rework the page walk logic
5650cac9fb util: implement simple iova tree
88534312e0 intel-iommu: trace domain id during page walk
8ac709ee1c intel-iommu: pass in address space when page walk
1947a6e8f5 intel-iommu: introduce vtd_page_walk_info
9625294a9c intel-iommu: only do page walk for MAP notifiers
dde4ff462d intel-iommu: add iommu lock
f2f4e4a75c intel-iommu: remove IntelIOMMUNotifierNode
e38afacae2 intel-iommu: send PSI always even if across PDEs
0d36159676 nvdimm: fix typo in label-size definition
471bf9c78b contrib/vhost-user-blk: enable protocol feature for vhost-user-blk
a9ae0b2add hw/virtio: Fix brace Werror with clang 6.0.0
7cbfe1743d libvhost-user: Send messages with no data
195b5bd713 vhost-user+postcopy: Use qemu_set_nonblock
011e39054e virtio: support setting memory region based host notifier
7ecf626817 vhost-user: support receiving file descriptors in slave_read
2f00a166ec vhost-user: add Net prefix to internal state structure
d30090016d linux-headers: add kvm header for mips
6d6f92c8b0 linux-headers: add unistd.h on all arches
0e21fd6465 update-linux-headers.sh: unistd.h, kvm consistency
082496d2b5 linux-headers: drop kvm_para.h
0365135fac x86/cpu: use standard-headers/asm-x86.kvm_para.h
2317795f33 include/standard-headers: add asm-x86/kvm_para.h
1be5f83d10 update-linux-headers.sh: drop kvm_para.h hacks
964e0ee4cf vhost: add trace for IOTLB miss
e679e9a70f virtio-balloon: add hugetlb page allocation counts
a800e0a24e allocate pci id for mdpy
b33c444024 hw/pci-host/q35: Replace hardcoded value with macro

=== OUTPUT BEGIN ===
Checking PATCH 1/28: hw/pci-host/q35: Replace hardcoded value with macro...
WARNING: line over 80 characters
#27: FILE: hw/pci-host/q35.c:538:
+                             mch->pci_address_space, 
MCH_HOST_BRIDGE_SMRAM_C_BASE,

WARNING: line over 80 characters
#29: FILE: hw/pci-host/q35.c:540:
+    memory_region_add_subregion_overlap(mch->system_memory, 
MCH_HOST_BRIDGE_SMRAM_C_BASE,

total: 0 errors, 2 warnings, 35 lines checked

Your patch has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
Checking PATCH 2/28: allocate pci id for mdpy...
Checking PATCH 3/28: virtio-balloon: add hugetlb page allocation counts...
Checking PATCH 4/28: vhost: add trace for IOTLB miss...
Checking PATCH 5/28: update-linux-headers.sh: drop kvm_para.h hacks...
ERROR: line over 90 characters
#48: FILE: scripts/update-linux-headers.sh:121:
+        cp_portable "$tmpdir/include/asm/kvm_para.h" 
"$output/include/standard-headers/asm-$arch"

total: 1 errors, 0 warnings, 44 lines checked

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

Checking PATCH 6/28: include/standard-headers: add asm-x86/kvm_para.h...
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#13: 
new file mode 100644

total: 0 errors, 1 warnings, 121 lines checked

Your patch has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
Checking PATCH 7/28: x86/cpu: use standard-headers/asm-x86.kvm_para.h...
Checking PATCH 8/28: linux-headers: drop kvm_para.h...
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#12: 
deleted file mode 100644

total: 0 errors, 1 warnings, 0 lines checked

Your patch has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
Checking PATCH 9/28: update-linux-headers.sh: unistd.h, kvm consistency...
Checking PATCH 10/28: linux-headers: add unistd.h on all arches...
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#14: 
new file mode 100644

total: 0 errors, 1 warnings, 1017 lines checked

Your patch has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
Checking PATCH 11/28: linux-headers: add kvm header for mips...
Checking PATCH 12/28: vhost-user: add Net prefix to internal state structure...
Checking PATCH 13/28: vhost-user: support receiving file descriptors in 
slave_read...
Checking PATCH 14/28: virtio: support setting memory region based host 
notifier...
Checking PATCH 15/28: vhost-user+postcopy: Use qemu_set_nonblock...
Checking PATCH 16/28: libvhost-user: Send messages with no data...
Checking PATCH 17/28: hw/virtio: Fix brace Werror with clang 6.0.0...
Checking PATCH 18/28: contrib/vhost-user-blk: enable protocol feature for 
vhost-user-blk...
Checking PATCH 19/28: nvdimm: fix typo in label-size definition...
Checking PATCH 20/28: intel-iommu: send PSI always even if across PDEs...
Checking PATCH 21/28: intel-iommu: remove IntelIOMMUNotifierNode...
Checking PATCH 22/28: intel-iommu: add iommu lock...
Checking PATCH 23/28: intel-iommu: only do page walk for MAP notifiers...
Checking PATCH 24/28: intel-iommu: introduce vtd_page_walk_info...
Checking PATCH 25/28: intel-iommu: pass in address space when page walk...
Checking PATCH 26/28: intel-iommu: trace domain id during page walk...
Checking PATCH 27/28: util: implement simple iova tree...
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#32: 
new file mode 100644

total: 0 errors, 1 warnings, 265 lines checked

Your patch has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
Checking PATCH 28/28: intel-iommu: rework the page walk logic...
=== OUTPUT END ===

Test command exited with code: 1


---
Email generated automatically by Patchew [http://patchew.org/].
Please send your feedback to address@hidden