[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1 6/7] kvm/x86: Hyper-V SynIC message slot pend
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v1 6/7] kvm/x86: Hyper-V SynIC message slot pending clearing at SINT ack |
Date: |
Wed, 25 Nov 2015 18:14:43 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 25/11/2015 17:55, Andrey Smetanin wrote:
>>
>> + gpa = synic->msg_page & PAGE_MASK;
>> + page = kvm_vcpu_gfn_to_page(vcpu, gpa >> PAGE_SHIFT);
>> + if (is_error_page(page)) {
>> + vcpu_err(vcpu, "Hyper-V SynIC can't get msg page, gpa 0x%llx\n",
>> + gpa);
>> + return;
>> + }
>> + msg_page = kmap_atomic(page);
>
> But the message page is not being pinned, is it?
>
> Actually I don't know anything about pinning.
> Is it pinning against page swapping ?
Yes. Unless the page is pinned, kmap_atomic can fail.
However, I don't think that kvm_hv_notify_acked_sint is called from
atomic context. It is only called from apic_set_eoi. Could you just
use kvm_vcpu_write_guest_page?
By the way, do you need to do this also in kvm_get_apic_interrupt, for
auto EOI interrupts?
Thanks,
Paolo
> Could you please clarify and provide an API to use in this case ?
- [Qemu-devel] [PATCH v1 1/7] drivers/hv: Move HV_SYNIC_STIMER_COUNT into Hyper-V UAPI x86 header, (continued)
- [Qemu-devel] [PATCH v1 1/7] drivers/hv: Move HV_SYNIC_STIMER_COUNT into Hyper-V UAPI x86 header, Andrey Smetanin, 2015/11/25
- [Qemu-devel] [PATCH v1 3/7] kvm/x86: Rearrange func's declarations inside Hyper-V header, Andrey Smetanin, 2015/11/25
- [Qemu-devel] [PATCH v1 4/7] kvm/x86: Added Hyper-V vcpu_to_hv_vcpu()/hv_vcpu_to_vcpu() helpers, Andrey Smetanin, 2015/11/25
- [Qemu-devel] [PATCH v1 2/7] drivers/hv: Move struct hv_message into UAPI Hyper-V x86 header, Andrey Smetanin, 2015/11/25
- [Qemu-devel] [PATCH v1 5/7] kvm/x86: Hyper-V internal helper to read MSR HV_X64_MSR_TIME_REF_COUNT, Andrey Smetanin, 2015/11/25
- [Qemu-devel] [PATCH v1 7/7] kvm/x86: Hyper-V SynIC timers, Andrey Smetanin, 2015/11/25
- [Qemu-devel] [PATCH v1 6/7] kvm/x86: Hyper-V SynIC message slot pending clearing at SINT ack, Andrey Smetanin, 2015/11/25
- Re: [Qemu-devel] [PATCH v1 6/7] kvm/x86: Hyper-V SynIC message slot pending clearing at SINT ack, Paolo Bonzini, 2015/11/25
- Re: [Qemu-devel] [PATCH v1 6/7] kvm/x86: Hyper-V SynIC message slot pending clearing at SINT ack, Andrey Smetanin, 2015/11/25
- Re: [Qemu-devel] [PATCH v1 6/7] kvm/x86: Hyper-V SynIC message slot pending clearing at SINT ack,
Paolo Bonzini <=
- Re: [Qemu-devel] [PATCH v1 6/7] kvm/x86: Hyper-V SynIC message slot pending clearing at SINT ack, Andrey Smetanin, 2015/11/26
- Re: [Qemu-devel] [PATCH v1 6/7] kvm/x86: Hyper-V SynIC message slot pending clearing at SINT ack, Paolo Bonzini, 2015/11/26
- Re: [Qemu-devel] [PATCH v1 6/7] kvm/x86: Hyper-V SynIC message slot pending clearing at SINT ack, Andrey Smetanin, 2015/11/26
- Re: [Qemu-devel] [PATCH v1 6/7] kvm/x86: Hyper-V SynIC message slot pending clearing at SINT ack, Paolo Bonzini, 2015/11/26
Re: [Qemu-devel] [PATCH v1 6/7] kvm/x86: Hyper-V SynIC message slot pending clearing at SINT ack, Roman Kagan, 2015/11/27
Re: [Qemu-devel] [PATCH v1 0/7] KVM: Hyper-V SynIC timers, Wanpeng Li, 2015/11/26