mbox series

[0/6] Update Linux headers to 4.17-rc6

Message ID 20180525132755.21839-1-peter.maydell@linaro.org
Headers show
Series Update Linux headers to 4.17-rc6 | expand

Message

Peter Maydell May 25, 2018, 1:27 p.m. UTC
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%)

Comments

no-reply@patchew.org May 25, 2018, 1:35 p.m. UTC | #1
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
Michael S. Tsirkin May 25, 2018, 2 p.m. UTC | #2
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
Paolo Bonzini May 28, 2018, 7:38 a.m. UTC | #3
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
>