[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v9 12/12] migration: Add multifd test
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH v9 12/12] migration: Add multifd test |
Date: |
Tue, 17 Oct 2017 16:27:47 +0100 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
* Juan Quintela (address@hidden) wrote:
> We set the x-multifd-page-count and x-multifd-channels.
>
> Signed-off-by: Juan Quintela <address@hidden>
<snip>
We *must* find a way to share all the boiler plate I've snipped
out; even this test_migrate function is almost identical to
your other test series with just those few extra parameter sets.
> +static void test_migrate(void)
> +{
> + char *uri = g_strdup_printf("unix:%s/migsocket", tmpfs);
> + QTestState *global = global_qtest, *from, *to;
> + unsigned char dest_byte_a, dest_byte_b, dest_byte_c, dest_byte_d;
> + gchar *cmd, *cmd_src, *cmd_dst;
> + QDict *rsp;
> +
> + char *bootpath = g_strdup_printf("%s/bootsect", tmpfs);
> + const char *arch = qtest_get_arch();
> +
> + got_stop = false;
> +
> + if (strcmp(arch, "i386") == 0 || strcmp(arch, "x86_64") == 0) {
> + init_bootfile_x86(bootpath);
> + cmd_src = g_strdup_printf("-machine accel=kvm:tcg -m 150M"
> + " -name pcsource,debug-threads=on"
> + " -serial file:%s/src_serial"
> + " -drive file=%s,format=raw",
> + tmpfs, bootpath);
> + cmd_dst = g_strdup_printf("-machine accel=kvm:tcg -m 150M"
> + " -name pcdest,debug-threads=on"
> + " -serial file:%s/dest_serial"
> + " -drive file=%s,format=raw"
> + " -incoming %s",
> + tmpfs, bootpath, uri);
> + } else if (strcmp(arch, "ppc64") == 0) {
> + const char *accel;
> +
> + /* On ppc64, the test only works with kvm-hv, but not with kvm-pr */
> + accel = access("/sys/module/kvm_hv", F_OK) ? "tcg" : "kvm:tcg";
> + init_bootfile_ppc(bootpath);
> + cmd_src = g_strdup_printf("-machine accel=%s -m 256M"
> + " -name pcsource,debug-threads=on"
> + " -serial file:%s/src_serial"
> + " -drive file=%s,if=pflash,format=raw",
> + accel, tmpfs, bootpath);
> + cmd_dst = g_strdup_printf("-machine accel=%s -m 256M"
> + " -name pcdest,debug-threads=on"
> + " -serial file:%s/dest_serial"
> + " -incoming %s",
> + accel, tmpfs, uri);
> + } else {
> + g_assert_not_reached();
> + }
> +
> + g_free(bootpath);
> +
> + from = qtest_start(cmd_src);
> + g_free(cmd_src);
> +
> + to = qtest_init(cmd_dst);
> + g_free(cmd_dst);
> +
> + global_qtest = from;
> + rsp = qmp("{ 'execute': 'migrate-set-capabilities',"
> + "'arguments': { "
> + "'capabilities': [ {"
> + "'capability': 'x-multifd',"
> + "'state': true } ] } }");
> + g_assert(qdict_haskey(rsp, "return"));
> + QDECREF(rsp);
> +
> + global_qtest = to;
> + rsp = qmp("{ 'execute': 'migrate-set-capabilities',"
> + "'arguments': { "
> + "'capabilities': [ {"
> + "'capability': 'x-multifd',"
> + "'state': true } ] } }");
> + g_assert(qdict_haskey(rsp, "return"));
> + QDECREF(rsp);
> +
> + /* We want to pick a speed slow enough that the test completes
> + * quickly, but that it doesn't complete precopy even on a slow
> + * machine, so also set the downtime.
> + */
> + global_qtest = from;
> + rsp = qmp("{ 'execute': 'migrate-set-parameters',"
> + "'arguments': { 'max-bandwidth': 100000000 } }");
> + g_assert(qdict_haskey(rsp, "return"));
> + QDECREF(rsp);
> +
> + /* 200ms downtime */
> + rsp = qmp("{ 'execute': 'migrate-set-parameters',"
> + "'arguments': { 'downtime-limit': 300 } }");
Note 200 vs 300 !
> + g_assert(qdict_haskey(rsp, "return"));
> + QDECREF(rsp);
> +
> + /* set 4 channels */
> + global_qtest = to;
> + rsp = qmp("{ 'execute': 'migrate-set-parameters',"
> + "'arguments': { 'x-multifd-channels': 4 } }");
> + g_assert(qdict_haskey(rsp, "return"));
> + QDECREF(rsp);
> +
> + rsp = qmp("{ 'execute': 'migrate-set-parameters',"
> + "'arguments': { 'x-multifd-page-count': 64 } }");
> + g_assert(qdict_haskey(rsp, "return"));
> + QDECREF(rsp);
> +
> + /* set 4 channels */
> + global_qtest = from;
> + rsp = qmp("{ 'execute': 'migrate-set-parameters',"
> + "'arguments': { 'x-multifd-channels': 4 } }");
> + g_assert(qdict_haskey(rsp, "return"));
> + QDECREF(rsp);
> +
> + rsp = qmp("{ 'execute': 'migrate-set-parameters',"
> + "'arguments': { 'x-multifd-page-count': 64 } }");
> + g_assert(qdict_haskey(rsp, "return"));
> + QDECREF(rsp);
> +
> +
> + /* Wait for the first serial output from the source */
> + wait_for_serial("src_serial");
> +
> + cmd = g_strdup_printf("{ 'execute': 'migrate',"
> + "'arguments': { 'uri': '%s' } }",
> + uri);
> + rsp = qmp(cmd);
> + g_free(cmd);
> + g_assert(qdict_haskey(rsp, "return"));
> + QDECREF(rsp);
> +
> + wait_for_migration_pass();
> +
> + if (!got_stop) {
> + qmp_eventwait("STOP");
> + }
> +
> + global_qtest = to;
> + qmp_eventwait("RESUME");
> +
> + wait_for_serial("dest_serial");
> + global_qtest = from;
> + wait_for_migration_complete();
> +
> + qtest_quit(from);
> +
> + global_qtest = to;
> +
> + qtest_memread(to, start_address, &dest_byte_a, 1);
> +
> + /* Destination still running, wait for a byte to change */
> + do {
> + qtest_memread(to, start_address, &dest_byte_b, 1);
> + usleep(10 * 1000);
> + } while (dest_byte_a == dest_byte_b);
Are there any multifd stats we can check? I guess the migration
wont start until all channels are connected, so we know they're all
in use in theory, so there's nothing specific.
Dave
> + qmp_discard_response("{ 'execute' : 'stop'}");
> + /* With it stopped, check nothing changes */
> + qtest_memread(to, start_address, &dest_byte_c, 1);
> + sleep(1);
> + qtest_memread(to, start_address, &dest_byte_d, 1);
> + g_assert_cmpint(dest_byte_c, ==, dest_byte_d);
> +
> + check_guests_ram();
> +
> + qtest_quit(to);
> + g_free(uri);
> +
> + global_qtest = global;
> +
> + cleanup("bootsect");
> + cleanup("migsocket");
> + cleanup("src_serial");
> + cleanup("dest_serial");
> +}
> +
> +int main(int argc, char **argv)
> +{
> + char template[] = "/tmp/multifd-test-XXXXXX";
> + int ret;
> +
> + g_test_init(&argc, &argv, NULL);
> +
> + tmpfs = mkdtemp(template);
> + if (!tmpfs) {
> + g_test_message("mkdtemp on path (%s): %s\n", template,
> strerror(errno));
> + }
> + g_assert(tmpfs);
> +
> + module_call_init(MODULE_INIT_QOM);
> +
> + qtest_add_func("/multifd", test_migrate);
> +
> + ret = g_test_run();
> +
> + g_assert_cmpint(ret, ==, 0);
> +
> + ret = rmdir(tmpfs);
> + if (ret != 0) {
> + g_test_message("unable to rmdir: path (%s): %s\n",
> + tmpfs, strerror(errno));
> + }
> +
> + return ret;
> +}
> --
> 2.13.5
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
- Re: [Qemu-devel] [PATCH v9 08/12] migration: Test new fd infrastructure, (continued)
- [Qemu-devel] [PATCH v9 05/12] migration: Create ram_multifd_page, Juan Quintela, 2017/10/04
- [Qemu-devel] [PATCH v9 09/12] migration: Rename initial_bytes, Juan Quintela, 2017/10/04
- [Qemu-devel] [PATCH v9 11/12] migration: Flush receive queue, Juan Quintela, 2017/10/04
- [Qemu-devel] [PATCH v9 10/12] migration: Transfer pages over new channels, Juan Quintela, 2017/10/04
- [Qemu-devel] [PATCH v9 12/12] migration: Add multifd test, Juan Quintela, 2017/10/04
- Re: [Qemu-devel] [PATCH v9 12/12] migration: Add multifd test,
Dr. David Alan Gilbert <=