<div dir="ltr"><div><div><div>Thank you.<br><br></div>I will go through your comments just in case I'm going to push any other changes.<br><br></div>Sorry for the inconvenience from my patches :-/.<br><br></div>Pavel<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, May 25, 2015 at 5:58 PM, Michal Privoznik <span dir="ltr"><<a href="mailto:mprivozn@redhat.com" target="_blank">mprivozn@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On <a href="tel:21.05.2015%2013" value="+12105201513">21.05.2015 13</a>:07, Pavel Boldin wrote:<br>
> The patchset represented in the mail thread implements the selective block<br>
> device migration for the QEMU driver. This closes the referenced bug [1].<br>
><br>
> First the supplementary API implemented allowing for multiple key-values pair<br>
> in the virTypedParameter arrays. This is used to pass the list of block<br>
> devices to migrate in the following patches. Unit tests for this are added as<br>
> well. This is the subject of the first four patches.<br>
><br>
> Fifth patch is adding the necessary parameter `migrate_disks' and passes it<br>
> through the QEMU driver call stack. The introduced `qemuMigrateDisk' function<br>
> is then checks that the disk is to be migrated because it is in the list. If<br>
> there is no list specified then the legacy check is used: the device is migrate<br>
> if it is not shared, not readonly and has a source.<br>
><br>
> Sixth and the last patch is adding the necessary code to the `virsh' utility<br>
> making it able for user to specify a comma-separated list of the block device<br>
> names that are to be migrated.<br>
><br>
> The implemented Python bindings patch is to be presented.<br>
><br>
> Changes in v2:<br>
>  * Addressed review comments<br>
>  * Removed all the duplicates check in the commit 'multi-value parameters in<br>
>     virTypedParamsAdd*'<br>
>  * reimplemented virTypedParamsPick as virTypedParamsFilter<br>
>  * renamed virTypedParamsPackStrings to virTypedParamsAddStringList<br>
><br>
> [1] <a href="https://bugzilla.redhat.com/show_bug.cgi?id=1203032" target="_blank">https://bugzilla.redhat.com/show_bug.cgi?id=1203032</a><br>
><br>
> Pavel Boldin (6):<br>
>   util: multi-value virTypedParameter<br>
>   util: multi-value parameters in virTypedParamsAdd*<br>
>   util: virTypedParams{Filter,PickStrings}<br>
>   util: add virTypedParamsAddStringList<br>
>   qemu: migration: selective block device migration<br>
>   virsh: selective block device migration<br>
><br>
>  include/libvirt/libvirt-domain.h |   9 ++<br>
>  include/libvirt/libvirt-host.h   |  16 +++<br>
>  src/libvirt_public.syms          |   7 +<br>
>  src/qemu/qemu_driver.c           |  66 ++++++---<br>
>  src/qemu/qemu_migration.c        | 174 +++++++++++++++--------<br>
>  src/qemu/qemu_migration.h        |  23 ++--<br>
>  src/util/virtypedparam.c         | 263 ++++++++++++++++++++++++++++-------<br>
>  src/util/virtypedparam.h         |  10 ++<br>
>  tests/Makefile.am                |   6 +<br>
>  tests/virtypedparamtest.c        | 291 +++++++++++++++++++++++++++++++++++++++<br>
>  tools/virsh-domain.c             |  23 ++++<br>
>  tools/virsh.pod                  |   5 +-<br>
>  12 files changed, 750 insertions(+), 143 deletions(-)<br>
>  create mode 100644 tests/virtypedparamtest.c<br>
><br>
<br>
</div></div>So, I think we are almost there. I've pointed out a few things.<br>
Moreover, as I've been reviewing I keep fixing the nits I'm raising. So<br>
instead of me giving you headache of posting v3, I'll do that.<br>
<span class="HOEnZb"><font color="#888888"><br>
Michal<br>
</font></span></blockquote></div><br></div>