[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] wrong ioctl error handling on dirty pages sync?
From: |
Michael Tokarev |
Subject: |
Re: [Qemu-devel] wrong ioctl error handling on dirty pages sync? |
Date: |
Tue, 5 Sep 2017 14:57:06 +0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 |
05.09.2017 13:33, Stefan Hajnoczi пишет:
> On Mon, Sep 04, 2017 at 02:52:57PM +0200, Ján Poctavek wrote:
>>
>> On 29. 8. 2017 12:08, Stefan Hajnoczi wrote:
>>> On Fri, Aug 25, 2017 at 01:42:55PM +0200, Ján Poctavek wrote:
>>>> Hi guys,
>>>>
>>>> Maybe it is just my lack of understanding, this seems like a bug to me:
>>>>
>>>> To get list of dirty pages, qemu calls kvm_vm_ioctl() with
>>>> KVM_GET_DIRTY_LOG:
>>>> https://github.com/qemu/qemu/blob/v2.10.0-rc4/accel/kvm/kvm-all.c#L494
>>>>
>>>> and considers the ioctl call failed when -1 is returned.
>>>>
>>>> But the kvm_vm_ioctl() itself returns -errno, not the -1 on error:
>>>> https://github.com/qemu/qemu/blob/v2.10.0-rc4/accel/kvm/kvm-all.c#L2142
>>>>
>>>> Thanks in advance for sheding some light into this for me.
>>> Looks like a bug to me. Do you want to send a patch?
>>>
>>> Guidelines on how to submit a patch are here:
>>> http://wiki.qemu.org/Contribute/SubmitAPatch
>>>
>>> Stefan
>>
>> It seems that the patch has already been created a long time ago. But it is
>> still not included:
>>
>> https://lists.nongnu.org/archive/html/qemu-devel/2014-03/msg03996.html
>>
>> Can I help with this somehow?
>
> Michael: Do you know what happened to this -trivial patch?
https://lists.gnu.org/archive/html/qemu-devel/2014-03/msg05347.html
https://lists.gnu.org/archive/html/qemu-devel/2014-03/msg05346.html
https://lists.gnu.org/archive/html/qemu-devel/2014-03/msg05402.html
-- according to the emails, it has been applied, but I don't see it in
the git tree. Hmm..
Peter&
/mjt