| Message ID | 20180525132755.21839-1-peter.maydell@linaro.org |
|---|---|
| Headers | show |
| Series | Update Linux headers to 4.17-rc6 | expand |
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: 20180525132755.21839-1-peter.maydell@linaro.org
Subject: [Qemu-devel] [PATCH 0/6] Update Linux headers to 4.17-rc6
=== 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/20180525132755.21839-1-peter.maydell@linaro.org -> patchew/20180525132755.21839-1-peter.maydell@linaro.org
Switched to a new branch 'test'
30d4ec272e target/i386/kvm.c: Remove compatibility shim for KVM_HINTS_REALTIME
c90891dc4e Update Linux headers to 4.17-rc6
70a9bbf798 target/i386/kvm.c: Handle renaming of KVM_HINTS_DEDICATED
fe7501a7db scripts/update-linux-headers: Handle kernel license no longer being one file
ac4d8a51d6 scripts/update-linux-headers: Handle __aligned_u64
c49a47a568 virtio-gpu-3d: Define VIRTIO_GPU_CAPSET_VIRGL2 elsewhere
=== OUTPUT BEGIN ===
Checking PATCH 1/6: virtio-gpu-3d: Define VIRTIO_GPU_CAPSET_VIRGL2 elsewhere...
Checking PATCH 2/6: scripts/update-linux-headers: Handle __aligned_u64...
Checking PATCH 3/6: scripts/update-linux-headers: Handle kernel license no longer being one file...
Checking PATCH 4/6: target/i386/kvm.c: Handle renaming of KVM_HINTS_DEDICATED...
Checking PATCH 5/6: Update Linux headers to 4.17-rc6...
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#13:
deleted file mode 100644
ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
#930: FILE: linux-headers/LICENSES/preferred/GPL-2.0:71:
+
^
ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
#985: FILE: linux-headers/LICENSES/preferred/GPL-2.0:126:
+
^
ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
#1043: FILE: linux-headers/LICENSES/preferred/GPL-2.0:184:
+
^
ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
#1100: FILE: linux-headers/LICENSES/preferred/GPL-2.0:241:
+
^
ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
#1153: FILE: linux-headers/LICENSES/preferred/GPL-2.0:294:
+
^
total: 5 errors, 1 warnings, 1281 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/6: target/i386/kvm.c: Remove compatibility shim for KVM_HINTS_REALTIME...
=== OUTPUT END ===
Test command exited with code: 1
---
Email generated automatically by Patchew [http://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
On Fri, May 25, 2018 at 02:27:49PM +0100, Peter Maydell wrote: > This series updates our copy of the Linux kernel headers to 4.17-rc6. > To do that we have to fix up some issues: > * we had a hand-hacked definition of VIRTIO_GPU_CAPSET_VIRGL2 > in our old header copy that needs to be moved to a header > that isn't auto-updated > * we need to turn __aligned_u64 into a portable type > * KVM_HINTS_DEDICATED was renamed to KVM_HINTS_REALTIME > * the kernel's licensing info is no longer solely in the COPYING file > > thanks > -- PMM Reviewed-by: Michael S. Tsirkin <mst@redhat.com> > Alex Williamson (1): > virtio-gpu-3d: Define VIRTIO_GPU_CAPSET_VIRGL2 elsewhere > > Peter Maydell (5): > scripts/update-linux-headers: Handle __aligned_u64 > scripts/update-linux-headers: Handle kernel license no longer being > one file > target/i386/kvm.c: Handle renaming of KVM_HINTS_DEDICATED > Update Linux headers to 4.17-rc6 > target/i386/kvm.c: Remove compatibility shim for KVM_HINTS_REALTIME > > include/hw/virtio/virtio-gpu.h | 6 + > include/standard-headers/asm-x86/hyperv.h | 1 - > include/standard-headers/asm-x86/kvm_para.h | 2 +- > include/standard-headers/linux/ethtool.h | 36 +- > include/standard-headers/linux/input.h | 4 +- > include/standard-headers/linux/pci_regs.h | 7 +- > .../standard-headers/linux/virtio_balloon.h | 15 + > include/standard-headers/linux/virtio_gpu.h | 1 - > .../standard-headers/rdma/vmw_pvrdma-abi.h | 49 +-- > linux-headers/asm-arm/kvm.h | 15 + > linux-headers/asm-arm64/kvm.h | 6 + > linux-headers/asm-x86/hyperv.h | 1 - > linux-headers/asm-x86/kvm.h | 19 +- > linux-headers/linux/kvm.h | 30 +- > linux-headers/linux/vfio.h | 27 ++ > target/i386/kvm.c | 2 +- > linux-headers/COPYING | 358 +----------------- > .../LICENSES/exceptions/Linux-syscall-note | 25 ++ > linux-headers/LICENSES/preferred/BSD-2-Clause | 32 ++ > linux-headers/LICENSES/preferred/BSD-3-Clause | 36 ++ > .../{COPYING => LICENSES/preferred/GPL-2.0} | 27 +- > scripts/update-linux-headers.sh | 17 +- > 22 files changed, 310 insertions(+), 406 deletions(-) > delete mode 100644 include/standard-headers/asm-x86/hyperv.h > delete mode 100644 linux-headers/asm-x86/hyperv.h > create mode 100644 linux-headers/LICENSES/exceptions/Linux-syscall-note > create mode 100644 linux-headers/LICENSES/preferred/BSD-2-Clause > create mode 100644 linux-headers/LICENSES/preferred/BSD-3-Clause > copy linux-headers/{COPYING => LICENSES/preferred/GPL-2.0} (96%) > > -- > 2.17.0
On 25/05/2018 16:00, Michael S. Tsirkin wrote: > On Fri, May 25, 2018 at 02:27:49PM +0100, Peter Maydell wrote: >> This series updates our copy of the Linux kernel headers to 4.17-rc6. >> To do that we have to fix up some issues: >> * we had a hand-hacked definition of VIRTIO_GPU_CAPSET_VIRGL2 >> in our old header copy that needs to be moved to a header >> that isn't auto-updated >> * we need to turn __aligned_u64 into a portable type >> * KVM_HINTS_DEDICATED was renamed to KVM_HINTS_REALTIME >> * the kernel's licensing info is no longer solely in the COPYING file >> >> thanks >> -- PMM > > Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Queued, thanks. Paolo > >> Alex Williamson (1): >> virtio-gpu-3d: Define VIRTIO_GPU_CAPSET_VIRGL2 elsewhere >> >> Peter Maydell (5): >> scripts/update-linux-headers: Handle __aligned_u64 >> scripts/update-linux-headers: Handle kernel license no longer being >> one file >> target/i386/kvm.c: Handle renaming of KVM_HINTS_DEDICATED >> Update Linux headers to 4.17-rc6 >> target/i386/kvm.c: Remove compatibility shim for KVM_HINTS_REALTIME >> >> include/hw/virtio/virtio-gpu.h | 6 + >> include/standard-headers/asm-x86/hyperv.h | 1 - >> include/standard-headers/asm-x86/kvm_para.h | 2 +- >> include/standard-headers/linux/ethtool.h | 36 +- >> include/standard-headers/linux/input.h | 4 +- >> include/standard-headers/linux/pci_regs.h | 7 +- >> .../standard-headers/linux/virtio_balloon.h | 15 + >> include/standard-headers/linux/virtio_gpu.h | 1 - >> .../standard-headers/rdma/vmw_pvrdma-abi.h | 49 +-- >> linux-headers/asm-arm/kvm.h | 15 + >> linux-headers/asm-arm64/kvm.h | 6 + >> linux-headers/asm-x86/hyperv.h | 1 - >> linux-headers/asm-x86/kvm.h | 19 +- >> linux-headers/linux/kvm.h | 30 +- >> linux-headers/linux/vfio.h | 27 ++ >> target/i386/kvm.c | 2 +- >> linux-headers/COPYING | 358 +----------------- >> .../LICENSES/exceptions/Linux-syscall-note | 25 ++ >> linux-headers/LICENSES/preferred/BSD-2-Clause | 32 ++ >> linux-headers/LICENSES/preferred/BSD-3-Clause | 36 ++ >> .../{COPYING => LICENSES/preferred/GPL-2.0} | 27 +- >> scripts/update-linux-headers.sh | 17 +- >> 22 files changed, 310 insertions(+), 406 deletions(-) >> delete mode 100644 include/standard-headers/asm-x86/hyperv.h >> delete mode 100644 linux-headers/asm-x86/hyperv.h >> create mode 100644 linux-headers/LICENSES/exceptions/Linux-syscall-note >> create mode 100644 linux-headers/LICENSES/preferred/BSD-2-Clause >> create mode 100644 linux-headers/LICENSES/preferred/BSD-3-Clause >> copy linux-headers/{COPYING => LICENSES/preferred/GPL-2.0} (96%) >> >> -- >> 2.17.0 >
This series updates our copy of the Linux kernel headers to 4.17-rc6. To do that we have to fix up some issues: * we had a hand-hacked definition of VIRTIO_GPU_CAPSET_VIRGL2 in our old header copy that needs to be moved to a header that isn't auto-updated * we need to turn __aligned_u64 into a portable type * KVM_HINTS_DEDICATED was renamed to KVM_HINTS_REALTIME * the kernel's licensing info is no longer solely in the COPYING file thanks -- PMM Alex Williamson (1): virtio-gpu-3d: Define VIRTIO_GPU_CAPSET_VIRGL2 elsewhere Peter Maydell (5): scripts/update-linux-headers: Handle __aligned_u64 scripts/update-linux-headers: Handle kernel license no longer being one file target/i386/kvm.c: Handle renaming of KVM_HINTS_DEDICATED Update Linux headers to 4.17-rc6 target/i386/kvm.c: Remove compatibility shim for KVM_HINTS_REALTIME include/hw/virtio/virtio-gpu.h | 6 + include/standard-headers/asm-x86/hyperv.h | 1 - include/standard-headers/asm-x86/kvm_para.h | 2 +- include/standard-headers/linux/ethtool.h | 36 +- include/standard-headers/linux/input.h | 4 +- include/standard-headers/linux/pci_regs.h | 7 +- .../standard-headers/linux/virtio_balloon.h | 15 + include/standard-headers/linux/virtio_gpu.h | 1 - .../standard-headers/rdma/vmw_pvrdma-abi.h | 49 +-- linux-headers/asm-arm/kvm.h | 15 + linux-headers/asm-arm64/kvm.h | 6 + linux-headers/asm-x86/hyperv.h | 1 - linux-headers/asm-x86/kvm.h | 19 +- linux-headers/linux/kvm.h | 30 +- linux-headers/linux/vfio.h | 27 ++ target/i386/kvm.c | 2 +- linux-headers/COPYING | 358 +----------------- .../LICENSES/exceptions/Linux-syscall-note | 25 ++ linux-headers/LICENSES/preferred/BSD-2-Clause | 32 ++ linux-headers/LICENSES/preferred/BSD-3-Clause | 36 ++ .../{COPYING => LICENSES/preferred/GPL-2.0} | 27 +- scripts/update-linux-headers.sh | 17 +- 22 files changed, 310 insertions(+), 406 deletions(-) delete mode 100644 include/standard-headers/asm-x86/hyperv.h delete mode 100644 linux-headers/asm-x86/hyperv.h create mode 100644 linux-headers/LICENSES/exceptions/Linux-syscall-note create mode 100644 linux-headers/LICENSES/preferred/BSD-2-Clause create mode 100644 linux-headers/LICENSES/preferred/BSD-3-Clause copy linux-headers/{COPYING => LICENSES/preferred/GPL-2.0} (96%)