On Wed, Nov 26, 2025 at 9:16 AM Andrew Bailey <[email protected]> wrote:
>
>
> + if (
> + test_run.ctx.topology.type != LinkTopology.NO_LINK
> + and test_run.ctx.sut_node.ports[0].config.os_driver_for_dpdk
> == "vfio-pci"
> + ):
> + test_run.ctx.sut_node.main_session.send_command("modprobe
> vfio")
> + test_run.ctx.sut_node.main_session.send_command("modprobe
> vfio-pci")
> +
> if test_run.config.use_virtual_functions:
> test_run.ctx.topology.instantiate_vf_ports()
>
>
>
Is there a need to check that test_run.ctx.sut_node.ports[0] is not None?
Also, what is your opinion of an os==linux check? By my recollection this
variable should be accessible at this stage. It would be okay to kick the
can on this aspect of supporting OS other than linux, but let's only do so
if we have an actual reason. :)
