From: Nicholas Piggin <npiggin@gmail.com>
To: Thomas Huth <thuth@redhat.com>
Cc: Nicholas Piggin <npiggin@gmail.com>,
kvm@vger.kernel.org, Laurent Vivier <lvivier@redhat.com>,
"Shaoqin Huang" <shahuang@redhat.com>,
Andrew Jones <andrew.jones@linux.dev>,
Nico Boehr <nrb@linux.ibm.com>,
Paolo Bonzini <pbonzini@redhat.com>,
Alexandru Elisei <alexandru.elisei@arm.com>,
Eric Auger <eric.auger@redhat.com>,
Janosch Frank <frankja@linux.ibm.com>,
Claudio Imbrenda <imbrenda@linux.ibm.com>,
David Hildenbrand <david@redhat.com>,
Marc Hartmayer <mhartmay@linux.ibm.com>,
linuxppc-dev@lists.ozlabs.org, linux-s390@vger.kernel.org,
kvmarm@lists.linux.dev, kvm-riscv@lists.infradead.org
Subject: [kvm-unit-tests PATCH v5 0/8] Multi-migration support
Date: Wed, 21 Feb 2024 13:27:49 +1000 [thread overview]
Message-ID: <20240221032757.454524-1-npiggin@gmail.com> (raw)
Now that strange arm64 hang is found to be QEMU bug, I'll repost.
Since arm64 requires Thomas's uart patch and it is worse affected
by the QEMU bug, I will just not build it on arm. The QEMU bug
still affects powerpc (and presumably s390x) but it's not causing
so much trouble for this test case.
I have another test case that can hit it reliably and doesn't
cause crashes but that takes some harness and common lib work so
I'll send that another time.
Since v4:
- Don't build selftest-migration on arm.
- Reduce selftest-migration iterations from 100 to 30 to make the
test run faster (it's ~0.5s per migration).
Since v3:
- Addressed Thomas's review comments:
- Patch 2 initrd cleanup unset the old variable in the correct place.
- Patch 4 multi migration removed the extra wait for "Now migrate the
VM" message, and updated comments around it.
- Patch 6 fix typo and whitespace in quiet migration support.
- Patch 8 fix typo and whitespace in migration selftest.
Since v2:
- Rebase on riscv port and auxvinfo fix was merged.
- Clean up initrd cleanup moves more commands into the new cleanup
function from the trap handler comands (suggested by Thomas).
- "arch-run: Clean up temporary files properly" patch is now renamed
to "arch-run: Fix TRAP handler..."
- Fix TRAP handler patch has redone changelog to be more precise about
the problem and including recipe to recreate it.
- Fix TRAP handler patch reworked slightly to remove the theoretical
race rather than just adding a comment about it.
- Patch 3 was missing a couple of fixes that leaked into patch 4,
those are moved into patch 3.
Thanks,
Nick
Nicholas Piggin (8):
arch-run: Fix TRAP handler recursion to remove temporary files
properly
arch-run: Clean up initrd cleanup
migration: use a more robust way to wait for background job
migration: Support multiple migrations
arch-run: rename migration variables
migration: Add quiet migration support
Add common/ directory for architecture-independent tests
migration: add a migration selftest
arm/sieve.c | 2 +-
common/selftest-migration.c | 29 ++++++
common/sieve.c | 51 ++++++++++
lib/migrate.c | 19 +++-
lib/migrate.h | 2 +
powerpc/Makefile.common | 1 +
powerpc/selftest-migration.c | 1 +
powerpc/unittests.cfg | 4 +
riscv/sieve.c | 2 +-
s390x/Makefile | 1 +
s390x/selftest-migration.c | 1 +
s390x/sieve.c | 2 +-
s390x/unittests.cfg | 4 +
scripts/arch-run.bash | 177 +++++++++++++++++++++++++----------
x86/sieve.c | 52 +---------
15 files changed, 240 insertions(+), 108 deletions(-)
create mode 100644 common/selftest-migration.c
create mode 100644 common/sieve.c
create mode 120000 powerpc/selftest-migration.c
create mode 120000 s390x/selftest-migration.c
mode change 100644 => 120000 x86/sieve.c
--
2.42.0
next reply other threads:[~2024-02-21 3:28 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-21 3:27 Nicholas Piggin [this message]
2024-02-21 3:27 ` [kvm-unit-tests PATCH v5 1/8] arch-run: Fix TRAP handler recursion to remove temporary files properly Nicholas Piggin
2024-02-21 3:27 ` [kvm-unit-tests PATCH v5 2/8] arch-run: Clean up initrd cleanup Nicholas Piggin
2024-02-21 3:27 ` [kvm-unit-tests PATCH v5 3/8] migration: use a more robust way to wait for background job Nicholas Piggin
2024-02-21 3:27 ` [kvm-unit-tests PATCH v5 4/8] migration: Support multiple migrations Nicholas Piggin
2024-02-21 3:27 ` [kvm-unit-tests PATCH v5 5/8] arch-run: rename migration variables Nicholas Piggin
2024-02-21 3:27 ` [kvm-unit-tests PATCH v5 6/8] migration: Add quiet migration support Nicholas Piggin
2024-02-21 3:27 ` [kvm-unit-tests PATCH v5 7/8] Add common/ directory for architecture-independent tests Nicholas Piggin
2024-02-21 7:43 ` Andrew Jones
2024-02-21 3:27 ` [kvm-unit-tests PATCH v5 8/8] migration: add a migration selftest Nicholas Piggin
2024-02-23 7:06 ` [kvm-unit-tests PATCH v5 0/8] Multi-migration support Thomas Huth
2024-02-26 8:10 ` Nicholas Piggin
2024-02-26 9:08 ` Thomas Huth
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=20240221032757.454524-1-npiggin@gmail.com \
--to=npiggin@gmail.com \
--cc=alexandru.elisei@arm.com \
--cc=andrew.jones@linux.dev \
--cc=david@redhat.com \
--cc=eric.auger@redhat.com \
--cc=frankja@linux.ibm.com \
--cc=imbrenda@linux.ibm.com \
--cc=kvm-riscv@lists.infradead.org \
--cc=kvm@vger.kernel.org \
--cc=kvmarm@lists.linux.dev \
--cc=linux-s390@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=lvivier@redhat.com \
--cc=mhartmay@linux.ibm.com \
--cc=nrb@linux.ibm.com \
--cc=pbonzini@redhat.com \
--cc=shahuang@redhat.com \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).