I could have sworn I had this is a previous iteration of the patches
but I guess it got lost in a re-base. As we are going to call
vulkaninfo to probe for "bad" drivers we need to skip if the binary
isn't available.

Fixes: 9f7e493d11 (tests/functional: skip vulkan tests with nVidia)
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
 tests/functional/test_aarch64_virt_gpu.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tests/functional/test_aarch64_virt_gpu.py b/tests/functional/test_aarch64_virt_gpu.py
index f19a47f8b6..314d994a7a 100755
--- a/tests/functional/test_aarch64_virt_gpu.py
+++ b/tests/functional/test_aarch64_virt_gpu.py
@@ -115,6 +115,7 @@ def test_aarch64_virt_with_virgl_blobs_gpu(self):
         self._run_virt_weston_test("glmark2-wayland -b:duration=1.0")
 
     @skipIfMissingCommands('zstd')
+    @skipIfMissingCommands('vulkaninfo')
     def test_aarch64_virt_with_vulkan_gpu(self):
 
         self.require_device('virtio-gpu-gl-pci')
-- 
2.39.5

Missing "i" in "mssing" in the subject.

On 12/03/2025 20.03, Alex Bennée wrote:
> I could have sworn I had this is a previous iteration of the patches
> but I guess it got lost in a re-base. As we are going to call
> vulkaninfo to probe for "bad" drivers we need to skip if the binary
> isn't available.
> 
> Fixes: 9f7e493d11 (tests/functional: skip vulkan tests with nVidia)
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
>   tests/functional/test_aarch64_virt_gpu.py | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/tests/functional/test_aarch64_virt_gpu.py b/tests/functional/test_aarch64_virt_gpu.py
> index f19a47f8b6..314d994a7a 100755
> --- a/tests/functional/test_aarch64_virt_gpu.py
> +++ b/tests/functional/test_aarch64_virt_gpu.py
> @@ -115,6 +115,7 @@ def test_aarch64_virt_with_virgl_blobs_gpu(self):
>           self._run_virt_weston_test("glmark2-wayland -b:duration=1.0")
>   
>       @skipIfMissingCommands('zstd')
> +    @skipIfMissingCommands('vulkaninfo')
>       def test_aarch64_virt_with_vulkan_gpu(self):

The check_output call in this function is already wrapped with a try-except 
statement, isn't that enough already?

  Thomas

On 13/03/2025 08.13, Thomas Huth wrote:
> 
> Missing "i" in "mssing" in the subject.
> 
> On 12/03/2025 20.03, Alex Bennée wrote:
>> I could have sworn I had this is a previous iteration of the patches
>> but I guess it got lost in a re-base. As we are going to call
>> vulkaninfo to probe for "bad" drivers we need to skip if the binary
>> isn't available.
>>
>> Fixes: 9f7e493d11 (tests/functional: skip vulkan tests with nVidia)
>> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
>> ---
>>   tests/functional/test_aarch64_virt_gpu.py | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/tests/functional/test_aarch64_virt_gpu.py b/tests/functional/ 
>> test_aarch64_virt_gpu.py
>> index f19a47f8b6..314d994a7a 100755
>> --- a/tests/functional/test_aarch64_virt_gpu.py
>> +++ b/tests/functional/test_aarch64_virt_gpu.py
>> @@ -115,6 +115,7 @@ def test_aarch64_virt_with_virgl_blobs_gpu(self):
>>           self._run_virt_weston_test("glmark2-wayland -b:duration=1.0")
>>       @skipIfMissingCommands('zstd')
>> +    @skipIfMissingCommands('vulkaninfo')
>>       def test_aarch64_virt_with_vulkan_gpu(self):
> 
> The check_output call in this function is already wrapped with a try-except 
> statement, isn't that enough already?

Hmm, looks like a missing vulkaninfo is giving some other kind of exception, 
so we need this patch indeed. I'll queue it for my pull request today.

  Thomas