[PATCH v5 0/6] migration: Test the new "file:" migration

Fabiano Rosas posted 6 patches 10 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20230712190742.22294-1-farosas@suse.de
Maintainers: Juan Quintela <quintela@redhat.com>, Peter Xu <peterx@redhat.com>, Leonardo Bras <leobras@redhat.com>, Thomas Huth <thuth@redhat.com>, Laurent Vivier <lvivier@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>
migration/migration.c             |   7 +-
tests/qtest/libqtest.c            |  33 ++++++
tests/qtest/libqtest.h            |  28 +++++
tests/qtest/meson.build           |   1 +
tests/qtest/migration-helpers.c   |  60 ++++++++++
tests/qtest/migration-helpers.h   |  10 ++
tests/qtest/migration-test.c      | 185 ++++++++++++++++++++++++++----
tests/qtest/virtio-net-failover.c |  77 ++-----------
8 files changed, 308 insertions(+), 93 deletions(-)
[PATCH v5 0/6] migration: Test the new "file:" migration
Posted by Fabiano Rosas 10 months ago
Based-on:
[PATCH V4 0/2] migration file URI
https://lore.kernel.org/r/1688135108-316997-1-git-send-email-steven.sistare@oracle.com

Since v4:

- Implemented a separate version of test_precopy_common to be used
  with the file transport (patch 6).

v4:
https://lore.kernel.org/r/20230706201927.15442-1-farosas@suse.de

v3:
https://lore.kernel.org/r/20230630212902.19925-1-farosas@suse.de

v2:
https://lore.kernel.org/r/20230628165542.17214-1-farosas@suse.de

v1:
https://lore.kernel.org/r/20230626182210.8792-1-farosas@suse.de

Fabiano Rosas (6):
  tests/qtest: migration: Expose migrate_set_capability
  tests/qtest: migration: Add migrate_incoming_qmp helper
  tests/qtest: migration: Use migrate_incoming_qmp where appropriate
  migration: Set migration status early in incoming side
  tests/qtest: migration: Add support for negative testing of
    qmp_migrate
  tests/qtest: migration-test: Add tests for file-based migration

 migration/migration.c             |   7 +-
 tests/qtest/libqtest.c            |  33 ++++++
 tests/qtest/libqtest.h            |  28 +++++
 tests/qtest/meson.build           |   1 +
 tests/qtest/migration-helpers.c   |  60 ++++++++++
 tests/qtest/migration-helpers.h   |  10 ++
 tests/qtest/migration-test.c      | 185 ++++++++++++++++++++++++++----
 tests/qtest/virtio-net-failover.c |  77 ++-----------
 8 files changed, 308 insertions(+), 93 deletions(-)

-- 
2.35.3
Re: [PATCH v5 0/6] migration: Test the new "file:" migration
Posted by Claudio Fontana 8 months, 2 weeks ago
Hello,

this series is all reviewed,

and is needed as a precondition for further work to improve dramatically the performance of virsh save, virsh restore
when migrating to disk, can it be merged?

Thanks,

Claudio

On 7/12/23 21:07, Fabiano Rosas wrote:
> Based-on:
> [PATCH V4 0/2] migration file URI
> https://lore.kernel.org/r/1688135108-316997-1-git-send-email-steven.sistare@oracle.com
> 
> Since v4:
> 
> - Implemented a separate version of test_precopy_common to be used
>   with the file transport (patch 6).
> 
> v4:
> https://lore.kernel.org/r/20230706201927.15442-1-farosas@suse.de
> 
> v3:
> https://lore.kernel.org/r/20230630212902.19925-1-farosas@suse.de
> 
> v2:
> https://lore.kernel.org/r/20230628165542.17214-1-farosas@suse.de
> 
> v1:
> https://lore.kernel.org/r/20230626182210.8792-1-farosas@suse.de
> 
> Fabiano Rosas (6):
>   tests/qtest: migration: Expose migrate_set_capability
>   tests/qtest: migration: Add migrate_incoming_qmp helper
>   tests/qtest: migration: Use migrate_incoming_qmp where appropriate
>   migration: Set migration status early in incoming side
>   tests/qtest: migration: Add support for negative testing of
>     qmp_migrate
>   tests/qtest: migration-test: Add tests for file-based migration
> 
>  migration/migration.c             |   7 +-
>  tests/qtest/libqtest.c            |  33 ++++++
>  tests/qtest/libqtest.h            |  28 +++++
>  tests/qtest/meson.build           |   1 +
>  tests/qtest/migration-helpers.c   |  60 ++++++++++
>  tests/qtest/migration-helpers.h   |  10 ++
>  tests/qtest/migration-test.c      | 185 ++++++++++++++++++++++++++----
>  tests/qtest/virtio-net-failover.c |  77 ++-----------
>  8 files changed, 308 insertions(+), 93 deletions(-)
>
Re: [PATCH v5 0/6] migration: Test the new "file:" migration
Posted by Juan Quintela 7 months ago
Fabiano Rosas <farosas@suse.de> wrote:
> Based-on:
> [PATCH V4 0/2] migration file URI
> https://lore.kernel.org/r/1688135108-316997-1-git-send-email-steven.sistare@oracle.com

Full series are integrated for next PULL request.

Thanks, Juan.

>
> Since v4:
>
> - Implemented a separate version of test_precopy_common to be used
>   with the file transport (patch 6).
>
> v4:
> https://lore.kernel.org/r/20230706201927.15442-1-farosas@suse.de
>
> v3:
> https://lore.kernel.org/r/20230630212902.19925-1-farosas@suse.de
>
> v2:
> https://lore.kernel.org/r/20230628165542.17214-1-farosas@suse.de
>
> v1:
> https://lore.kernel.org/r/20230626182210.8792-1-farosas@suse.de
>
> Fabiano Rosas (6):
>   tests/qtest: migration: Expose migrate_set_capability
>   tests/qtest: migration: Add migrate_incoming_qmp helper
>   tests/qtest: migration: Use migrate_incoming_qmp where appropriate
>   migration: Set migration status early in incoming side
>   tests/qtest: migration: Add support for negative testing of
>     qmp_migrate
>   tests/qtest: migration-test: Add tests for file-based migration
>
>  migration/migration.c             |   7 +-
>  tests/qtest/libqtest.c            |  33 ++++++
>  tests/qtest/libqtest.h            |  28 +++++
>  tests/qtest/meson.build           |   1 +
>  tests/qtest/migration-helpers.c   |  60 ++++++++++
>  tests/qtest/migration-helpers.h   |  10 ++
>  tests/qtest/migration-test.c      | 185 ++++++++++++++++++++++++++----
>  tests/qtest/virtio-net-failover.c |  77 ++-----------
>  8 files changed, 308 insertions(+), 93 deletions(-)