All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Maydell <peter.maydell@linaro.org>
To: Juan Quintela <quintela@redhat.com>
Cc: "Laurent Vivier" <lvivier@redhat.com>,
	"Corey Minyard" <cminyard@mvista.com>,
	"Thomas Huth" <thuth@redhat.com>,
	"Daniel P. Berrangé" <berrange@redhat.com>,
	"Eduardo Habkost" <ehabkost@redhat.com>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	"Stefan Weil" <sw@weilnetz.de>,
	"Jason Wang" <jasowang@redhat.com>,
	"QEMU Developers" <qemu-devel@nongnu.org>,
	"Dr. David Alan Gilbert" <dgilbert@redhat.com>,
	qemu-arm <qemu-arm@nongnu.org>, qemu-ppc <qemu-ppc@nongnu.org>,
	"Marc-André Lureau" <marcandre.lureau@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Stefan Berger" <stefanb@linux.ibm.com>,
	"Richard Henderson" <rth@twiddle.net>,
	"David Gibson" <david@gibson.dropbear.id.au>
Subject: Re: [PULL 00/30] Migration pull patches (take 4)
Date: Fri, 17 Jan 2020 12:05:16 +0000	[thread overview]
Message-ID: <CAFEAcA_n21bu1gk0fqbzSr7iwurYCVR+wS7-RpDAJWnrYdVQ5g@mail.gmail.com> (raw)
In-Reply-To: <20200114125254.4515-1-quintela@redhat.com>

On Tue, 14 Jan 2020 at 12:53, Juan Quintela <quintela@redhat.com> wrote:
>
> The following changes since commit 3c8a6575985b1652b45bfa670b5e1907d642cfa0:
>
>   Merge remote-tracking branch 'remotes/kraxel/tags/usb-20200113-pull-request' into staging (2020-01-13 14:19:57 +0000)
>
> are available in the Git repository at:
>
>   https://github.com/juanquintela/qemu.git tags/migration-pull-pull-request
>
> for you to fetch changes up to b039b02c25d1725cf0296721d98e35e024468b20:
>
>   migration: Support QLIST migration (2020-01-14 13:45:29 +0100)
>
> ----------------------------------------------------------------
> Migration pull request (take 4)
>
> - both patches for x32 archs in
> - reorder the pages to make git bisect happy
>
> Thanks a lot to Daniel for helping with the bugs.  Twice.

tests/qtest/migration-test seems to hang in the aarch32 environment:

peter.m+  3380  0.1  0.0  13484 12848 ?        S    11:10   0:03
                   \_ make -C build/all-a32 check V=1 GCC_COLORS= -j9
peter.m+  5471  0.0  0.0   1460   388 ?        S    11:11   0:00
                       \_ /bin/sh -c
MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RAND
peter.m+  5472  0.3  0.0  12984  2228 ?        Sl   11:11   0:09
                       |   \_ tests/qtest/migration-test -m=quick -k
--tap
peter.m+ 17798 99.8  0.9 646064 150512 ?       Sl   11:13  43:04
                       |   |   \_ aarch64-softmmu/qemu-system-aarch64
-qtest unix:
peter.m+ 17814  0.0  0.0      0     0 ?        Z    11:13   0:00
                       |   |   \_ [qemu-system-aar] <defunct>
peter.m+  5473  0.0  0.0   8176  6652 ?        S    11:11   0:00
                       |   \_ perl ./scripts/tap-driver.pl
--test-name=migration-t
peter.m+ 17200  0.0  0.0   1460   404 ?        S    11:13   0:00
                       \_ /bin/sh -c
MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RAND
peter.m+ 17201  0.3  0.0  13080  2340 ?        Sl   11:13   0:08
                       |   \_ tests/qtest/migration-test -m=quick -k
--tap
peter.m+ 24275  100  0.8 549340 144160 ?       Sl   11:15  41:32
                       |   |   \_ i386-softmmu/qemu-system-i386 -qtest
unix:/tmp/q
peter.m+ 24308  0.0  0.0      0     0 ?        Z    11:15   0:00
                       |   |   \_ [qemu-system-i38] <defunct>
peter.m+ 17202  0.0  0.0   8172  6772 ?        S    11:13   0:00
                       |   \_ perl ./scripts/tap-driver.pl
--test-name=migration-t
peter.m+ 25399  0.0  0.0   1460   384 ?        S    11:15   0:00
                       \_ /bin/sh -c
MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RAND
peter.m+ 25400  0.3  0.0  13092  2280 ?        Sl   11:15   0:08
                           \_ tests/qtest/migration-test -m=quick -k
--tap
peter.m+  2755  100  0.8 557600 145696 ?       Sl   11:17  39:20
                           |   \_ x86_64-softmmu/qemu-system-x86_64
-qtest unix:/t
peter.m+  2763  0.0  0.0      0     0 ?        Z    11:17   0:00
                           |   \_ [qemu-system-x86] <defunct>
peter.m+ 25401  0.0  0.0   8172  6692 ?        S    11:15   0:00
                           \_ perl ./scripts/tap-driver.pl
--test-name=migration-t

(processes started 45 mins ago, unreaped processes in zombie state.)

It looks like the multifd/tcp test fails, but doesn't manage to
actually turn the failure into the test case exiting:

