[Qemu-devel] [PATCH 0/2] hyperv: own protocol header instead of kernel's

Roman Kagan posted 2 patches 6 years, 9 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20170713201522.13765-1-rkagan@virtuozzo.com
Test FreeBSD passed
Test checkpatch passed
Test docker passed
Test s390x passed
target/i386/cpu.h               |  10 +-
target/i386/hyperv-proto.h      | 257 ++++++++++++++++++++++++++++++++++++++++
target/i386/cpu.c               |   4 +-
target/i386/hyperv.c            |   6 +-
target/i386/kvm.c               |  57 +++++----
target/i386/machine.c           |  15 +--
scripts/update-linux-headers.sh |   4 +-
7 files changed, 304 insertions(+), 49 deletions(-)
create mode 100644 target/i386/hyperv-proto.h
[Qemu-devel] [PATCH 0/2] hyperv: own protocol header instead of kernel's
Posted by Roman Kagan 6 years, 9 months ago
Supersede kernel's header for Hyper-V protocol definitions with our own.
The reason is that, since this is a third-party protocol and not a
kernel API, the kernel folks are not happy exposing it in the kernel
uapi.

The patchset is done to allow gradual transition from the kernel's
hyperv.h to the new one: the first patch introduces the new header that
doesn't conflict with the old one, and the second makes sure the old one
isn't imported from the kernel any longer, so that the maintainers can
do the next round of kernel header import at their leisure and things
will keep working.

Once (if) this lands in QEMU I'll post patches to KVM to move its
hyperv.h out of uapi.

This is the first part split out of my former biggish "hyperv fixes and
enhancements" series.  The only change is the underscore replaced with a
dash in the header file name.

Roman Kagan (2):
  hyperv: add header with protocol definitions
  update-linux-headers: prepare for hyperv.h removal

 target/i386/cpu.h               |  10 +-
 target/i386/hyperv-proto.h      | 257 ++++++++++++++++++++++++++++++++++++++++
 target/i386/cpu.c               |   4 +-
 target/i386/hyperv.c            |   6 +-
 target/i386/kvm.c               |  57 +++++----
 target/i386/machine.c           |  15 +--
 scripts/update-linux-headers.sh |   4 +-
 7 files changed, 304 insertions(+), 49 deletions(-)
 create mode 100644 target/i386/hyperv-proto.h

-- 
2.13.0


Re: [Qemu-devel] [PATCH 0/2] hyperv: own protocol header instead of kernel's
Posted by Roman Kagan 6 years, 7 months ago
On Thu, Jul 13, 2017 at 11:15:20PM +0300, Roman Kagan wrote:
> Supersede kernel's header for Hyper-V protocol definitions with our own.
> The reason is that, since this is a third-party protocol and not a
> kernel API, the kernel folks are not happy exposing it in the kernel
> uapi.
> 
> The patchset is done to allow gradual transition from the kernel's
> hyperv.h to the new one: the first patch introduces the new header that
> doesn't conflict with the old one, and the second makes sure the old one
> isn't imported from the kernel any longer, so that the maintainers can
> do the next round of kernel header import at their leisure and things
> will keep working.
> 
> Once (if) this lands in QEMU I'll post patches to KVM to move its
> hyperv.h out of uapi.
> 
> This is the first part split out of my former biggish "hyperv fixes and
> enhancements" series.  The only change is the underscore replaced with a
> dash in the header file name.
> 
> Roman Kagan (2):
>   hyperv: add header with protocol definitions
>   update-linux-headers: prepare for hyperv.h removal
> 
>  target/i386/cpu.h               |  10 +-
>  target/i386/hyperv-proto.h      | 257 ++++++++++++++++++++++++++++++++++++++++
>  target/i386/cpu.c               |   4 +-
>  target/i386/hyperv.c            |   6 +-
>  target/i386/kvm.c               |  57 +++++----
>  target/i386/machine.c           |  15 +--
>  scripts/update-linux-headers.sh |   4 +-
>  7 files changed, 304 insertions(+), 49 deletions(-)
>  create mode 100644 target/i386/hyperv-proto.h
> 
> -- 
> 2.13.0
> 
> 

Ping?

Roman.

Re: [Qemu-devel] [PATCH 0/2] hyperv: own protocol header instead of kernel's
Posted by Paolo Bonzini 6 years, 7 months ago
On 31/08/2017 18:37, Roman Kagan wrote:
> On Thu, Jul 13, 2017 at 11:15:20PM +0300, Roman Kagan wrote:
>> Supersede kernel's header for Hyper-V protocol definitions with our own.
>> The reason is that, since this is a third-party protocol and not a
>> kernel API, the kernel folks are not happy exposing it in the kernel
>> uapi.
>>
>> The patchset is done to allow gradual transition from the kernel's
>> hyperv.h to the new one: the first patch introduces the new header that
>> doesn't conflict with the old one, and the second makes sure the old one
>> isn't imported from the kernel any longer, so that the maintainers can
>> do the next round of kernel header import at their leisure and things
>> will keep working.
>>
>> Once (if) this lands in QEMU I'll post patches to KVM to move its
>> hyperv.h out of uapi.
>>
>> This is the first part split out of my former biggish "hyperv fixes and
>> enhancements" series.  The only change is the underscore replaced with a
>> dash in the header file name.

Queued, thanks.

Paolo

>> Roman Kagan (2):
>>   hyperv: add header with protocol definitions
>>   update-linux-headers: prepare for hyperv.h removal
>>
>>  target/i386/cpu.h               |  10 +-
>>  target/i386/hyperv-proto.h      | 257 ++++++++++++++++++++++++++++++++++++++++
>>  target/i386/cpu.c               |   4 +-
>>  target/i386/hyperv.c            |   6 +-
>>  target/i386/kvm.c               |  57 +++++----
>>  target/i386/machine.c           |  15 +--
>>  scripts/update-linux-headers.sh |   4 +-
>>  7 files changed, 304 insertions(+), 49 deletions(-)
>>  create mode 100644 target/i386/hyperv-proto.h
>>
>> -- 
>> 2.13.0
>>
>>
> 
> Ping?
> 
> Roman.
>