[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 04/22] check: Only test isa-testdev when it i
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v2 04/22] check: Only test isa-testdev when it is compiled in |
Date: |
Mon, 13 Aug 2018 13:31:24 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 |
On 10/08/2018 17:17, Juan Quintela wrote:
> Paolo Bonzini <address@hidden> wrote:
>> On 10/08/2018 13:08, Juan Quintela wrote:
>>>> Emanuele's qos-test would do this automatically. It would be nicer to
>>>> convert the whole ISA subsystem to use libqos and qos-test...
>>> Can we get this in until we get some volunteer for that other work?
>>> /me full of things on his plate at the moment.
>>
>> Sure, but since this is RFC, I commented. :)
>
> I didn't expect less of you O:-)
>
>> With qos-test, all tests
>> end up in a single binary that is generic for all QEMU targets, and
>> there's no need to synchronize the configuration with the testing.
>>
>> The disadvantage of doing it at the Makefile level is that it relies on
>> the devices being either in all targets or none. For example
>> CONFIG_ISA_TESTDEV cannot be removed from just
>> default-configs/ppc-softmmu.mak if you are building both a PPC and an
>> x86 target.
Anyway, I'm not opposed to merging this part, it's just that they may be
unnecessary in the future. The virtio split is more interesting to me,
and I am very happy that you are doing it!
Paolo
- Re: [Qemu-devel] [PATCH v2 02/22] config: CONFIG_SERIAL* is already in pci.mak, (continued)
[Qemu-devel] [PATCH v2 03/22] check: Use land/lor when possible, Juan Quintela, 2018/08/08
[Qemu-devel] [PATCH v2 04/22] check: Only test isa-testdev when it is compiled in, Juan Quintela, 2018/08/08
[Qemu-devel] [PATCH v2 01/22] configure: We don't want to clean configuration files, Juan Quintela, 2018/08/08
[Qemu-devel] [PATCH v2 07/22] check: Only test ivshm when it is compiled in, Juan Quintela, 2018/08/08
[Qemu-devel] [PATCH v2 05/22] check: Only test usb-xhci-nec when it is compiled in, Juan Quintela, 2018/08/08
[Qemu-devel] [PATCH v2 06/22] i386-softmmu: Configuration is identical to x86_64-softmmu, Juan Quintela, 2018/08/08
[Qemu-devel] [PATCH v2 09/22] virtio: Remove unneeded includes, Juan Quintela, 2018/08/08