From: Gavin Shan <gshan@redhat.com>
To: Andrew Jones <drjones@redhat.com>,
qemu-devel@nongnu.org, qemu-arm@nongnu.org
Cc: f4bug@amsat.org, alex.bennee@linaro.org, cohuck@redhat.com
Subject: Re: [PATCH v2] tests: Prefer max cpu type when using AArch64 virt machine
Date: Fri, 3 Jun 2022 21:51:48 +0800 [thread overview]
Message-ID: <370c0e42-4daa-3370-ad76-a5ec1d10cb0d@redhat.com> (raw)
In-Reply-To: <20220603111849.1481100-1-drjones@redhat.com>
On 6/3/22 7:18 PM, Andrew Jones wrote:
> The max cpu type is the best default cpu type for tests to use
> when specifying the cpu type for AArch64 mach-virt. Switch all
> tests to it.
>
> Cc: Alex Bennée <alex.bennee@linaro.org>
> Signed-off-by: Andrew Jones <drjones@redhat.com>
> ---
> tests/avocado/boot_xen.py | 6 +++---
> tests/avocado/replay_kernel.py | 2 +-
> tests/avocado/reverse_debugging.py | 2 +-
> tests/avocado/tcg_plugins.py | 6 +++---
> tests/qtest/bios-tables-test.c | 12 ++++++------
> tests/qtest/machine-none-test.c | 4 ++--
> tests/vm/aarch64vm.py | 2 +-
> tests/vm/ubuntu.aarch64 | 2 +-
> 8 files changed, 18 insertions(+), 18 deletions(-)
>
Reviewed-by: Gavin Shan <gshan@redhat.com>
> diff --git a/tests/avocado/boot_xen.py b/tests/avocado/boot_xen.py
> index fc2faeedb559..899c396bd55c 100644
> --- a/tests/avocado/boot_xen.py
> +++ b/tests/avocado/boot_xen.py
> @@ -66,7 +66,7 @@ def test_arm64_xen_411_and_dom0(self):
> """
> :avocado: tags=arch:aarch64
> :avocado: tags=accel:tcg
> - :avocado: tags=cpu:cortex-a57
> + :avocado: tags=cpu:max
> :avocado: tags=machine:virt
> """
>
> @@ -84,7 +84,7 @@ def test_arm64_xen_414_and_dom0(self):
> """
> :avocado: tags=arch:aarch64
> :avocado: tags=accel:tcg
> - :avocado: tags=cpu:cortex-a57
> + :avocado: tags=cpu:max
> :avocado: tags=machine:virt
> """
>
> @@ -102,7 +102,7 @@ def test_arm64_xen_415_and_dom0(self):
> """
> :avocado: tags=arch:aarch64
> :avocado: tags=accel:tcg
> - :avocado: tags=cpu:cortex-a57
> + :avocado: tags=cpu:max
> :avocado: tags=machine:virt
> """
>
> diff --git a/tests/avocado/replay_kernel.py b/tests/avocado/replay_kernel.py
> index 0b2b0dc692b1..c19022ea977d 100644
> --- a/tests/avocado/replay_kernel.py
> +++ b/tests/avocado/replay_kernel.py
> @@ -147,7 +147,7 @@ def test_aarch64_virt(self):
> """
> :avocado: tags=arch:aarch64
> :avocado: tags=machine:virt
> - :avocado: tags=cpu:cortex-a53
> + :avocado: tags=cpu:max
> """
> kernel_url = ('https://archives.fedoraproject.org/pub/archive/fedora'
> '/linux/releases/29/Everything/aarch64/os/images/pxeboot'
> diff --git a/tests/avocado/reverse_debugging.py b/tests/avocado/reverse_debugging.py
> index d2921e70c3b4..d6a6d7277235 100644
> --- a/tests/avocado/reverse_debugging.py
> +++ b/tests/avocado/reverse_debugging.py
> @@ -198,7 +198,7 @@ def test_aarch64_virt(self):
> """
> :avocado: tags=arch:aarch64
> :avocado: tags=machine:virt
> - :avocado: tags=cpu:cortex-a53
> + :avocado: tags=cpu:max
> """
> kernel_url = ('https://archives.fedoraproject.org/pub/archive/fedora'
> '/linux/releases/29/Everything/aarch64/os/images/pxeboot'
> diff --git a/tests/avocado/tcg_plugins.py b/tests/avocado/tcg_plugins.py
> index 642d2e49e305..2bbf62f5036e 100644
> --- a/tests/avocado/tcg_plugins.py
> +++ b/tests/avocado/tcg_plugins.py
> @@ -68,7 +68,7 @@ def test_aarch64_virt_insn(self):
> :avocado: tags=accel:tcg
> :avocado: tags=arch:aarch64
> :avocado: tags=machine:virt
> - :avocado: tags=cpu:cortex-a53
> + :avocado: tags=cpu:max
> """
> kernel_path = self._grab_aarch64_kernel()
> kernel_command_line = (self.KERNEL_COMMON_COMMAND_LINE +
> @@ -94,7 +94,7 @@ def test_aarch64_virt_insn_icount(self):
> :avocado: tags=accel:tcg
> :avocado: tags=arch:aarch64
> :avocado: tags=machine:virt
> - :avocado: tags=cpu:cortex-a53
> + :avocado: tags=cpu:max
> """
> kernel_path = self._grab_aarch64_kernel()
> kernel_command_line = (self.KERNEL_COMMON_COMMAND_LINE +
> @@ -120,7 +120,7 @@ def test_aarch64_virt_mem_icount(self):
> :avocado: tags=accel:tcg
> :avocado: tags=arch:aarch64
> :avocado: tags=machine:virt
> - :avocado: tags=cpu:cortex-a53
> + :avocado: tags=cpu:max
> """
> kernel_path = self._grab_aarch64_kernel()
> kernel_command_line = (self.KERNEL_COMMON_COMMAND_LINE +
> diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
> index a4a46e97f0b8..7c3a58a97460 100644
> --- a/tests/qtest/bios-tables-test.c
> +++ b/tests/qtest/bios-tables-test.c
> @@ -1270,7 +1270,7 @@ static void test_acpi_virt_tcg_memhp(void)
>
> data.variant = ".memhp";
> test_acpi_one(" -machine nvdimm=on"
> - " -cpu cortex-a57"
> + " -cpu max"
> " -m 256M,slots=3,maxmem=1G"
> " -object memory-backend-ram,id=ram0,size=128M"
> " -object memory-backend-ram,id=ram1,size=128M"
> @@ -1363,7 +1363,7 @@ static void test_acpi_virt_tcg_numamem(void)
> };
>
> data.variant = ".numamem";
> - test_acpi_one(" -cpu cortex-a57"
> + test_acpi_one(" -cpu max"
> " -object memory-backend-ram,id=ram0,size=128M"
> " -numa node,memdev=ram0",
> &data);
> @@ -1397,7 +1397,7 @@ static void test_acpi_virt_tcg_pxb(void)
> "if=none,media=cdrom,id=drive-scsi0-0-0-1,readonly=on"
> " -device scsi-cd,bus=scsi0.0,scsi-id=0,"
> "drive=drive-scsi0-0-0-1,id=scsi0-0-0-1,bootindex=1"
> - " -cpu cortex-a57"
> + " -cpu max"
> " -device pxb-pcie,bus_nr=128",
> &data);
>
> @@ -1511,7 +1511,7 @@ static void test_acpi_virt_tcg(void)
>
> data.smbios_cpu_max_speed = 2900;
> data.smbios_cpu_curr_speed = 2700;
> - test_acpi_one("-cpu cortex-a57 "
> + test_acpi_one("-cpu max "
> "-smbios type=4,max-speed=2900,current-speed=2700", &data);
> free_test_data(&data);
> }
> @@ -1591,7 +1591,7 @@ static void test_acpi_virt_viot(void)
> .scan_len = 128ULL * 1024 * 1024,
> };
>
> - test_acpi_one("-cpu cortex-a57 "
> + test_acpi_one("-cpu max "
> "-device virtio-iommu-pci", &data);
> free_test_data(&data);
> }
> @@ -1699,7 +1699,7 @@ static void test_acpi_oem_fields_virt(void)
> char *args;
>
> args = test_acpi_create_args(&data,
> - "-cpu cortex-a57 "OEM_TEST_ARGS, true);
> + "-cpu max "OEM_TEST_ARGS, true);
> data.qts = qtest_init(args);
> test_acpi_load_tables(&data, true);
> test_oem_fields(&data);
> diff --git a/tests/qtest/machine-none-test.c b/tests/qtest/machine-none-test.c
> index d0f8cd990245..d47533a835b7 100644
> --- a/tests/qtest/machine-none-test.c
> +++ b/tests/qtest/machine-none-test.c
> @@ -24,8 +24,8 @@ struct arch2cpu {
>
> static struct arch2cpu cpus_map[] = {
> /* tested targets list */
> - { "arm", "cortex-a15" },
> - { "aarch64", "cortex-a57" },
> + { "arm", "max" },
> + { "aarch64", "max" },
> { "avr", "avr6-avr-cpu" },
> { "x86_64", "qemu64,apic-id=0" },
> { "i386", "qemu32,apic-id=0" },
> diff --git a/tests/vm/aarch64vm.py b/tests/vm/aarch64vm.py
> index b00cce07eb81..a82b4dd7a91a 100644
> --- a/tests/vm/aarch64vm.py
> +++ b/tests/vm/aarch64vm.py
> @@ -31,7 +31,7 @@
> # parameters that works for backwards compatibility.
> DEFAULT_CONFIG = {'kvm' : {'cpu' : "host",
> 'machine' : "virt,gic-version=host"},
> - 'tcg' : {'cpu' : "cortex-a57",
> + 'tcg' : {'cpu' : "max",
> 'machine' : "virt"},
> }
>
> diff --git a/tests/vm/ubuntu.aarch64 b/tests/vm/ubuntu.aarch64
> index b291945a7e9d..e2958ccd2a44 100755
> --- a/tests/vm/ubuntu.aarch64
> +++ b/tests/vm/ubuntu.aarch64
> @@ -18,7 +18,7 @@ import aarch64vm
> import ubuntuvm
>
> DEFAULT_CONFIG = {
> - 'cpu' : "cortex-a57",
> + 'cpu' : "max",
> 'machine' : "virt,gic-version=3",
> 'install_cmds' : "apt-get update,"\
> "apt-get build-dep -y --arch-only qemu,"\
>
next prev parent reply other threads:[~2022-06-03 13:53 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-06-03 11:18 [PATCH v2] tests: Prefer max cpu type when using AArch64 virt machine Andrew Jones
2022-06-03 11:55 ` Cornelia Huck
2022-06-03 13:51 ` Gavin Shan [this message]
2022-06-03 14:04 ` Richard Henderson
2022-06-03 15:15 ` Andrew Jones
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=370c0e42-4daa-3370-ad76-a5ec1d10cb0d@redhat.com \
--to=gshan@redhat.com \
--cc=alex.bennee@linaro.org \
--cc=cohuck@redhat.com \
--cc=drjones@redhat.com \
--cc=f4bug@amsat.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
/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 NNTP newsgroup(s).