Re: [Qemu-devel] [PATCH 16/23] hyperv: map overlay pages after updating msrs

2017-06-14 Thread Paolo Bonzini
On 14/06/2017 14:41, Roman Kagan wrote: > On Wed, Jun 14, 2017 at 02:11:56PM +0200, Paolo Bonzini wrote: >> On 14/06/2017 13:54, Roman Kagan wrote: Why not disable the zeroing for host-initiated MSR writes? This is pretty clearly a KVM bug, we can push it to stable kernels too. >>> >>>

Re: [Qemu-devel] [PATCH 16/23] hyperv: map overlay pages after updating msrs

2017-06-14 Thread Roman Kagan
On Wed, Jun 14, 2017 at 02:11:56PM +0200, Paolo Bonzini wrote: > On 14/06/2017 13:54, Roman Kagan wrote: > >> Why not disable the zeroing for host-initiated MSR writes? This is > >> pretty clearly a KVM bug, we can push it to stable kernels too. > > > > The only problem with this is that QEMU will

Re: [Qemu-devel] [PATCH 16/23] hyperv: map overlay pages after updating msrs

2017-06-14 Thread Paolo Bonzini
On 14/06/2017 13:54, Roman Kagan wrote: >> Why not disable the zeroing for host-initiated MSR writes? This is >> pretty clearly a KVM bug, we can push it to stable kernels too. > > The only problem with this is that QEMU will have no reliable way to > know if the KVM it runs with has this bug fi

Re: [Qemu-devel] [PATCH 16/23] hyperv: map overlay pages after updating msrs

2017-06-14 Thread Roman Kagan
On Wed, Jun 14, 2017 at 01:12:12PM +0200, Paolo Bonzini wrote: > > > On 06/06/2017 20:19, Roman Kagan wrote: > > There is a design flaw in the Hyper-V SynIC implementation in KVM: when > > message page or event flags page is enabled by setting the corresponding > > msr, KVM zeroes it out. This v

Re: [Qemu-devel] [PATCH 16/23] hyperv: map overlay pages after updating msrs

2017-06-14 Thread Paolo Bonzini
On 06/06/2017 20:19, Roman Kagan wrote: > There is a design flaw in the Hyper-V SynIC implementation in KVM: when > message page or event flags page is enabled by setting the corresponding > msr, KVM zeroes it out. This violates the spec in general (per spec, > the pages have to be overlay ones

[Qemu-devel] [PATCH 16/23] hyperv: map overlay pages after updating msrs

2017-06-06 Thread Roman Kagan
There is a design flaw in the Hyper-V SynIC implementation in KVM: when message page or event flags page is enabled by setting the corresponding msr, KVM zeroes it out. This violates the spec in general (per spec, the pages have to be overlay ones and only zeroed at cpu reset), but it's non-fatal