[Qemu-devel] [PULL 0/8] Migration patches

Juan Quintela posted 8 patches 4 years, 10 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190612104808.6398-1-quintela@redhat.com
Test s390x passed
Test checkpatch passed
Test asan passed
Test docker-mingw@fedora passed
Test docker-clang@ubuntu passed
Test FreeBSD passed
Maintainers: Thomas Huth <thuth@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Laurent Vivier <lvivier@redhat.com>, Juan Quintela <quintela@redhat.com>, "Dr. David Alan Gilbert" <dgilbert@redhat.com>
There is a newer version of this series
migration/ram.c        |  41 ++++++++++------
migration/trace-events |   2 +-
tests/migration-test.c | 103 ++++++++++++++++++++++++++++++-----------
util/cutils.c          |   8 ++--
4 files changed, 107 insertions(+), 47 deletions(-)
[Qemu-devel] [PULL 0/8] Migration patches
Posted by Juan Quintela 4 years, 10 months ago
The following changes since commit 219dca61ebf41625831d4f96a720852baf44b762:

  Merge remote-tracking branch 'remotes/ehabkost/tags/x86-next-pull-request' into staging (2019-06-11 16:02:07 +0100)

are available in the Git repository at:

  https://github.com/juanquintela/qemu.git tags/migration-pull-request

for you to fetch changes up to 87fa344f0684d7c62b0859632de4b6c28e32f2c7:

  migratioin/ram.c: reset complete_round when we gets a queued page (2019-06-12 12:38:12 +0200)

----------------------------------------------------------------
Migration pull request

* misc fixes from wei
* Fixes for the multifd compression tree (juan)

Please apply

----------------------------------------------------------------

Juan Quintela (3):
  migration: fix multifd_recv event typo
  migration-test: rename parameter to parameter_int
  migration-test: Add migration multifd test

Wei Yang (5):
  migration/multifd: call multifd_send_sync_main when sending
    RAM_SAVE_FLAG_EOS
  migration/xbzrle: update cache and current_data in one place
  cutils: remove one unnecessary pointer operation
  migration/multifd: sync packet_num after all thread are done
  migratioin/ram.c: reset complete_round when we gets a queued page

 migration/ram.c        |  41 ++++++++++------
 migration/trace-events |   2 +-
 tests/migration-test.c | 103 ++++++++++++++++++++++++++++++-----------
 util/cutils.c          |   8 ++--
 4 files changed, 107 insertions(+), 47 deletions(-)

-- 
2.21.0


Re: [Qemu-devel] [PULL 0/8] Migration patches
Posted by Peter Maydell 4 years, 10 months ago
On Wed, 12 Jun 2019 at 11:54, Juan Quintela <quintela@redhat.com> wrote:
>
> The following changes since commit 219dca61ebf41625831d4f96a720852baf44b762:
>
>   Merge remote-tracking branch 'remotes/ehabkost/tags/x86-next-pull-request' into staging (2019-06-11 16:02:07 +0100)
>
> are available in the Git repository at:
>
>   https://github.com/juanquintela/qemu.git tags/migration-pull-request
>
> for you to fetch changes up to 87fa344f0684d7c62b0859632de4b6c28e32f2c7:
>
>   migratioin/ram.c: reset complete_round when we gets a queued page (2019-06-12 12:38:12 +0200)
>
> ----------------------------------------------------------------
> Migration pull request
>
> * misc fixes from wei
> * Fixes for the multifd compression tree (juan)
>
> Please apply

Hi. This fails 'make check' on aarch32 (possibly on all
32-bit guests, aarch32 is my only 32-bit host):

MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
QTEST_QEMU_BINARY=aarch64-softmmu/qemu-system-aarch64
QTEST_QEMU_IMG=qemu-img tests/migration-test -m=quick -k --tap <
/dev/null | ./scripts/tap-driver.pl --test-name="migration-test"
PASS 1 migration-test /aarch64/migration/deprecated
PASS 2 migration-test /aarch64/migration/bad_dest
PASS 3 migration-test /aarch64/migration/fd_proto
PASS 4 migration-test /aarch64/migration/postcopy/unix
PASS 5 migration-test /aarch64/migration/postcopy/recovery
PASS 6 migration-test /aarch64/migration/precopy/unix
PASS 7 migration-test /aarch64/migration/precopy/tcp
PASS 8 migration-test /aarch64/migration/xbzrle/unix
malloc_consolidate(): invalid chunk size
Broken pipe
qemu-system-aarch64: check_section_footer: Read section footer failed: -5
qemu-system-aarch64: load of migration failed: Invalid argument
/home/peter.maydell/qemu/tests/libqtest.c:137: kill_qemu() tried to
terminate QEMU process but encountered exit status 1
Aborted
ERROR - too few tests run (expected 9, got 8)
/home/peter.maydell/qemu/tests/Makefile.include:888: recipe for target
'check-qtest-aarch64' failed

thanks
-- PMM

Re: [Qemu-devel] [PULL 0/8] Migration patches
Posted by Juan Quintela 4 years, 10 months ago
Peter Maydell <peter.maydell@linaro.org> wrote:
> On Wed, 12 Jun 2019 at 11:54, Juan Quintela <quintela@redhat.com> wrote:
>>
>> The following changes since commit 219dca61ebf41625831d4f96a720852baf44b762:
>>
>>   Merge remote-tracking branch
>> 'remotes/ehabkost/tags/x86-next-pull-request' into staging
>> (2019-06-11 16:02:07 +0100)
>>
>> are available in the Git repository at:
>>
>>   https://github.com/juanquintela/qemu.git tags/migration-pull-request
>>
>> for you to fetch changes up to 87fa344f0684d7c62b0859632de4b6c28e32f2c7:
>>
>>   migratioin/ram.c: reset complete_round when we gets a queued page
>> (2019-06-12 12:38:12 +0200)
>>
>> ----------------------------------------------------------------
>> Migration pull request
>>
>> * misc fixes from wei
>> * Fixes for the multifd compression tree (juan)
>>
>> Please apply
>
> Hi. This fails 'make check' on aarch32 (possibly on all
> 32-bit guests, aarch32 is my only 32-bit host):
>
> MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
> QTEST_QEMU_BINARY=aarch64-softmmu/qemu-system-aarch64
> QTEST_QEMU_IMG=qemu-img tests/migration-test -m=quick -k --tap <
> /dev/null | ./scripts/tap-driver.pl --test-name="migration-test"
> PASS 1 migration-test /aarch64/migration/deprecated
> PASS 2 migration-test /aarch64/migration/bad_dest
> PASS 3 migration-test /aarch64/migration/fd_proto
> PASS 4 migration-test /aarch64/migration/postcopy/unix
> PASS 5 migration-test /aarch64/migration/postcopy/recovery
> PASS 6 migration-test /aarch64/migration/precopy/unix
> PASS 7 migration-test /aarch64/migration/precopy/tcp
> PASS 8 migration-test /aarch64/migration/xbzrle/unix
> malloc_consolidate(): invalid chunk size
> Broken pipe
> qemu-system-aarch64: check_section_footer: Read section footer failed: -5
> qemu-system-aarch64: load of migration failed: Invalid argument
> /home/peter.maydell/qemu/tests/libqtest.c:137: kill_qemu() tried to
> terminate QEMU process but encountered exit status 1
> Aborted
> ERROR - too few tests run (expected 9, got 8)
> /home/peter.maydell/qemu/tests/Makefile.include:888: recipe for target
> 'check-qtest-aarch64' failed

Thanks, looking into it.

Later, Juan.