|
From: | Philippe Mathieu-Daudé |
Subject: | Re: [PATCH 02/12] tests/qtest: Do not run lsi53c895a test if device is not present |
Date: | Mon, 6 Feb 2023 19:52:34 +0100 |
User-agent: | Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.6.1 |
On 6/2/23 18:46, Fabiano Rosas wrote:
Philippe Mathieu-Daudé <philmd@linaro.org> writes:On 6/2/23 16:04, Fabiano Rosas wrote:The tests are built once for all the targets, so as long as one QEMU binary is built with CONFIG_LSI_SCSI_PCI=y, this test will run. However some binaries might not include the device. So check this again in runtime. Signed-off-by: Fabiano Rosas <farosas@suse.de> --- tests/qtest/fuzz-lsi53c895a-test.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/tests/qtest/fuzz-lsi53c895a-test.c b/tests/qtest/fuzz-lsi53c895a-test.c index 392a7ae7ed..a9254b455d 100644 --- a/tests/qtest/fuzz-lsi53c895a-test.c +++ b/tests/qtest/fuzz-lsi53c895a-test.c @@ -112,6 +112,10 @@ static void test_lsi_do_dma_empty_queue(void)int main(int argc, char **argv){ + if (!qtest_has_device("lsi53c895a")) { + return 0; + }Shouldn't we update Kconfig to now add the test unconditionally?
(I meant meson.build, not Kconfig).
Squashing:I think we'd still want to not build this test if nothing selected CONFIG_LSI_SCSI_PCI.
Yeah, no need to waste resources building/testing if not available. OTOH the qtest's meson.build is too complicated.
[Prev in Thread] | Current Thread | [Next in Thread] |