/i386/migration/multifd/tcp: qemu-system-i386: -accel kvm: invalid
accelerator kvm
qemu-system-i386: falling back to tcg
qemu-system-i386: -accel kvm: invalid accelerator kvm
qemu-system-i386: falling back to tcg
qemu-system-i386: multifd_send_sync_main: multifd_send_pages fail
qemu-system-i386: failed to save SaveStateEntry with id(name): 3(ram)
qemu-system-i386: Unable to write to socket: Broken pipe
qemu-system-i386: Unknown combination of migration flags: 0
qemu-system-i386: error while loading state section id 3(ram)
qemu-system-i386: load of migration failed: Invalid argument
[hangs here]

I think you need to find a system which has 32-bit ram_addr_t
and test this, because this is about the fourth time round
for this patchset failing on this configuration.

thanks
-- PMM


  parent reply	other threads:[~2020-01-17 12:06 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-01-14 12:52 [PULL 00/30] Migration pull patches (take 4) Juan Quintela
2020-01-14 12:52 ` [PULL 01/30] multifd: Initialize local variable Juan Quintela
2020-01-14 12:52 ` [PULL 02/30] multifd: Allocate uint64_t instead of ram_addr_t Juan Quintela
2020-01-14 12:52 ` [PULL 03/30] migration-test: Add migration multifd test Juan Quintela
2020-01-14 12:52 ` [PULL 04/30] migration: Make sure that we don't call write() in case of error Juan Quintela
2020-01-14 12:52 ` [PULL 05/30] migration-test: introduce functions to handle string parameters Juan Quintela
2020-01-14 12:52 ` [PULL 06/30] migration-test: ppc64: fix FORTH test program Juan Quintela
2020-01-14 12:52 ` [PULL 07/30] runstate: ignore finishmigrate -> prelaunch transition Juan Quintela
2020-01-14 12:52 ` [PULL 08/30] ram.c: remove unneeded labels Juan Quintela
2020-01-14 12:52 ` [PULL 09/30] migration: Rate limit inside host pages Juan Quintela
2020-01-14 12:52 ` [PULL 10/30] migration: Fix incorrect integer->float conversion caught by clang Juan Quintela
2020-01-14 12:52 ` [PULL 11/30] migration: Fix the re-run check of the migrate-incoming command Juan Quintela
2020-01-14 12:52 ` [PULL 12/30] misc: use QEMU_IS_ALIGNED Juan Quintela
2020-01-14 12:52 ` [PULL 13/30] migration: add savevm_state_handler_remove() Juan Quintela
2020-01-14 12:52 ` [PULL 14/30] migration: savevm_state_handler_insert: constant-time element insertion Juan Quintela
2020-01-14 12:52 ` [PULL 15/30] migration/ram: Yield periodically to the main loop Juan Quintela
2020-01-14 12:52 ` [PULL 16/30] migration/postcopy: reduce memset when it is zero page and matches_target_page_size Juan Quintela
2020-01-14 12:52 ` [PULL 17/30] migration/postcopy: wait for decompress thread in precopy Juan Quintela
2020-01-14 12:52 ` [PULL 18/30] migration/postcopy: count target page number to decide the place_needed Juan Quintela
2020-01-14 12:52 ` [PULL 19/30] migration/postcopy: set all_zero to true on the first target page Juan Quintela
2020-01-14 12:52 ` [PULL 20/30] migration/postcopy: enable random order target page arrival Juan Quintela
2020-01-14 12:52 ` [PULL 21/30] migration/postcopy: enable compress during postcopy Juan Quintela
2020-01-14 12:52 ` [PULL 22/30] migration/multifd: clean pages after filling packet Juan Quintela
2020-01-14 12:52 ` [PULL 23/30] migration/multifd: not use multifd during postcopy Juan Quintela
2020-01-14 12:52 ` [PULL 24/30] migration/multifd: fix nullptr access in terminating multifd threads Juan Quintela
2020-01-14 12:52 ` [PULL 25/30] migration/multifd: fix destroyed mutex " Juan Quintela
2020-01-14 12:52 ` [PULL 26/30] Bug #1829242 correction Juan Quintela
2020-01-14 12:52 ` [PULL 27/30] migration: Define VMSTATE_INSTANCE_ID_ANY Juan Quintela
2020-01-14 12:52 ` [PULL 28/30] migration: Change SaveStateEntry.instance_id into uint32_t Juan Quintela
2020-01-14 12:52 ` [PULL 29/30] apic: Use 32bit APIC ID for migration instance ID Juan Quintela
2020-01-14 12:52 ` [PULL 30/30] migration: Support QLIST migration Juan Quintela
2020-01-17 12:05 ` Peter Maydell [this message]
2020-01-17 12:22   ` [PULL 00/30] Migration pull patches (take 4) Juan Quintela
2020-01-17 12:41     ` Juan Quintela

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAFEAcA_n21bu1gk0fqbzSr7iwurYCVR+wS7-RpDAJWnrYdVQ5g@mail.gmail.com \
    --to=peter.maydell@linaro.org \
    --cc=berrange@redhat.com \
    --cc=cminyard@mvista.com \
    --cc=david@gibson.dropbear.id.au \
    --cc=dgilbert@redhat.com \
    --cc=ehabkost@redhat.com \
    --cc=jasowang@redhat.com \
    --cc=lvivier@redhat.com \
    --cc=marcandre.lureau@redhat.com \
    --cc=mst@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.org \
    --cc=quintela@redhat.com \
    --cc=rth@twiddle.net \
    --cc=stefanb@linux.ibm.com \
    --cc=sw@weilnetz.de \
    --cc=thuth@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.