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 >