migration-test: machine_opts is really arch specific
And it needs to be in both source and target, so put it on arch_opts. Reviewed-by: Peter Xu <peterx@redhat.com> Message-ID: <20230608224943.3877-7-quintela@redhat.com> Signed-off-by: Juan Quintela <quintela@redhat.com>
This commit is contained in:
parent
832c732c5d
commit
bc28a6111d
@ -711,7 +711,6 @@ static int test_migrate_start(QTestState **from, QTestState **to,
|
||||
g_autofree char *shmem_opts = NULL;
|
||||
g_autofree char *shmem_path = NULL;
|
||||
const char *arch = qtest_get_arch();
|
||||
const char *machine_opts = NULL;
|
||||
const char *memory_size;
|
||||
|
||||
if (args->use_shmem) {
|
||||
@ -739,7 +738,6 @@ static int test_migrate_start(QTestState **from, QTestState **to,
|
||||
start_address = S390_TEST_MEM_START;
|
||||
end_address = S390_TEST_MEM_END;
|
||||
} else if (strcmp(arch, "ppc64") == 0) {
|
||||
machine_opts = "-machine vsmt=8";
|
||||
memory_size = "256M";
|
||||
start_address = PPC_TEST_MEM_START;
|
||||
end_address = PPC_TEST_MEM_END;
|
||||
@ -747,12 +745,12 @@ static int test_migrate_start(QTestState **from, QTestState **to,
|
||||
"'nvramrc=hex .\" _\" begin %x %x "
|
||||
"do i c@ 1 + i c! 1000 +loop .\" B\" 0 "
|
||||
"until'", end_address, start_address);
|
||||
arch_opts = g_strdup("-nodefaults");
|
||||
arch_opts = g_strdup("-nodefaults -machine vsmt=8");
|
||||
} else if (strcmp(arch, "aarch64") == 0) {
|
||||
init_bootfile(bootpath, aarch64_kernel, sizeof(aarch64_kernel));
|
||||
machine_opts = "-machine virt,gic-version=max";
|
||||
memory_size = "150M";
|
||||
arch_opts = g_strdup_printf("-cpu max -kernel %s", bootpath);
|
||||
arch_opts = g_strdup_printf("-machine virt,gic-version=max -cpu max "
|
||||
"-kernel %s", bootpath);
|
||||
start_address = ARM_TEST_MEM_START;
|
||||
end_address = ARM_TEST_MEM_END;
|
||||
|
||||
@ -787,14 +785,13 @@ static int test_migrate_start(QTestState **from, QTestState **to,
|
||||
shmem_opts = g_strdup("");
|
||||
}
|
||||
|
||||
cmd_source = g_strdup_printf("-accel kvm%s -accel tcg %s "
|
||||
cmd_source = g_strdup_printf("-accel kvm%s -accel tcg "
|
||||
"-name source,debug-threads=on "
|
||||
"-m %s "
|
||||
"-serial file:%s/src_serial "
|
||||
"%s %s %s %s %s",
|
||||
args->use_dirty_ring ?
|
||||
",dirty-ring-size=4096" : "",
|
||||
machine_opts ? machine_opts : "",
|
||||
memory_size, tmpfs,
|
||||
arch_opts ? arch_opts : "",
|
||||
arch_source ? arch_source : "",
|
||||
@ -808,7 +805,7 @@ static int test_migrate_start(QTestState **from, QTestState **to,
|
||||
&got_src_stop);
|
||||
}
|
||||
|
||||
cmd_target = g_strdup_printf("-accel kvm%s -accel tcg %s "
|
||||
cmd_target = g_strdup_printf("-accel kvm%s -accel tcg "
|
||||
"-name target,debug-threads=on "
|
||||
"-m %s "
|
||||
"-serial file:%s/dest_serial "
|
||||
@ -816,7 +813,6 @@ static int test_migrate_start(QTestState **from, QTestState **to,
|
||||
"%s %s %s %s %s",
|
||||
args->use_dirty_ring ?
|
||||
",dirty-ring-size=4096" : "",
|
||||
machine_opts ? machine_opts : "",
|
||||
memory_size, tmpfs, uri,
|
||||
arch_opts ? arch_opts : "",
|
||||
arch_target ? arch_target : "",
|
||||
|
Loading…
x
Reference in New Issue
Block a user