[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 7/9] tests/qtest/migration: Define a machine for all archi
From: |
Fabiano Rosas |
Subject: |
Re: [PATCH v2 7/9] tests/qtest/migration: Define a machine for all architectures |
Date: |
Tue, 17 Oct 2023 09:53:31 -0300 |
Juan Quintela <quintela@redhat.com> writes:
> Fabiano Rosas <farosas@suse.de> wrote:
>> Stop relying on defaults and select a machine explicitly for every
>> architecture.
>>
>> This is a prerequisite for being able to select machine types for
>> migration using different QEMU binaries for source and destination.
>>
>> Signed-off-by: Fabiano Rosas <farosas@suse.de>
>> ---
>> tests/qtest/migration-test.c | 11 ++++++++++-
>> 1 file changed, 10 insertions(+), 1 deletion(-)
>>
>> diff --git a/tests/qtest/migration-test.c b/tests/qtest/migration-test.c
>> index 46f1c275a2..7c10ac925b 100644
>> --- a/tests/qtest/migration-test.c
>> +++ b/tests/qtest/migration-test.c
>> @@ -746,6 +746,7 @@ static int test_migrate_start(QTestState **from,
>> QTestState **to,
>> const char *kvm_opts = NULL;
>> const char *arch = qtest_get_arch();
>> const char *memory_size;
>> + const char *machine;
>>
>> if (args->use_shmem) {
>> if (!g_file_test("/dev/shm", G_FILE_TEST_IS_DIR)) {
>> @@ -758,11 +759,13 @@ static int test_migrate_start(QTestState **from,
>> QTestState **to,
>> got_dst_resume = false;
>> if (strcmp(arch, "i386") == 0 || strcmp(arch, "x86_64") == 0) {
>> memory_size = "150M";
>> + machine = "pc";
>
> I would suggest:
>
> if (strcmp(arch, "i386")) {
> machine = "pc";
> } else {
> machine = "q35";
> }
Turns out we cannot run the tests with the q35 currently. It seems the
bootsector we use to print the A and Bs is not recognized by seabios on
that machine. I'm investigating.