On Fri, Dec 16, 2016 at 01:32:04PM -0800, no-re...@patchew.org wrote: > Hi, > > Your series seems to have some coding style problems. See output below for > more information: > > Type: series > Subject: [Qemu-devel] [PULL for-2.9 0/9] virtio, vhost, pc: fixes > Message-id: 1481922841-4324-1-git-send-email-...@redhat.com > > === TEST SCRIPT BEGIN === > #!/bin/bash > > BASE=base > n=1 > total=$(git log --oneline $BASE.. | wc -l) > failed=0 > > # Useful git options > git config --local diff.renamelimit 0 > git config --local diff.renames True > > 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/1481922841-4324-1-git-send-email-...@redhat.com > -> patchew/1481922841-4324-1-git-send-email-...@redhat.com > Switched to a new branch 'test' > f99ab3b virtio: avoid using guest_notifier_mask in vhost-user mode > 3fb701c pci: fix error message for express slots > 88360e6 i386: amd_iommu: fix MMIO register count and access > 65e5776 tests/vhost-user-bridge: use contrib/libvhost-user > b70c976 contrib: add libvhost-user > b641366 tests/vhost-user-bridge: do not accept more than one connection > a1416a7 tests/vhost-user-bridge: indicate peer disconnected > 5ced0b7 tests/vhost-user-bridge: remove unnecessary dispatcher_remove > ee5fbf8 tests/vhost-user-bridge: remove false comment
The strange thing here is that this actually seems to apply patches from list instead of getting the tag from the tree. > === OUTPUT BEGIN === > Checking PATCH 1/9: tests/vhost-user-bridge: remove false comment... > Checking PATCH 2/9: tests/vhost-user-bridge: remove unnecessary > dispatcher_remove... > Checking PATCH 3/9: tests/vhost-user-bridge: indicate peer disconnected... > Checking PATCH 4/9: tests/vhost-user-bridge: do not accept more than one > connection... > Checking PATCH 5/9: contrib: add libvhost-user... > ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8 > #75: FILE: contrib/libvhost-user/libvhost-user.c:9: > + * Marc-Andr Lureau <mlur...@redhat.com> > ^ > > ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8 > #1579: FILE: contrib/libvhost-user/libvhost-user.h:8: > + * Marc-Andr Lureau <mlur...@redhat.com> > ^ > > total: 2 errors, 0 warnings, 1950 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/9: tests/vhost-user-bridge: use contrib/libvhost-user... > Checking PATCH 7/9: i386: amd_iommu: fix MMIO register count and access... > Checking PATCH 8/9: pci: fix error message for express slots... > Checking PATCH 9/9: virtio: avoid using guest_notifier_mask in vhost-user > mode... > === OUTPUT END === > > Test command exited with code: 1 > > > --- > Email generated automatically by Patchew [http://patchew.org/]. > Please send your feedback to patchew-de...@freelists.org
