Thomas Huth <th...@redhat.com> writes:
> On 10/12/2024 21.43, Alex Bennée wrote:
>> Now we have virtio-gpu Vulkan support lets add a test for it.
>> Currently this is using images build by buildroot:
>> https://lists.buildroot.org/pipermail/buildroot/2024-December/768196.html
>> Signed-off-by: Alex Bennée <alex.ben...@linaro.org>
<snip>
>> + self.vm.set_console()
>> + kernel_command_line = (self.KERNEL_COMMON_COMMAND_LINE +
>> + 'console=ttyAMA0 root=/dev/vda')
>> + self.require_accelerator("tcg")
>
> Same, please move to the beginning since it can skip the test.
>
>> + self.vm.add_args("-accel", "tcg")
Actually this could run under KVM if we have it for Aarch64. Can we
represent that?
>> + self.vm.add_args("-cpu", "neoverse-v1,pauth-impdef=on")
I guess in that case we'd use -cpu host as well.
>> + self.vm.add_args("-machine",
>> + "virt,virtualization=on,"
>> + "gic-version=max",
>> + '-kernel', kernel_path,
>> + '-append', kernel_command_line)
>> + self.vm.add_args("-smp", "2", "-m", "2048")
>> + self.vm.add_args("-device",
>> "virtio-gpu-gl-pci,hostmem=4G,blob=on,venus=on")
>> + self.vm.add_args("-display", "egl-headless")
>> + self.vm.add_args("-display", "dbus,gl=on")
>> + self.vm.add_args("-device", "virtio-blk-device,drive=hd0")
>> + self.vm.add_args("-blockdev",
>> +
>> "driver=raw,file.driver=file,node-name=hd0,read-only=on,"
>> + f"file.filename={image_path}")
>> + self.vm.add_args("--snapshot")
>
> Any reason for using double dashes just here and not for the other commands?
>
>> + try:
>> + self.vm.launch()
>> + except VMLaunchFailure as e:
>> + if "old virglrenderer, blob resources unsupported" in e.output:
>> + self.skipTest("No blob support for virtio-gpu")
>> + elif "old virglrenderer, venus unsupported" in e.output:
>> + self.skipTest("No venus support for virtio-gpu")
>> + else:
>> + self.log.info(f"un-handled launch failure: {e.output}")
>
> s/un-handled/unhandled/ ?
>
> Thomas
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
