Re: [libvirt] [PATCH v2 06/12] qemu: drop fetch and update status functions

2017-02-17 Thread Nikolay Shirokovskiy
On 17.02.2017 11:23, Jiri Denemark wrote: > On Thu, Feb 16, 2017 at 16:15:02 +0100, Jiri Denemark wrote: >> On Wed, Dec 28, 2016 at 17:39:15 +0300, Nikolay Shirokovskiy wrote: >>> qemuMigrationFetchJobStatus is rather inconvinient. If we poll >>> stats for status only then we don't need to

Re: [libvirt] [PATCH v2 06/12] qemu: drop fetch and update status functions

2017-02-17 Thread Jiri Denemark
On Thu, Feb 16, 2017 at 16:15:02 +0100, Jiri Denemark wrote: > On Wed, Dec 28, 2016 at 17:39:15 +0300, Nikolay Shirokovskiy wrote: > > qemuMigrationFetchJobStatus is rather inconvinient. If we poll > > stats for status only then we don't need to update elapsed time. > > Next on some paths we use

Re: [libvirt] [PATCH v2 06/12] qemu: drop fetch and update status functions

2017-02-16 Thread Jiri Denemark
On Wed, Dec 28, 2016 at 17:39:15 +0300, Nikolay Shirokovskiy wrote: > qemuMigrationFetchJobStatus is rather inconvinient. If we poll > stats for status only then we don't need to update elapsed time. > Next on some paths we use this function to get stats only we don't > what to unexpectedly update