[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v16 96/99] tests/qtest: split the cdrom-test into arm/aarch64
From: |
Alex Bennée |
Subject: |
Re: [PATCH v16 96/99] tests/qtest: split the cdrom-test into arm/aarch64 |
Date: |
Tue, 08 Jun 2021 16:35:24 +0100 |
User-agent: |
mu4e 1.5.13; emacs 28.0.50 |
Thomas Huth <thuth@redhat.com> writes:
> On 08/06/2021 16.27, Alex Bennée wrote:
>> Richard Henderson <richard.henderson@linaro.org> writes:
>>
>>> On 6/4/21 8:53 AM, Alex Bennée wrote:
>>>> The assumption that the qemu-system-aarch64 image can run all 32 bit
>>>> machines is about to be broken...
>>>
>>> Um, what?
>> Really what we want is to probe the -M (machines) that a binary
>> supports rather than just barfing the test because we've built a QEMU
>> that doesn't support all the random 32 bit machines.
>>
>>> r~
>>>
>>>
>>>
>>> and besides it's not likely this is
>>>> improving out coverage by much. Test the "virt" machine for both arm
>>>> and aarch64 as it can be used by either architecture.
>> I think this point still stands though, I don't think we get much
>> from
>> running the cdrom test with realview et all on qemu-system-aarch64.
>
> In a lot of CI pipelines, we are either building aarch64 or arm, but
> not both, so I think it might be good to keep the tests in here.
We do test instantiating the cdrom with -M virt, exactly how many extra
lines of coverage do we get for the rest?
>
> Thomas
--
Alex Bennée
- Re: [PATCH v16 77/99] target/arm: arch_dump: restrict ELFCLASS64 to AArch64, (continued)
- [PATCH v16 36/99] target/arm: move arm_mmu_idx* to cpu-mmu, Alex Bennée, 2021/06/04
- [PATCH v16 86/99] target/arm: cpu-pauth: change arm_cpu_pauth_finalize name and sig, Alex Bennée, 2021/06/04
- [PATCH v16 96/99] tests/qtest: split the cdrom-test into arm/aarch64, Alex Bennée, 2021/06/04
- Re: [PATCH v16 96/99] tests/qtest: split the cdrom-test into arm/aarch64, Richard Henderson, 2021/06/05
- Re: [PATCH v16 96/99] tests/qtest: split the cdrom-test into arm/aarch64, Alex Bennée, 2021/06/08
- Re: [PATCH v16 96/99] tests/qtest: split the cdrom-test into arm/aarch64, Thomas Huth, 2021/06/08
- Re: [PATCH v16 96/99] tests/qtest: split the cdrom-test into arm/aarch64, Richard Henderson, 2021/06/08
- Re: [PATCH v16 96/99] tests/qtest: split the cdrom-test into arm/aarch64, Thomas Huth, 2021/06/08
- Re: [PATCH v16 96/99] tests/qtest: split the cdrom-test into arm/aarch64,
Alex Bennée <=
- Re: [PATCH v16 96/99] tests/qtest: split the cdrom-test into arm/aarch64, Thomas Huth, 2021/06/08
- Re: [PATCH v16 96/99] tests/qtest: split the cdrom-test into arm/aarch64, John Snow, 2021/06/08
Re: [PATCH v16 96/99] tests/qtest: split the cdrom-test into arm/aarch64, John Snow, 2021/06/08
[PATCH v16 75/99] target/arm: make is_aa64 and arm_el_is_aa64 a macro for !TARGET_AARCH64, Alex Bennée, 2021/06/04
[PATCH v16 80/99] target/arm: tcg-sve: import narrow_vq and change_el functions, Alex Bennée, 2021/06/04