From 18a8b8df03f25e696a0400f2b4ccd0f8ef6ea478 Mon Sep 17 00:00:00 2001 From: Jeremi Piotrowski Date: Fri, 8 Sep 2023 15:14:56 +0200 Subject: [PATCH] runtime: Remove redundant check in checkPCIeConfig There is no way for this branch to be hit, as port is only set when it is different than config.NoPort. Signed-off-by: Jeremi Piotrowski (cherry picked from commit bfc93927fbc4fcd13ec2f182091e14ded4be7760) --- src/runtime/pkg/katautils/config.go | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/runtime/pkg/katautils/config.go b/src/runtime/pkg/katautils/config.go index 0a2c8022c..a4e17b239 100644 --- a/src/runtime/pkg/katautils/config.go +++ b/src/runtime/pkg/katautils/config.go @@ -1712,10 +1712,6 @@ func checkPCIeConfig(coldPlug config.PCIePort, hotPlug config.PCIePort, machineT if hotPlug != config.NoPort { port = hotPlug } - if port == config.NoPort { - return fmt.Errorf("invalid vfio_port=%s setting, use on of %s, %s, %s", - port, config.BridgePort, config.RootPort, config.SwitchPort) - } if port == config.BridgePort || port == config.RootPort || port == config.SwitchPort { return nil }