Re: [PATCH v2 00/12] qemu: migration: show disks stats for nbd migration

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



ping

On 28.12.2016 17:39, Nikolay Shirokovskiy wrote:
> diff from v1:
> ============
> 1. patch  "qemu: clean out unused migrate to unix" is dropped
>   as it is already pushed.
> 2. a lot of refactoring patches added, namely all except 
>   the last patch.
> 3. fetching mirroring stats is done separately from getting
>   migration status. Generally speaking refactorings patches
>   removes the function to fetch migrations status altogether.
> 
> Current migration stats will show something like [1] when in
> the process of mirroring of non shared disks. This gives very
> little info on the migration progress. Likewise completed stats miss
> disks mirroring info.
> 
> This patch provides disks stats in the said phase like in [2] so 
> user can now understand what's going on. However data stats miss
> memory stats, so data total and remaining will change when memory
> migration starts.
> 
> AFAIU disks stats were available before the nbd based migration
> becomes the default. So this patch returns disks stats back at
> some level.
> 
>   [1]
> Job type:         Unbounded   
> Time elapsed:     4964         ms
> 
>   [2]
> Job type:         Unbounded   
> Time elapsed:     4964         ms
> Data processed:   146.000 MiB
> Data remaining:   854.000 MiB
> Data total:       1000.000 MiB
> File processed:   146.000 MiB
> File remaining:   854.000 MiB
> File total:       1000.000 MiB
> 
> Nikolay Shirokovskiy (12):
>   qemu: qemuDomainJobInfoToParams drop unused code
>   qemu: introduce qemu domain job status
>   qemu: introduce QEMU_DOMAIN_JOB_STATUS_POSTCOPY
>   qemu: drop QEMU_MIGRATION_COMPLETED_UPDATE_STATS
>   qemu: drop excessive zero-out in qemuMigrationFetchJobStatus
>   qemu: drop fetch and update status functions
>   qemu: simplify getting completed job stats
>   qemu: drop unused code in qemuDomainGetJobStatsInternal
>   qemu: drop fetch flag in qemuDomainGetJobStatsInternal
>   qemu: split getting stats for migration and others
>   qemu: introduce QEMU_DOMAIN_JOB_STATUS_PREPARE
>   qemu: show disks stats for nbd migration
> 
>  docs/news.html.in         |   4 +
>  src/qemu/qemu_domain.c    |  32 ++++++--
>  src/qemu/qemu_domain.h    |  13 +++-
>  src/qemu/qemu_driver.c    | 100 ++++++++++++++----------
>  src/qemu/qemu_migration.c | 195 ++++++++++++++++++++++++++--------------------
>  src/qemu/qemu_migration.h |   8 +-
>  src/qemu/qemu_process.c   |   6 +-
>  7 files changed, 214 insertions(+), 144 deletions(-)
> 

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux